On Wed, Feb 10, 2016 at 04:11:40PM +0100, Thomas Monjalon wrote: > 2016-02-10 14:33, John McNamara: > > /** > > * Major version number i.e. the x in x.y.z > > */ > > -#define RTE_VER_MAJOR 2 > > +#define RTE_VER_MAJOR 16 > > > > /** > > * Minor version number i.e. the y in x.y.z > > */ > > -#define RTE_VER_MINOR 3 > > +#define RTE_VER_MINOR 4 > > > > I liked the idea of Bruce of renaming these constants to > *_YEAR and *_MONTH. > I also think RTE_VER_PATCH_LEVEL should be RTE_VER_MINOR. > And maybe that RTE_VER_PATCH_RELEASE should be RTE_VER_RELEASE. > > It shouldn't hurt applications as they must use RTE_VERSION > and rte_version(). > > Opinions?
Well, I always tend to like Bruce's ideas too, so +1 from me :-) /Bruce
