Skip to content
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

"Solve the bug of multiple css queries with '>' #1753

Closed
wants to merge 2 commits into from

Conversation

JeffXiesk
Copy link

@JeffXiesk JeffXiesk commented Apr 23, 2022

fix #1707
As a Jsoup user, I want to query some data from the Internet using multiple css queries so that I can improve the efficiency and do it more conveniently.

@JeffXiesk JeffXiesk changed the title "Solve the bug of multiple css queries with '>' and Solve Testfailure when connection is unable "Solve the bug of multiple css queries with '>' Apr 23, 2022
@@ -141,6 +141,7 @@ private void combinator(char combinator) {

private String consumeSubQuery() {
StringBuilder sq = StringUtil.borrowBuilder();
boolean is_new_class_query = false;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not Python... Please use Java and local style

else
tq.consume();
else
else{
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't get why we're popping it off and then re-adding. Can't it just happen in one step? Seems kludgy

@jhy
Copy link
Owner

jhy commented May 17, 2022

Please make sure the PR description includes an actual description of the change and what is wrong with the current implementation. Writing it like a user story like this doesn't help me understand the change and seems like a force fit.

@jhy
Copy link
Owner

jhy commented Jun 13, 2022

Please see previous comment - not inclined to merge as is

@jhy jhy closed this Jun 13, 2022
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.

Multiple selectors for direct descendants catches indirect descendants as well
2 participants