Skip to content

Fix some strict gcc-12 warnings#16375

Closed
bernd-edlinger wants to merge 1 commit intoopenssl:OpenSSL_1_1_1-stablefrom
bernd-edlinger:fix_gcc_12_warnings_1
Closed

Fix some strict gcc-12 warnings#16375
bernd-edlinger wants to merge 1 commit intoopenssl:OpenSSL_1_1_1-stablefrom
bernd-edlinger:fix_gcc_12_warnings_1

Conversation

@bernd-edlinger
Copy link
Member

@bernd-edlinger bernd-edlinger commented Aug 22, 2021

this is for the 1.1.1 branch.
apparently the master branch was already fixed.

@bernd-edlinger bernd-edlinger added the branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL) label Aug 22, 2021
@t8m t8m added triaged: bug The issue/pr is/fixes a bug approval: done This pull request has the required number of approvals labels Aug 24, 2021
@openssl-machine
Copy link
Collaborator

openssl-machine commented Aug 25, 2021

24 hours has passed since 'approval: done' was set, but this PR has failing CI tests. Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually.

@bernd-edlinger
Copy link
Member Author

bernd-edlinger commented Aug 25, 2021

@t-j-h the tests did pass, why does the bot complain?

@t8m t8m added approval: ready to merge The 24 hour grace period has passed, ready to merge and removed approval: done This pull request has the required number of approvals labels Aug 25, 2021
openssl-machine pushed a commit that referenced this pull request Aug 25, 2021
@bernd-edlinger @t8m
Fix some strict gcc-12 warnings
18622c7 
Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from #16375)
@t8m
Copy link
Member

t8m commented Aug 25, 2021

Merged to 1.1.1 branch. Thank you.

@t8m t8m closed this Aug 25, 2021
baentsch added a commit to open-quantum-safe/openssl that referenced this pull request Dec 15, 2021
@baentsch @beldmit
@DDvO @levitte @t8m @mattcaswell @tim-hitchins-ekkosense @mspncp @arminfuerst @bernd-edlinger @jay @kaduk @bsdjhb @zjd87 @jwalch @mszabo @romen @mamapanda @paulidale @dongbeiouba @NanXiao @yursha @mohd-akram @dcoombs @tmshort @niclasr @slontis @tomato42 @iguessthislldo @devnexen @bonniegong @botovq @tiran @jpboivin @p-steuer @goodpaperman @larsimmisch-adobe @luyahan @davidben @ueno @ischwarze @bbbrumley @ifranzki @uudiin @zhaozg @Dani-Hub @ellert @lprimak @PeiweiHu @x2018 @allanjude
Merging OpenSSL 1.1.1m (#346