[PATCH] Beautified MacroLogFeature

[Allen Winter]

On Friday 20 February 2009 3:45:32 am Sebastian Trüg wrote:
> Now that I use my own copy in K3b I changed it a bit again. Just in case that 
> anyone is interested I attach the new version.
> 
We are very interested.
Just want to make something new we can all be happy with.
I agree with you that the current formatting needs some improvements.

I have played with this macro a bit over the years and haven't found
a perfect solution yet.  Hopefully, we can make something better together.

-Allen

> Cheers,
> Sebastian
> 
> On Thursday 19 February 2009 23:57:54 Loïc Corbasson wrote:
> > On Thu, Feb 19, 2009 at 21:24, Alexander Neundorf <neundorf at kde.org> wrote:
> > > On Thursday 19 February 2009, Sebastian Trüg wrote:
> > > > On Thursday 19 February 2009 18:23:01 Sebastian Trüg wrote:
> > > > > > I agree with Allen -- when I compile a module, the information "you
> > >
> > > are
> > >
> > > > > > missing this and that" is much more important than the information
> > > > > > "I found this and that". For the missing stuff I might have to take
> > > > > > action. For the found stuff there is no problem, so there is little
> > > > > > point in filling 3 screens with "I found this and that, so you'll
> > > > > > get such and such features"... we don't list the features that one
> > > > > > gets
> > >
> > > in
> > >
> > > > > > all cases, either
> > > > > >
> > > > > > :-) I think all that's needed is a short confirmation that "foo"
> > > > > > : was found
> > > > > >
> > > > > > after you installed it.
> > > > >
> > > > > well, OK then. I will just use my own local copy for K3b then,
> > > > > because frankly, I really don't like the output as it is now.
> > > >
> > > > Sorry, this sounds way to aggressive. And that was not my intention.
> > > > What
> > >
> > > I
> > >
> > > > should have written is:
> > >
> > > No problem.
> > > Actually I agree with you, and also also agree that your version looks
> > > nicer.
> > > But one of the things was that there should be only one line per package,
> > > so
> > > it's easy to grep for stuff.
> > > And it's also correct that the things which were found shouldn't take too
> > > much
> > > space.
> > >
> > > Alex
> >
> > How about making this an option then, that you can switch on and off, and
> > which is non-verbose like today as a default? I too like Sebastian's work
> > and find it very useful -- but I understand that it doesn't fit all use
> > cases, e.g. making grepping difficult.
> >
> > Cheers,
> > Loïc
> 
>