[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [lmi] C++ modernization
From: |
Vadim Zeitlin |
Subject: |
Re: [lmi] C++ modernization |
Date: |
Wed, 11 Jan 2017 22:10:46 +0100 |
On Tue, 10 Jan 2017 15:41:30 +0000 Greg Chicares <address@hidden> wrote:
[...]
GC> But, once we get past those comic asides, we come to a real problem:
GC>
GC> /opt/lmi/src/lmi[0]$make $coefficiency check_concinnity
GC> make[1]: Entering directory '/opt/lmi/src/lmi'
GC> make[2]: 'test_coding_rules.exe' is up to date.
GC> make[1]: Leaving directory '/opt/lmi/src/lmi'
GC> Problems detected by xmllint:
GC> Miscellaneous problems:
GC> File 'test_coding_rules.cpp' must not include 'config.hpp'.
GC>
GC> Happily we observe that the regex flavor in this source is similar
GC> enough to vim's that we can just type this:
GC>
GC> /# *include *[<\"]config.hpp[>\"]
GC>
GC> and find six lines that now match.
GC>
GC> I could re-cook the literals, but that's nasty. I thought of making
GC> the ten-character string {config.hpp} a macro, because it looks like
GC> it's not quoted...huh?...no, it's just that I haven't upgraded to
GC> vim++11 so relying on highlighting misled me. Maybe I could rewrite
GC> it as /[#]config.hpp/ ...
GC>
GC> ...which led me to try to write a regex to match the set of all
GC> regexes that do not match themselves, so that we could automate the
GC> detection of problems like this...and to consider whether it would
GC> match itself...
GC>
GC> ...and then I awoke and did what I hope is the sensible thing, but
GC> maybe you should review it nevertheless.
I just wanted to confirm that I did review it and I agree that just
excluding test_coding_rules.cpp from this check is by far the simplest and,
hence, the best solution to this problem.
And please accept my traditional excuses for failing to run the test after
these changes,
VZ
- [lmi] Replacing boost with std C++11 [Was: Fix value_cast defect shown by the unit test], (continued)
- [lmi] Replacing boost with std C++11 [Was: Fix value_cast defect shown by the unit test], Greg Chicares, 2017/01/09
- Re: [lmi] Replacing boost with std C++11 [Was: Fix value_cast defect shown by the unit test], Vadim Zeitlin, 2017/01/09
- [lmi] C++ modernization [Was: Replacing boost with std C++11], Greg Chicares, 2017/01/09
- Re: [lmi] C++ modernization [Was: Replacing boost with std C++11], Greg Chicares, 2017/01/10
- [lmi] Transient git resource unavailability [Was: C++ modernization], Greg Chicares, 2017/01/10
- Re: [lmi] C++ modernization, Vadim Zeitlin, 2017/01/10
- Re: [lmi] C++ modernization, Greg Chicares, 2017/01/10
- Re: [lmi] C++ modernization [Was: Replacing boost with std C++11], Greg Chicares, 2017/01/10
- Re: [lmi] C++ modernization, Vadim Zeitlin, 2017/01/10
- Re: [lmi] C++ modernization, Greg Chicares, 2017/01/10
- Re: [lmi] C++ modernization,
Vadim Zeitlin <=
- [lmi] static_assert and :argdo [Was: Replacing boost with std C++11], Greg Chicares, 2017/01/10
- Re: [lmi] static_assert and :argdo [Was: Replacing boost with std C++11], Vadim Zeitlin, 2017/01/10
- Re: [lmi] static_assert and :argdo [Was: Replacing boost with std C++11], Greg Chicares, 2017/01/11
- Re: [lmi] static_assert and :argdo [Was: Replacing boost with std C++11], Vadim Zeitlin, 2017/01/11
- Re: [lmi] static_assert and :argdo [Was: Replacing boost with std C++11], Greg Chicares, 2017/01/11
- Re: [lmi] Replacing boost with std C++11 [Was: Fix value_cast defect shown by the unit test], Greg Chicares, 2017/01/11
- Re: [lmi] Replacing boost with std C++11 [Was: Fix value_cast defect shown by the unit test], Greg Chicares, 2017/01/11
- Re: [lmi] Replacing boost with std C++11 [Was: Fix value_cast defect shown by the unit test], Vadim Zeitlin, 2017/01/11
- Re: [lmi] Replacing boost with std C++11, Greg Chicares, 2017/01/20
- Re: [lmi] Replacing boost with std C++11, Vadim Zeitlin, 2017/01/20