[MPlayer-cvslog] r33777 - in trunk/gui: interface.c ui/gtk/equalizer.c
Diego Biurrun
diego at biurrun.de
Thu Jun 30 13:31:35 CEST 2011
On Thu, Jun 30, 2011 at 10:30:54AM +0200, Ingo Brückl wrote:
> Diego Biurrun wrote on Wed, 29 Jun 2011 20:35:20 +0200:
>
> > BTW, I'd appreciate git-style "gui:" prefixes for your GUI commits.
>
> I don't think much of it unless it has become a development rule.
>
> It hardly is possible to clearly categorize all commits and the changed files
> are listed anyway. There cannot be all information available in the commit
> message.
Strangely, I never have a problem categorizing your commits, I'd label
them all as "gui:". :)
> > Didn't you run all of the GUI through uncrustify already?
>
> Until now I only did that for the files I'm currently revising / working on,
> i.e. whenever I turned towards a new file I did that one-shot formatting.
>
> But since these big, one-shot formattings didn't seem to be a winner
> (although they were a good starting point for me) I might merely uncrustify
> the remaining files soon and do the cosmetic formatting (like newlines,
> comments, reordering etc.) one by one afterwards, if that way of proceeding
> is preferred.
I say go for it.
Diego
More information about the MPlayer-cvslog
mailing list