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

Fix incorrect pop at end of function. #341

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jfgoog
Copy link
Contributor

@jfgoog jfgoog commented May 3, 2022

No description provided.

@landley
Copy link
Owner

landley commented May 7, 2022

I'd like to add a test case demonstrating the failure this fixes. Do you have one handy?

@jfgoog
Copy link
Contributor Author

jfgoog commented May 7, 2022

https://android.googlesource.com/platform/ndk/+/refs/heads/master/build/ndk-build

Conditional nested declaration of the log() function, around line 103.

@enh-google
Copy link
Collaborator

https://android.googlesource.com/platform/ndk/+/refs/heads/master/build/ndk-build

Conditional nested declaration of the log() function, around line 103.

gittiles (and internal cs/) tip you might not have come across: if you click on the line number, it'll give you a link to that line, such as https://android.googlesource.com/platform/ndk/+/refs/heads/master/build/ndk-build#103

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.

None yet

3 participants