search

bloomberg / bde

Basic Development Environment - a set of foundational C++ libraries used at Bloomberg.
Apache License 2.0
1.67k stars 316 forks source link

fix __LINE__ issues #265

Closed seanbaxter closed 4 years ago

seanbaxter commented 4 years ago

https://github.com/bloomberg/bde/pull/262

I'd like the problem described here resolved for all affected tests. It's one of the few things remaining preventing my compiler from hitting 100% test compliance. Thanks.

cppguru commented 4 years ago

bslmt_readerwriterlockassert.t.cpp bdls_testutil.t.cpp ball_log.t.cpp ball_logthrottle.t.cpp

are going to be addressed by me in a separate pull request. Alisdair in the meantime will either fix the remaining "old style" names or not in #262, it is up to him, as I have looked at the whole file and it already has both style constants. :( Which means this change set did not introduce an inconsistency.

seanbaxter commented 4 years ago

Works! thx.