Date: Fri, 31 Aug 2012 03:44:52 -0600 From: Scott Long <scottl@samsco.org> To: Dimitry Andric <dimitry@andric.com> Cc: deeptech71@gmail.com, freebsd-current@freebsd.org Subject: Re: sys/dev/amr build error with Clang Message-ID: <414BFBFF-5F7C-45A5-8E0A-FC85FD39E29E@samsco.org> In-Reply-To: <50407947.50907@andric.com> References: <504068F2.6010300@gmail.com> <50407947.50907@andric.com>
index | next in thread | previous in thread | raw e-mail
On Aug 31, 2012, at 2:43 AM, Dimitry Andric <dimitry@andric.com> wrote: > On 2012-08-31 09:34, deeptech71@gmail.com wrote: >> /usr/src/sys/modules/amr/../../dev/amr/amr.c:970:1: error: function >> 'amr_periodic' is not needed and will not be emitted >> [-Werror,-Wunneeded-internal-declaration] > > The one call to get the callout to amr_periodic() started seems to have > been commented out in r239912: > > http://svnweb.freebsd.org/base/head/sys/dev/amr/amr.c?r1=239912&r2=239911&pathrev=239912 > > If the function isn't necessary anymore, it could just be deleted, or > #ifdef'd out. Fixed in r239939. Thanks for the report. Scotthelp
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?414BFBFF-5F7C-45A5-8E0A-FC85FD39E29E>