search

boostorg / locale

Boost.Locale
Boost Software License 1.0
31 stars 70 forks source link

Use Boost.Predef to detect Linux and remove trailing spaces #89

Closed Flamefire closed 2 years ago

Flamefire commented 2 years ago

The __linux define may not be set. __linux__ or linux mayb be used. So use Boost.Predef BOOST_OS_LINUX instead. Closes https://github.com/boostorg/locale/pull/49

Removing the trailing spaces in a single commit avoids having git highlight them as possible issues on every commit.

codecov[bot] commented 2 years ago

Codecov Report

Merging #89 (7677b1b) into develop (cf016db) will not change coverage. The diff coverage is 76.87%.

@@           Coverage Diff            @@
##           develop      #89   +/-   ##
========================================
  Coverage    79.92%   79.92%           
========================================
  Files           76       76           
  Lines         5944     5944           
========================================
  Hits          4751     4751           
  Misses        1193     1193
Impacted Files Coverage Δ
include/boost/locale/conversion.hpp 100.00% <ø> (ø)
include/boost/locale/generator.hpp 100.00% <ø> (ø)
include/boost/locale/localization_backend.hpp 100.00% <ø> (ø)
include/boost/locale/util.hpp 0.00% <0.00%> (ø)
src/encoding/codepage.cpp 92.50% <ø> (ø)
src/encoding/conv.hpp 88.88% <ø> (ø)
src/posix/codecvt.cpp 84.42% <ø> (ø)
src/shared/format.cpp 0.00% <0.00%> (ø)
src/shared/ids.cpp 90.00% <ø> (ø)
src/shared/mo_lambda.hpp 100.00% <ø> (ø)
... and 63 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update cf016db...7677b1b. Read the comment docs.