lilypond-devel
[Top][All Lists]
Advanced

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

Re: more comments (issue 13051044)


From: frederic . bron . 1995
Subject: Re: more comments (issue 13051044)
Date: Tue, 20 Aug 2013 10:58:03 +0000


https://codereview.appspot.com/13051044/diff/1/flower/include/std-string.hh
File flower/include/std-string.hh (right):

https://codereview.appspot.com/13051044/diff/1/flower/include/std-string.hh#newcode40
flower/include/std-string.hh:40: typedef size_t ssize; ///< I believe it
is in some c... header.
Indeed, this looks messy.  in addition to these two, there's ssize_t.
I think that using just size_t is what we should do.

No: ssize_t is the signed version of size_t used for file streams. It is
important that is remains signed. I looked at it and saw that it is used
because we use the C FILE for i/o which we shouldn't use in C++. The
equivalent in standard C++ is streamsize.

https://codereview.appspot.com/13051044/



reply via email to

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