Skip to content

Conversation

@dk2k
Copy link
Contributor

@dk2k dk2k commented Aug 2, 2024

TIKA-4290: Need to use returned value of substring()

@THausherr
Copy link
Contributor

THausherr commented Aug 3, 2024

There are two typos

@THausherr
Copy link
Contributor

Do you have an example where this part of the code is used, i.e. did you hit a file where it happened? Or did you look at the source code and you or the IDE noticed the bug?

@dk2k
Copy link
Contributor Author

dk2k commented Aug 3, 2024

@THausherr It's result of code inspection, I've got more to show in additional PRs.
I haven't hit this issue while using softaware.
Another option is to remove entire if {} as it has no effect

@THausherr
Copy link
Contributor

Please include "TIKA-4290: " in front of your PR title, and further PRs that deal with bugs / real risks. Don't submit PRs about formatting or style issues or pet peeves. For some reason the boilerplate text didn't appear in your PR, see e.g. #1860. I've created https://issues.apache.org/jira/browse/TIKA-4290 for you.

@dk2k dk2k changed the title Need to use returned value of substring() TIKA-4290: Need to use returned value of substring() Aug 3, 2024
@THausherr THausherr merged commit 203d401 into apache:main Aug 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants