On Thu, 2005-07-14 at 19:27 +0200, nick vajberg wrote:
> > On Jul 14, 2005, at 11:59 AM, John Peacock wrote:
> > >
> > > svn_log(DEBUG, logger, message);
> > > svn_log(INFO, logger, message);
> > > svn_log(WARN, logger, message);
> > > svn_log(ERROR, logger, message);
> > > svn_log(FATAL, logger, message);
> > >
> > > The log level is an *variable* of logging API, not
> > a top level
> > > object. There is really no justification for
> > seperate functions,
> > > since they should all be using the same lower
> > level code. If you
> > > really like your signatures, there could always be
> > syntactic sugar
> > > #define's.
> >
> > Agreed, this is how apache logging API does it.
>
> I must disagree with
>
> svn_log(DEBUG, logger, "problem")
>
> beeing more readable than
>
> svn_log_debug(logger, "problem")
>
> More importantly: Quite a few developers beeing used
> to the existing logging frameworks expects these
> methods to be around. It has become something of a
> de-facto standard (ref. my other post)
This is *such* a bikeshed.
I prefer for my bikeshed to be more of a reddish color... something like
http://tomato.bikeshed.com/
-Fitz
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Jul 14 20:11:05 2005