changed import to use brackets <> for openssl #907

pull whb07 wants to merge 1 commits into bitcoin-core:master from whb07:refactor_openssl_import changing 1 files +4 −4

whb07 commented at 1:01 AM on March 21, 2021: contributor

Changes:

OpenSSL imports were changed to use angle brackets for standardization. Typically #import <foo> is used for system and or 3rd party imports, rather than the local #import "bar".

Tests: Ran on Ubuntu 20.04 fine after building with ./configure --enable-openssl-tests. Exhaustive and other flags were ran, as well as "make check".
changed import to use brackets <> for openssl as they are not local to the project 4504472269
real-or-random approved
real-or-random commented at 11:48 AM on March 21, 2021: contributor

ACK 4504472269df06b8765b134d41f86619cdcdf8f6

Indeed, it doesn't matter here but it's cleaner to use the foo <foo> syntax in this case, and I don't think there there was a specific reason to use "foo".
whb07 commented at 7:56 PM on March 21, 2021: contributor

Right! Standards make it easier to understand what’s going on.
jonasnick commented at 2:00 PM on March 26, 2021: contributor

ACK 4504472269df06b8765b134d41f86619cdcdf8f6
jonasnick merged this on Mar 26, 2021
jonasnick closed this on Mar 26, 2021
jonasnick cross-referenced this on Jun 14, 2021 from issue Upstream PRs 831, 907, 903, 889, 918, 906, 928, 922, 933, Merge bitcoin-core/secp256k1#936: Fix gen_context/ASM build on ARM, 925, 937, 926, Merge bitcoin-core/secp256k1#940: contrib: Explain explicit header guards, 850, 930, 941, 846, 947, 662, 950 by jonasnick

Contributors