[Gluster-devel] Message id's for components
atumball at redhat.com
Thu Dec 14 05:08:07 UTC 2017
This one is a good cleanup Xavi!
It is now merged!
On Wed, Dec 13, 2017 at 4:20 AM, Xavi Hernandez <jahernan at redhat.com> wrote:
> I've uploaded a patch  to change the way used to reserve a range of
> messages to components and to define message id's inside a component.
> The old method was error prone because adding a new component needed to
> define some macros based on previous macros (in fact there was already an
> invalid definition for a component). The definition of message id's for a
> component also required more than one modification and in some cases it was
> not correctly done.
> The new patch defines the message constants using an enum, which automates
> all assignments, preventing most of the errors. It also defines a couple of
> macros that makes it even easier to create the component ranges and message
> Any feedback will be appreciated.
>  https://review.gluster.org/19029
> Gluster-devel mailing list
> Gluster-devel at gluster.org
Amar Tumballi (amarts)
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Gluster-devel