[pacman-dev] [PATCH] libalpm/remove.c - add doxygen
dpmcgee at gmail.com
Mon May 21 00:10:02 EDT 2012
On Sun, May 20, 2012 at 11:02 PM, Andrew Gregory
<andrew.gregory.8 at gmail.com> wrote:
> On Sun, 20 May 2012 22:46:58 -0500
> Dan McGee <dpmcgee at gmail.com> wrote:
>> On Sun, May 20, 2012 at 9:18 PM, <andrew.gregory.8 at gmail.com> wrote:
>> > -/** Transaction preparation for remove actions.
>> > +/**
>> > + * @brief Transaction preparation for remove actions.
>> Why the switch away from the existing format? Honestly outside of
>> alpm_list.c, we barely use @brief format anywhere. Same with addition
>> of newlines, etc.
> I switched to this format because it's what the vim doxygen toolkit
> defaults to and I just updated the old comments for consistency. I'm
> sure I could hack together a macro for the old format if you really
> prefer it.
Consistency and ability to update easily are the main things I care
about. As long as we are only using one format in a given file; I
suppose I'm fine with it (while still addressing the space alignment
More information about the pacman-dev