From mboxrd@z Thu Jan 1 00:00:00 1970 From: AKASHI Takahiro Date: Wed, 27 Nov 2019 10:27:07 +0900 Subject: [U-Boot] [PATCH v3 06/16] linux/time.h: include vsprintf.h In-Reply-To: References: <20191113004502.29986-1-takahiro.akashi@linaro.org> <20191113004502.29986-7-takahiro.akashi@linaro.org> <6cce30a5-7e79-9c49-2d78-a3a38da7e143@gmx.de> Message-ID: <20191127012706.GI22427@linaro.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Heinrich, On Tue, Nov 26, 2019 at 08:31:18AM +0100, Heinrich Schuchardt wrote: > On 11/26/19 4:56 AM, Heinrich Schuchardt wrote: > >On 11/13/19 1:44 AM, AKASHI Takahiro wrote: > >>Without this commit, time.h possibly causes a build error as > >>asctime_r() uses sprintf(). > > > >asctime_r() is not a Linux symbol (as of next-20191119) I simply don't get your point here. > > > >ctime_r() and asctime_r() are defined as inline functions. ctime_r() is > >used in multiple places and so we may end up duplicating code. So I > >would prefer the inline functions in time.h to be moved to a separate C > >file in lib/. Basically I'm reluctant to do so. I have never touched ctime_r() nor asctime_r() as they were introduced by Wolfgang in 2002. Since then, nobody complained. So why now? > >Best regards > > > >Heinrich > > Could it be that in one of your C files you simply didn't follow the > U-Boot coding style convention to include common.h first I will address this issue in general in the future. > and we don't > need this patch at all? I commented against this above. I believe that *hidden* dependency of include files should not be exposed to users. In this case, for example, any code that uses asctime_r() should not be bothered with such a dependency as the code doesn't know what functions asctime_r() internally uses or even if that function is inline or not. I also mentioned this in another thread in different words: https://lists.denx.de/pipermail/u-boot/2019-November/391959.html Thanks, -Takahiro Akashi > https://www.denx.de/wiki/U-Boot/CodingStyle#Include_files > > Best regards > > Heinrich