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

Updated to fix compilation error in linux.cc #1

Merged
merged 1 commit into from Feb 14, 2014

Conversation

Projects
None yet
2 participants
@influenza
Contributor

influenza commented Feb 12, 2014

By separating the declaration of the sock_fprog structure from the
member initialization, I was able to successfully compile with gcc
version 4.6.3

Prior to this commit, I was getting the following error:

linux.cc: In function ‘void Linux_NoNewPrivs_ForkTest()’:
linux.cc:713:28: error: expected primary-expression before ‘.’ token
linux.cc:714:28: error: expected primary-expression before ‘.’ token

With this commit, compilation if fine.

Updated to fix compilation error
By separating the declaration of the sock_fprog structure from the
member initialization, I was able to successfully compile with gcc
version 4.6.3

Prior to this commit, I was getting the following error:

    linux.cc: In function ‘void Linux_NoNewPrivs_ForkTest()’:
    linux.cc:713:28: error: expected primary-expression before ‘.’ token
    linux.cc:714:28: error: expected primary-expression before ‘.’ token

With this commit, compilation if fine.
@daviddrysdale

This comment has been minimized.

Contributor

daviddrysdale commented Feb 13, 2014

Thanks -- could you fill out a contributor license agreement please?

(individual CLA, corporate CLA)

@influenza

This comment has been minimized.

Contributor

influenza commented Feb 13, 2014

Individual CLA completed! I'm 'Ronald E. Dahlgren'.

daviddrysdale added a commit that referenced this pull request Feb 14, 2014

Merge pull request #1 from influenza/dev
Updated to fix compilation error in linux.cc

@daviddrysdale daviddrysdale merged commit b9c4e34 into google:dev Feb 14, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment