From owner-svn-src-head@freebsd.org Mon Aug 29 18:16:35 2016 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id D0DA9BC74B6 for ; Mon, 29 Aug 2016 18:16:35 +0000 (UTC) (envelope-from mailing-machine@vniz.net) Received: from mail-lf0-f51.google.com (mail-lf0-f51.google.com [209.85.215.51]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 60E7CC8E for ; Mon, 29 Aug 2016 18:16:35 +0000 (UTC) (envelope-from mailing-machine@vniz.net) Received: by mail-lf0-f51.google.com with SMTP id f93so106768954lfi.2 for ; Mon, 29 Aug 2016 11:16:34 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:subject:to:references:cc:from:message-id:date :user-agent:mime-version:in-reply-to:content-transfer-encoding; bh=A+i2nZvMoWawk2Ubn46rhon+wPXJ7GR3GJhMnHdDRgc=; b=JrqR6x1QWigJ5u8hM1sZkRZu5UdHc9hc/h118jYPh6kwFA/DwmYSsryWUC1o0HVgzA 81uTaWy9NdoHFquKjcyf71DglEN1lCNho6rmky3deLsKn3zJQYGeeU9ujKut0kToEDH4 nTEGOo4Y1Z+X8zQsueTqefCUehHfx7hoR+fONvnogtLTNlcs7HaCi9WWZL6eN1tE8/Uc h8TXMXh9A8ycl3OoxA/AGFH0HBIPwzIjQKfsUmnbNPJZbvryxYKOlQCzde3gAfLzBEWz fVcuuXVXMlXGXiZcLK2B6dj/GeuHwrG+aoYEGcPao5eYXynPKTI4JjZOfdGFcbUJdVu7 DCcw== X-Gm-Message-State: AE9vXwO4W8ffQGTUaaZ5I6vZ6np8qkQyCByDqoB/QBejxqsqZq44un68k1TtJ1X1BqQqHA== X-Received: by 10.25.216.26 with SMTP id p26mr6028639lfg.122.1472494592903; Mon, 29 Aug 2016 11:16:32 -0700 (PDT) Received: from [192.168.1.2] ([89.169.173.68]) by smtp.gmail.com with ESMTPSA id h68sm6674155ljh.38.2016.08.29.11.16.31 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 29 Aug 2016 11:16:32 -0700 (PDT) Subject: Re: svn commit: r304928 - in head/lib/libc: amd64/sys i386/sys sys To: Konstantin Belousov , John Baldwin References: <201608272303.u7RN3N0D078505@repo.freebsd.org> <1595604.93PBdSz0kX@ralph.baldwin.cx> <20160829065813.GP83214@kib.kiev.ua> <18206408.GMLM77D01s@ralph.baldwin.cx> <20160829180454.GU83214@kib.kiev.ua> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org From: Andrey Chernov Message-ID: Date: Mon, 29 Aug 2016 21:16:29 +0300 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <20160829180454.GU83214@kib.kiev.ua> Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 29 Aug 2016 18:16:35 -0000 On 29.08.2016 21:04, Konstantin Belousov wrote: > On Mon, Aug 29, 2016 at 08:46:47PM +0300, Andrey Chernov wrote: >> Either we implement this wrapper or left all things as is, we need to >> document internal errno clearing additionally, to not make people wonder >> why errno becomes 0, probably with the mention that program should not >> relay on this obsoleted implementation feature. > It was done in r304928. John want to revert this change, it seems. > Previous text is documented everywhere and describing usual good practice, so it should remains in place, in that means r304928 should be reverted, because replace proper way of doing things with obsoleted feature description. What I suggest is not _replace_ old text with new, but _add_ new text as describing current and not recommended way, with the direct mention that this feature is obsoleted and not recommended for relay on it.