Re: [Patch] adding macro to check for a specific major/minor

Marcus Rueckert wrote:
> this is a little macro i found in the apache source.
> i used it in [1] already and it works nicely.
> Justin Erenkrantz from the apache folks said it would be ok to use it in
> subversion.
>
> [[[
> Index: subversion/include/svn_version.h
> ===================================================================
> --- subversion/include/svn_version.h (revision 14707)
> +++ subversion/include/svn_version.h (working copy)
> @@ -238,6 +238,13 @@
> */
> const svn_version_t *svn_subr_version (void);
>
> +/* little macro borrowed from the apache source code.
> + * Allows to check for a specific svn version.
> + */
> +#define SVN_VER_AT_LEAST(major,minor) \
> + ((major) < SVN_VER_MAJOR \
> + || ((major) == SVN_VER_MAJOR \
> + && (minor) <= SVN_VER_MINOR))
>
> #ifdef __cplusplus
> ]]]

Is there any developer willing to review this patch in the next day or
two? If not, I'll file an issue for it.

-- 
Michael W Thelen
It is a mistake to think you can solve any major problems just with
potatoes.       -- Douglas Adams
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org