[pacman-dev] [PATCH] libalpm/remove.c - add doxygen

[Dan McGee]

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
comments, etc.).

-Dan