On 23.12.2016 1:39, Conrad Meyer wrote: > This was unjustified and inappropriate.
I don't think so. While being able to back it out or fix it by another way by yourself, you just doing nothing for several days. Your personal reasons of inability to act properly so long time weights less than project consistency at whole, so backing it out from anybody is justified and appropriate in such situations. > On Thu, Dec 22, 2016 at 2:30 PM, Ngie Cooper <n...@freebsd.org> wrote: >> Author: ngie >> Date: Thu Dec 22 22:30:42 2016 >> New Revision: 310433 >> URL: https://svnweb.freebsd.org/changeset/base/310433 >> >> Log: >> Revert r310138 >> >> Adding %b support to vfprintf for parity with kernel space requires >> more discussion/review. >> >> In particular, many parties were concerned over introducing a >> non-standard format qualifier to *printf(3) which didn't already >> exist in other OSes, e.g. Linux, thus making code which used %b >> harder to port to other operating systems. >> >> Requested by: many >> >> Modified: >> head/lib/libc/stdio/vfprintf.c >> >> Modified: head/lib/libc/stdio/vfprintf.c >> ============================================================================== >> --- head/lib/libc/stdio/vfprintf.c Thu Dec 22 21:56:41 2016 >> (r310432) >> +++ head/lib/libc/stdio/vfprintf.c Thu Dec 22 22:30:42 2016 >> (r310433) >> @@ -611,37 +611,6 @@ reswitch: switch (ch) { >> case 'z': >> flags |= SIZET; >> goto rflag; >> - case 'b': >> - { >> - const char *q; >> - int anybitset, bit; >> - >> - ulval = (u_int)GETARG(int); >> - cp = GETARG(char *); >> - >> - q = __ultoa(ulval, buf + BUF, *cp++, 0, xdigs_lower); >> - PRINT(q, buf + BUF - q); >> - >> - if (ulval == 0) >> - break; >> - >> - for (anybitset = 0; *cp;) { >> - bit = *cp++; >> - if (ulval & (1 << (bit - 1))) { >> - PRINT(anybitset ? "," : "<", 1); >> - q = cp; >> - for (; (bit = *cp) > ' '; ++cp) >> - continue; >> - PRINT(q, cp - q); >> - anybitset = 1; >> - } else >> - for (; *cp > ' '; ++cp) >> - continue; >> - } >> - if (anybitset) >> - PRINT(">", 1); >> - } >> - continue; >> case 'C': >> flags |= LONGINT; >> /*FALLTHROUGH*/ >> > _______________________________________________ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"