Re: [PATCH v6 02/19] fsck: Introduce identifiers for fsck messages

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Johannes Schindelin <johannes.schindelin@xxxxxx> writes:

> Instead of specifying whether a message by the fsck machinery constitutes
> an error or a warning, let's specify an identifier relating to the
> concrete problem that was encountered. This is necessary for upcoming
> support to be able to demote certain errors to warnings.
>
> In the process, simplify the requirements on the calling code: instead of
> having to handle full-blown varargs in every callback, we now send a
> string buffer ready to be used by the callback.
>
> We could use a simple enum for the message IDs here, but we want to
> guarantee that the enum values are associated with the appropriate
> message types (i.e. error or warning?). Besides, we want to introduce a
> parser in the next commit that maps the string representation to the
> enum value, hence we use the slightly ugly preprocessor construct that
> is extensible for use with said parser.
>
> Signed-off-by: Johannes Schindelin <johannes.schindelin@xxxxxx>
> ---

Nicely implemented.  Looks good.
--
To unsubscribe from this list: send the line "unsubscribe git" in



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]