[pacman-dev] output issues status

[Dan McGee]

On Dec 4, 2007 12:11 PM, Xavier <shiningxc at gmail.com> wrote:
> On Mon, Dec 03, 2007 at 06:42:11PM -0600, Dan McGee wrote:
> > For now, this looks quite good enough. Great work. You did this as
> > cleanly as one would possibly be able to- the only other concern would
> > be with a interrupt, which I haven't tested.
> >
> > I'm going to let it bake on my working branch tonight (I made a few
> > small changes but nothing huge). If everyone could give it a test that
> > would be great.
> >
>
> Nothing wrong with your small changes, but it reminded me of something I
> didn't mention:
> the delayed output is only done when --debug is NOT set, because when --debug
> is used, the progress bar is disabled with noprogressbar = 1.  So in
> pm_vasprintf, the logmask will never overlap with LOG_DEBUG or LOG_FUNCTION.
>
> However, that only concerns this particular usage of pm_vasprintf, so you
> were right to add LOG_DEBUG and LOG_FUNCTION handling to pm_vasprintf. It
> just isn't used currently.

Yeah, I realized this, but one of two things should be done:
1. Copy the existing switch statement with all the options (as I ended
up doing).
2. Make an explicit comment in the code why certain statements were omitted.

-Dan