Re: CVS commit: src/common/lib/libprop

2020-06-11 Thread Joerg Sonnenberger
On Fri, Jun 12, 2020 at 02:59:40AM +0200, Kamil Rytarowski wrote: > On 12.06.2020 02:07, Joerg Sonnenberger wrote: > > On Fri, Jun 12, 2020 at 01:28:15AM +0200, Kamil Rytarowski wrote: > >> Please list legitimate false positives. There is practically nothing > >> like that possible for using

Re: CVS commit: src/common/lib/libprop

2020-06-11 Thread Kamil Rytarowski
On 12.06.2020 02:07, Joerg Sonnenberger wrote: > On Fri, Jun 12, 2020 at 01:28:15AM +0200, Kamil Rytarowski wrote: >> Please list legitimate false positives. There is practically nothing >> like that possible for using deprecated APIs (at least kept longer >> term). Besides that, the report shall

Re: CVS commit: src/common/lib/libprop

2020-06-11 Thread Joerg Sonnenberger
On Fri, Jun 12, 2020 at 01:28:15AM +0200, Kamil Rytarowski wrote: > Please list legitimate false positives. There is practically nothing > like that possible for using deprecated APIs (at least kept longer > term). Besides that, the report shall be lowered to warning (like it > used to be for

Re: CVS commit: src/common/lib/libprop

2020-06-11 Thread Jason Thorpe
> On Jun 11, 2020, at 4:28 PM, Kamil Rytarowski wrote: > >> Repeating that statement doesn't make it true. The amount of legit >> problems found by them is dwarfed by far by the number of false >> positives seen. That's complete ignoring basic QoI issues like "where is >> this actually

Re: CVS commit: src/common/lib/libprop

2020-06-11 Thread Christos Zoulas
As Joerg mentioned, he believes that the error behavior is the way to go. I think that it is not useful behavior to error out on these warnings. If we wanted this behavior, we would not use linker warnings, we'd outright remove the deprecated symbols. christos > On Jun 11, 2020, at 6:58 PM,

Re: CVS commit: src/common/lib/libprop

2020-06-11 Thread Kamil Rytarowski
On 12.06.2020 01:11, Joerg Sonnenberger wrote: > On Fri, Jun 12, 2020 at 12:58:45AM +0200, Kamil Rytarowski wrote: >> On 12.06.2020 00:55, Christos Zoulas wrote: >>> In article <20200611222544.6d3a6f...@cvs.netbsd.org>, >>> Joerg Sonnenberger wrote: -=-=-=-=-=- Module Name:

Re: CVS commit: src/common/lib/libprop

2020-06-11 Thread Joerg Sonnenberger
On Fri, Jun 12, 2020 at 12:58:45AM +0200, Kamil Rytarowski wrote: > On 12.06.2020 00:55, Christos Zoulas wrote: > > In article <20200611222544.6d3a6f...@cvs.netbsd.org>, > > Joerg Sonnenberger wrote: > >> -=-=-=-=-=- > >> > >> Module Name: src > >> Committed By: joerg > >> Date:

Re: CVS commit: src/common/lib/libprop

2020-06-11 Thread Joerg Sonnenberger
On Thu, Jun 11, 2020 at 10:55:16PM -, Christos Zoulas wrote: > In article <20200611222544.6d3a6f...@cvs.netbsd.org>, > Joerg Sonnenberger wrote: > >-=-=-=-=-=- > > > >Module Name: src > >Committed By:joerg > >Date:Thu Jun 11 22:25:44 UTC 2020 > > > >Modified Files: > >

Re: CVS commit: src/common/lib/libprop

2020-06-11 Thread Kamil Rytarowski
On 12.06.2020 00:55, Christos Zoulas wrote: > In article <20200611222544.6d3a6f...@cvs.netbsd.org>, > Joerg Sonnenberger wrote: >> -=-=-=-=-=- >> >> Module Name: src >> Committed By:joerg >> Date:Thu Jun 11 22:25:44 UTC 2020 >> >> Modified Files: >>

Re: CVS commit: src/common/lib/libprop

2020-06-11 Thread Christos Zoulas
In article <20200611222544.6d3a6f...@cvs.netbsd.org>, Joerg Sonnenberger wrote: >-=-=-=-=-=- > >Module Name: src >Committed By: joerg >Date: Thu Jun 11 22:25:44 UTC 2020 > >Modified Files: > src/common/lib/libprop: prop_object_impl.h > >Log Message: >Unbreak clang builds by

Re: CVS commit: src/share/misc

2020-06-11 Thread Sevan Janiyan
On 11/06/2020 10:15, Sevan Janiyan wrote: > Sync with r359561 That should've been r361673 Sevan