Re: svn commit: r352838 - head/share/man/man3

2019-09-28 Thread Ian Lepore
On Sat, 2019-09-28 at 09:37 +, Edward Tomasz Napierala wrote:
> Author: trasz
> Date: Sat Sep 28 09:37:05 2019
> New Revision: 352838
> URL: https://svnweb.freebsd.org/changeset/base/352838
> 
> Log:
>   Sort MLINKS for arb(3), and actually make them work by fixing a '='
> vs '+='
>   mixup.
>   

::sigh::  This wouldn't happen if we just always used += for all lines
in lists of things, but sadly (IMO, surrealistically), style.Makefile
explicitly prohibits that.  I'm not sure what value the prohibition is
supposed to provide, but the downsides are making this mistake over and
over again, and if you manage to avoid the mistake your consolation
prize is a diff that's less-clear because you have to change lines that
aren't actually involed in the change you're making.

-- Ian


___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r352838 - head/share/man/man3

2019-09-28 Thread Edward Tomasz Napierala
Author: trasz
Date: Sat Sep 28 09:37:05 2019
New Revision: 352838
URL: https://svnweb.freebsd.org/changeset/base/352838

Log:
  Sort MLINKS for arb(3), and actually make them work by fixing a '=' vs '+='
  mixup.
  
  MFC after:2 weeks
  Sponsored by: Klara Inc, Netflix

Modified:
  head/share/man/man3/Makefile

Modified: head/share/man/man3/Makefile
==
--- head/share/man/man3/MakefileSat Sep 28 09:22:52 2019
(r352837)
+++ head/share/man/man3/MakefileSat Sep 28 09:37:05 2019
(r352838)
@@ -33,43 +33,43 @@ MAN=arb.3 \
timeradd.3 \
tree.3
 
-MLINKS+=   arb.3 ARB8_ENTRY.3 \
-   arb.3 ARB16_ENTRY.3 \
-   arb.3 ARB32_ENTRY.3 \
+MLINKS=arb.3 ARB8_ENTRY.3 \
arb.3 ARB8_HEAD.3 \
+   arb.3 ARB16_ENTRY.3 \
arb.3 ARB16_HEAD.3 \
+   arb.3 ARB32_ENTRY.3 \
arb.3 ARB32_HEAD.3 \
arb.3 ARB_ALLOCSIZE.3 \
-   arb.3 ARB_INITIALIZER.3 \
-   arb.3 ARB_ROOT.3 \
-   arb.3 ARB_EMPTY.3 \
-   arb.3 ARB_FULL.3 \
arb.3 ARB_CURNODES.3 \
-   arb.3 ARB_MAXNODES.3 \
-   arb.3 ARB_NEXT.3 \
-   arb.3 ARB_PREV.3 \
-   arb.3 ARB_MIN.3 \
-   arb.3 ARB_MAX.3 \
+   arb.3 ARB_EMPTY.3 \
arb.3 ARB_FIND.3 \
-   arb.3 ARB_NFIND.3 \
-   arb.3 ARB_LEFT.3 \
-   arb.3 ARB_LEFTIDX.3 \
-   arb.3 ARB_RIGHT.3 \
-   arb.3 ARB_RIGHTIDX.3 \
-   arb.3 ARB_PARENT.3 \
-   arb.3 ARB_PARENTIDX.3 \
-   arb.3 ARB_GETFREE.3 \
-   arb.3 ARB_FREEIDX.3 \
arb.3 ARB_FOREACH.3 \
arb.3 ARB_FOREACH_FROM.3 \
-   arb.3 ARB_FOREACH_SAFE.3 \
arb.3 ARB_FOREACH_REVERSE.3 \
arb.3 ARB_FOREACH_REVERSE_FROM.3 \
arb.3 ARB_FOREACH_REVERSE_SAFE.3 \
+   arb.3 ARB_FOREACH_SAFE.3 \
+   arb.3 ARB_FREEIDX.3 \
+   arb.3 ARB_FULL.3 \
+   arb.3 ARB_GETFREE.3 \
arb.3 ARB_INIT.3 \
+   arb.3 ARB_INITIALIZER.3 \
arb.3 ARB_INSERT.3 \
-   arb.3 ARB_REMOVE.3
-MLINKS=ATOMIC_VAR_INIT.3 atomic_compare_exchange_strong.3 \
+   arb.3 ARB_LEFT.3 \
+   arb.3 ARB_LEFTIDX.3 \
+   arb.3 ARB_MAX.3 \
+   arb.3 ARB_MAXNODES.3 \
+   arb.3 ARB_MIN.3 \
+   arb.3 ARB_NEXT.3 \
+   arb.3 ARB_NFIND.3 \
+   arb.3 ARB_PARENT.3 \
+   arb.3 ARB_PARENTIDX.3 \
+   arb.3 ARB_PREV.3 \
+   arb.3 ARB_REMOVE.3 \
+   arb.3 ARB_RIGHT.3 \
+   arb.3 ARB_RIGHTIDX.3 \
+   arb.3 ARB_ROOT.3
+MLINKS+=   ATOMIC_VAR_INIT.3 atomic_compare_exchange_strong.3 \
ATOMIC_VAR_INIT.3 atomic_compare_exchange_strong_explicit.3 \
ATOMIC_VAR_INIT.3 atomic_compare_exchange_weak.3 \
ATOMIC_VAR_INIT.3 atomic_compare_exchange_weak_explicit.3 \
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"