On Thu, Nov 15, 2007 at 10:03:02PM +0100, Jean-Marc Lasgouttes wrote:
> [EMAIL PROTECTED] writes:
> 
> > Author: poenitz
> > Date: Thu Nov 15 21:04:51 2007
> > New Revision: 21631
> >
> > URL: http://www.lyx.org/trac/changeset/21631
> > Log:
> > reduce line noise
> >  Buffer::Buffer(string const & file, bool readonly)
> >     : pimpl_(new Impl(*this, FileName(file), readonly)), gui_(0)
> >  {
> > -   LYXERR(Debug::INFO) << "Buffer::Buffer()" << endl;
> > +   LYXERR(Debug::INFO, "Buffer::Buffer()");
> >  }
> 
> I don't like it much... What is the point?

- shorter to type by 10 characters,
- less visual clutter, in quite a few cases no multi-line statements
  are needed anymore,
- now if-else safe by using the  do { ... } while (0) idiom,
- we can have BOOST_FUNCTION and/or something equivalent in the
  defintion if wanted and would have even less clutter in the user code,
- easy to shortcut for timing purposes by using #define LYXERR(a, b) /*nothing*/

Andre'

Reply via email to