From owner-cvs-all@FreeBSD.ORG  Tue May 27 21:41:16 2008
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 60D051065673;
	Tue, 27 May 2008 21:41:16 +0000 (UTC)
	(envelope-from sobomax@FreeBSD.org)
Received: from sippysoft.com (gk1.360sip.com [72.236.70.240])
	by mx1.freebsd.org (Postfix) with ESMTP id 2BEDB8FC0A;
	Tue, 27 May 2008 21:41:15 +0000 (UTC)
	(envelope-from sobomax@FreeBSD.org)
Received: from [192.168.0.40] ([204.244.149.125]) (authenticated bits=0)
	by sippysoft.com (8.13.8/8.13.8) with ESMTP id m4RLfBdN017336
	(version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO);
	Tue, 27 May 2008 14:41:12 -0700 (PDT)
	(envelope-from sobomax@FreeBSD.org)
Message-ID: <483C7FF2.6000607@FreeBSD.org>
Date: Tue, 27 May 2008 14:41:06 -0700
From: Maxim Sobolev <sobomax@FreeBSD.org>
Organization: Sippy Software, Inc.
User-Agent: Thunderbird 2.0.0.14 (Windows/20080421)
MIME-Version: 1.0
To: Xin LI <delphij@FreeBSD.org>
References: <200805272004.m4RK4SZt029194@repoman.freebsd.org>
In-Reply-To: <200805272004.m4RK4SZt029194@repoman.freebsd.org>
Content-Type: text/plain; charset=KOI8-U; format=flowed
Content-Transfer-Encoding: 7bit
Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject: Re: cvs commit: src/include string.h src/lib/libc/string
 Makefile.inc memchr.3 memrchr.c src/sys/sys param.h
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 27 May 2008 21:41:16 -0000

Xin LI wrote:
> delphij     2008-05-27 20:04:27 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:        (Branch: RELENG_6)
>     include              string.h 
>     lib/libc/string      Makefile.inc memchr.3 
>     sys/sys              param.h 
>   Added files:           (Branch: RELENG_6)
>     lib/libc/string      memrchr.c 
>   Log:
>   MFC: Add memrchr(3).

I think this is not very good idea to MFC that into stable releases 6.x 
and 7.x. The reason is that configure scripts for some packages might 
detect up this API and enable it. Which means that some binary-only 
packages build for say 6.4 won't work on 6.3 and down. AFAIK, both 
forward and backward compatibility is required (or at least desired?) 
for stable branches.

While it's "nice-to-have" feature, I see no pressing need to MFC this 
interface.

-Maxim