What is the use of first_capital_postprocess and last_capital_postprocess in text_postprocessors.py

[wangluyi]

In file https://github.com/open-compass/opencompass/blob/0.3.8/opencompass/utils/text_postprocessors.py, there is code snippet from line 141:

@TEXT_POSTPROCESSORS.register_module('first-capital')
def first_capital_postprocess(text: str) -> str:
    for t in text:
        if t.isupper():
            return t
    return ''


@TEXT_POSTPROCESSORS.register_module('last-capital')
def last_capital_postprocess(text: str) -> str:
    for t in text[::-1]:
        if t.isupper():
            return t
    return ''

What is exactly the use of these code? When the input text is None, it throws exception of "TypeError: 'NoneType' object is not iterable"
I modified the code like this. Is there any impact on the origin logic of the code? Thanks

@TEXT_POSTPROCESSORS.register_module('first-capital')
def first_capital_postprocess(text: str) -> str:
    if text is None or len(text) == 0:
        return ""
    for t in text:
        if t.isupper():
            return t
    return ''


@TEXT_POSTPROCESSORS.register_module('last-capital')
def last_capital_postprocess(text: str) -> str:
    if text is None or len(text) == 0:
        return ""
    for t in text[::-1]:
        if t.isupper():
            return t
    return ''