[pacman-dev] do we need debug in alpm_depcmp?

[Nagy Gabor]

> On Nov 19, 2007 6:55 AM, Dan McGee <dpmcgee at gmail.com> wrote:
> > So in short- submit a patch for *one* feature at a time, and justfiy
> > it in the commit message. Don't try to sneak things by. I do think
> > this may be a valid complaint though.
> 
> Even better? How about something similar to:
> 
> #ifdef PACMAN_DEBUG
> #    define DEBUG_LOG(...) _alpm_log(PM_LOG_DEBUG, __VA_ARGS__)
> #else
> #    define DEBUG_LOG(x)
> #endif
> 
> Then just:
> s/_alpm_log(PM_LOG_DEBUG,/DEBUG_LOG(/g
> 
> This way we could simply get rid of all debugging if we wanted to.
> Then you have both a useful version for debugging, and a "snappy"
> version that saves 1 or 2 seconds without having to output anything.

I like this idea, but:
What about user feedbacks? We will ask them to download a debugging-enabled
version of pacman?

Bye, ngaba


----------------------------------------------------
SZTE Egyetemi Könyvtár - http://www.bibl.u-szeged.hu
This mail sent through IMP: http://horde.org/imp/