lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Issue 4550: Avoid "using namespace std; " in included files (Take 2)


From: dak
Subject: Re: Issue 4550: Avoid "using namespace std; " in included files (Take 2) (issue 579240043 by address@hidden)
Date: Thu, 23 Jan 2020 03:54:28 -0800

On 2020/01/23 11:33:11, lemzwerg wrote:
> I'm quite sure that you have more experience with C++ than me,
however, I'm not
> really happy with this change since it makes the code substantially
uglier. 
> Isn't there another solution for the problem?
> 
>
https://codereview.appspot.com/579240043/diff/577360043/flower/include/file-name.hh
> File flower/include/file-name.hh (right):
> 
>
https://codereview.appspot.com/579240043/diff/577360043/flower/include/file-name.hh#newcode47
> flower/include/file-name.hh:47: std::string root_;
> This is very ugly.  What about using the `std` namespace within a
group instead?

I'd say that ship has sailed by virtue of the C++ standard committee. 
Most current changes of Dan are supposed to bring the LilyPond code base
in line with what C++ programmers are used to experiencing.  And for
better or worse, std:: most certainly has become part of that
experience.  So I am with Dan here.

https://codereview.appspot.com/579240043/



reply via email to

[Prev in Thread] Current Thread [Next in Thread]