Skip to content

Conversation

@danbev
Copy link
Contributor

This commit removes stdlib.h header as it does not seem to be used any
more.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

crypto

This commit removes stdlib.h header as it does not seem to be used any more.
@nodejs-github-botnodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. crypto Issues and PRs related to the crypto subsystem. labels Dec 19, 2017
@danbev
Copy link
ContributorAuthor

@addaleax
Copy link
Member

Landed in 85d5885

addaleax pushed a commit that referenced this pull request Dec 19, 2017
This commit removes stdlib.h header as it does not seem to be used any more. PR-URL: #17752 Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
MylesBorins pushed a commit that referenced this pull request Jan 8, 2018
This commit removes stdlib.h header as it does not seem to be used any more. PR-URL: #17752 Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
MylesBorins pushed a commit that referenced this pull request Jan 9, 2018
This commit removes stdlib.h header as it does not seem to be used any more. PR-URL: #17752 Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
@MylesBorinsMylesBorins mentioned this pull request Jan 10, 2018
MylesBorins pushed a commit that referenced this pull request Jan 23, 2018
This commit removes stdlib.h header as it does not seem to be used any more. PR-URL: #17752 Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
MylesBorins pushed a commit that referenced this pull request Jan 23, 2018
This commit removes stdlib.h header as it does not seem to be used any more. PR-URL: #17752 Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
gibfahn pushed a commit that referenced this pull request Jan 24, 2018
This commit removes stdlib.h header as it does not seem to be used any more. PR-URL: #17752 Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
@MylesBorinsMylesBorins mentioned this pull request Jan 24, 2018
MylesBorins pushed a commit that referenced this pull request Feb 11, 2018
This commit removes stdlib.h header as it does not seem to be used any more. PR-URL: #17752 Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
MylesBorins pushed a commit that referenced this pull request Feb 12, 2018
This commit removes stdlib.h header as it does not seem to be used any more. PR-URL: #17752 Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
MylesBorins pushed a commit that referenced this pull request Feb 13, 2018
This commit removes stdlib.h header as it does not seem to be used any more. PR-URL: #17752 Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
@danbevdanbev deleted the crypto_remove_unused_header_clienthello.h branch February 28, 2018 07:29
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++Issues and PRs that require attention from people who are familiar with C++.cryptoIssues and PRs related to the crypto subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@danbev@addaleax@cjihrig@gireeshpunathil@MylesBorins@nodejs-github-bot