-
Notifications
You must be signed in to change notification settings - Fork 2.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add StringUtil.joining()
and use in Element.cssSelectorComponent()
#1988
Conversation
77acc7e
to
bfccd32
Compare
bfccd32
to
c73befd
Compare
While this is a bit neater, I prefer the current version because it recycles in-use StringBuilders, vs allocating a new one every time. As this method gets called via Perhaps modify the StringUtil to implement a Collectors.joining that uses recycled builders? |
26f58a5
to
00ad0c7
Compare
# Conflicts: # pom.xml
5502bcf
to
13e0e95
Compare
@@ -5,11 +5,14 @@ | |||
|
|||
import java.net.MalformedURLException; | |||
import java.net.URL; | |||
import java.util.Arrays; | |||
import java.util.Collection; | |||
import java.util.Iterator; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you change your format settings so that we don't use wildcard imports? Prefer to list them all explicitly.
StringUtil.joining()
and use in Element.cssSelectorComponent()
Thanks, merged! |
Uncomment the
stream
block inElement
and make some improvements to it.