Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 19 Sep 2012 21:42:54 +0200
From:      Jilles Tjoelker <jilles@stack.nl>
To:        Brooks Davis <brooks@FreeBSD.org>
Cc:        svn-src-projects@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r240675 - in projects/mtree: include lib/libc/gen
Message-ID:  <20120919194254.GA75105@stack.nl>
In-Reply-To: <201209182028.q8IKS1lK056820@svn.freebsd.org>
References:  <201209182028.q8IKS1lK056820@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Sep 18, 2012 at 08:28:01PM +0000, Brooks Davis wrote:
> Author: brooks
> Date: Tue Sep 18 20:28:01 2012
> New Revision: 240675
> URL: http://svn.freebsd.org/changeset/base/240675

> Log:
>   Implement VIS_GLOB support including compatibilty symbols required by
>   overlapping flags between FreeBSD and NetBSD.

>   Add a compatibility symbol for unvis() due to NetBSD's allocation of
>   a flag value for UNVIS_END rather than just using 1.

>   Add symbol version entries for new vis and unvis functions.

> [snip]
> Modified: projects/mtree/include/vis.h
> ==============================================================================
> --- projects/mtree/include/vis.h	Tue Sep 18 20:20:29 2012	(r240674)
> +++ projects/mtree/include/vis.h	Tue Sep 18 20:28:01 2012	(r240675)
> @@ -62,6 +62,7 @@
>  #define	VIS_HTTP1866	0x200	/* http-style &#num; or &string; */
>  #define	VIS_NOESCAPE	0x400	/* don't decode `\' */
>  #define	_VIS_END	0x800	/* for unvis */
> +#define	VIS_GLOB	0x1000	/* encode glob(3) magics */
>  
>  /*
>   * unvis return codes
> 
> [snip]

Is it really necessary to be binary compatible with NetBSD in the VIS_*
constants or is it possible to keep our values for the VIS_* constants
we already have?

While symbol versioning works, it leaves additional code around forever,
which would not be necessary if compatibility had been kept.

-- 
Jilles Tjoelker



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20120919194254.GA75105>