Hi, I've been exploring unirest-java and appreciate the work you've put into this project. I noticed a small area in the param method where we might reduce the complexity slightly by directly replacing the parameter without pre-counting its occurrences.
This proposed change employs Matcher.find() to ensure the placeholder exists, followed by Matcher.replaceAll() to perform the substitution. It's a subtle tweak that I hope aligns with the project's coding practices.
Thank you for maintaining such a valuable resource for the community.
Hi, I've been exploring unirest-java and appreciate the work you've put into this project. I noticed a small area in the param method where we might reduce the complexity slightly by directly replacing the parameter without pre-counting its occurrences.
This proposed change employs
Matcher.find()
to ensure the placeholder exists, followed byMatcher.replaceAll()
to perform the substitution. It's a subtle tweak that I hope aligns with the project's coding practices.Thank you for maintaining such a valuable resource for the community.