-
Notifications
You must be signed in to change notification settings - Fork 29
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
Ensure hhvm 4.40 compatibility #40
Conversation
The needs_another_plus_plus hack is terrible. However, I must conditionally increment the pointer depending on - Is the LHS of the while's && false, then no. - Did we fall out of the loop because pointer is zero, then yes.
…nto hhvm_4_40_compat
I hope Slack is at least on HHVM 4.15, otherwise, you won't be able to use this. |
@@ -32,13 +32,12 @@ public function aliasRecentExpression(string $name): void { | |||
/** | |||
* The FROM clause of the query gets processed first, retrieving data from tables, executing subqueries, and handling joins | |||
* This is also where we build up the $columns list which is commonly used throughout the entire library to map column references to indexes in this dataset | |||
* @reviewer, we don't build up the $columns, since the variable is unused... |
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.
ah yeah, this is from an older version of this before schema was present I think!
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.
hooray for the unused variable linter!
Thank you for doing this 👏 |
The needs_another_plus_plus hack is terrible.
However, I must conditionally increment the pointer depending on