svn commit: r268565 - head/share/mk

2014-07-12 Thread Rui Paulo
Author: rpaulo Date: Sat Jul 12 07:46:18 2014 New Revision: 268565 URL: http://svnweb.freebsd.org/changeset/base/268565 Log: Remove bsd.dtrace.mk. MFC after:3 weeks Modified: head/share/mk/Makefile Modified: head/share/mk/Makefile

svn commit: r268566 - head/usr.bin/users

2014-07-12 Thread David Chisnall
Author: theraven Date: Sat Jul 12 07:47:50 2014 New Revision: 268566 URL: http://svnweb.freebsd.org/changeset/base/268566 Log: Turn off exceptions and rtti when building the c++ version of users. Neither is used in the program and this saves us 10KB (around 40%) in binary size. Modified:

Re: svn commit: r268491 - head/usr.bin/users

2014-07-12 Thread David Chisnall
On 10 Jul 2014, at 16:29, Ed Schouten e...@80386.nl wrote: With the patch above, the binary shrinks to 15640 bytes, so my concerns are somewhat addressed. :-) I wasn't seeing that saving, but I've now committed a tweak to the Makefile that turns off exceptions and RTTI. This shrinks the

svn commit: r268568 - in head/bin/sh: . tests/parameters

2014-07-12 Thread Jilles Tjoelker
Author: jilles Date: Sat Jul 12 10:27:30 2014 New Revision: 268568 URL: http://svnweb.freebsd.org/changeset/base/268568 Log: sh: Consistently treat ${01} like $1. Leading zeroes were ignored when checking whether a positional parameter is set, but not when expanding its value. Ignore

Re: svn commit: r268566 - head/usr.bin/users

2014-07-12 Thread Bruce Evans
On Sat, 12 Jul 2014, David Chisnall wrote: Log: Turn off exceptions and rtti when building the c++ version of users. Neither is used in the program and this saves us 10KB (around 40%) in binary size. This joke is bad. The C++ version is worse in every way. Source code: It becomes smaller

Re: svn commit: r268566 - head/usr.bin/users

2014-07-12 Thread Bruce Simpson
On 12/07/2014 12:34, Bruce Evans wrote: This joke is bad. The C++ version is worse in every way. I concur with everything you have said in your message. Moral of story: it is very, very easy to introduce bloat using C++, and using a higher level language does not automatically lead to

Re: svn commit: r268543 - in head/sys: dev/drm2 dev/drm2/i915 dev/drm2/radeon dev/etherswitch/arswitch dev/etherswitch/rtl8366 dev/iicbus sys

2014-07-12 Thread Ian Lepore
On Fri, 2014-07-11 at 18:48 -0700, Rui Paulo wrote: On Jul 11, 2014, at 18:42, Ian Lepore i...@freebsd.org wrote: On Sat, 2014-07-12 at 01:04 +, Rui Paulo wrote: Author: rpaulo Date: Sat Jul 12 01:04:10 2014 New Revision: 268543 URL: http://svnweb.freebsd.org/changeset/base/268543

Re: svn commit: r268566 - head/usr.bin/users

2014-07-12 Thread Alexey Dokuchaev
On Sat, Jul 12, 2014 at 01:37:49PM +0100, Bruce Simpson wrote: On 12/07/2014 12:34, Bruce Evans wrote: This joke is bad. The C++ version is worse in every way. I concur with everything you have said in your message. Moral of story: it is very, very easy to introduce bloat using C++, and

svn commit: r268569 - head/lib/libc/stdlib

2014-07-12 Thread Pedro F. Giffuni
Author: pfg Date: Sat Jul 12 15:19:30 2014 New Revision: 268569 URL: http://svnweb.freebsd.org/changeset/base/268569 Log: strfmon: reduce unnecessary snprintf. No need for the snprintf/asprintf dance; use fixed width formats. Obtained from:NetBSD (CVS rev. 1.8) MFC after:

Re: svn commit: r268566 - head/usr.bin/users

2014-07-12 Thread Ian Lepore
On Sat, 2014-07-12 at 13:27 +, Alexey Dokuchaev wrote: On Sat, Jul 12, 2014 at 01:37:49PM +0100, Bruce Simpson wrote: On 12/07/2014 12:34, Bruce Evans wrote: This joke is bad. The C++ version is worse in every way. I concur with everything you have said in your message. Moral of

Re: svn commit: r268566 - head/usr.bin/users

2014-07-12 Thread Steve Kargl
On Sat, Jul 12, 2014 at 09:31:23AM -0600, Ian Lepore wrote: On Sat, 2014-07-12 at 13:27 +, Alexey Dokuchaev wrote: On Sat, Jul 12, 2014 at 01:37:49PM +0100, Bruce Simpson wrote: On 12/07/2014 12:34, Bruce Evans wrote: This joke is bad. The C++ version is worse in every way. I

Re: svn commit: r268566 - head/usr.bin/users

2014-07-12 Thread Bruce Simpson
On 12/07/2014 16:31, Ian Lepore wrote: It is in no way established that any kind of mess exists. On the contrary, the metrics and analysis which bde@ has presented seem pretty clear to me. All we have is some people who appear to hate C++ expressing their opinions about how they hate C++.

Re: svn commit: r268570 - head/sys/kern

2014-07-12 Thread Konstantin Belousov
On Sat, Jul 12, 2014 at 03:35:04PM +, Mateusz Guzik wrote: Author: mjg Date: Sat Jul 12 15:35:04 2014 New Revision: 268570 URL: http://svnweb.freebsd.org/changeset/base/268570 Log: Clear nonblock and async on devctl close instaed of open. This is a purely cosmetic change.

Re: svn commit: r268570 - head/sys/kern

2014-07-12 Thread Mateusz Guzik
On Sat, Jul 12, 2014 at 07:18:01PM +0300, Konstantin Belousov wrote: On Sat, Jul 12, 2014 at 03:35:04PM +, Mateusz Guzik wrote: Author: mjg Date: Sat Jul 12 15:35:04 2014 New Revision: 268570 URL: http://svnweb.freebsd.org/changeset/base/268570 Log: Clear nonblock and async

Re: svn commit: r268570 - head/sys/kern

2014-07-12 Thread Konstantin Belousov
On Sat, Jul 12, 2014 at 06:53:47PM +0200, Mateusz Guzik wrote: On Sat, Jul 12, 2014 at 07:18:01PM +0300, Konstantin Belousov wrote: On Sat, Jul 12, 2014 at 03:35:04PM +, Mateusz Guzik wrote: Author: mjg Date: Sat Jul 12 15:35:04 2014 New Revision: 268570 URL:

Re: svn commit: r268566 - head/usr.bin/users

2014-07-12 Thread Tim Kientzle
On Jul 12, 2014, at 8:31 AM, Ian Lepore i...@freebsd.org wrote: It is in no way established that any kind of mess exists. All we have is some people who appear to hate C++ expressing their opinions about how they hate C++. All the whining about how many more bytes the compiled code is or

svn commit: r268575 - head

2014-07-12 Thread Rui Paulo
== --- head/ObsoleteFiles.inc Sat Jul 12 18:44:47 2014(r268574) +++ head/ObsoleteFiles.inc Sat Jul 12 21:24:28 2014(r268575) @@ -38,6 +38,8 @@ # xargs -n1 | sort | uniq -d; # done +# 20140712: Removal of bsd.dtrace.mk

Re: svn commit: r268491 - head/usr.bin/users

2014-07-12 Thread Jilles Tjoelker
On Sat, Jul 12, 2014 at 11:06:33PM +0200, Dimitry Andric wrote: On 10 Jul 2014, at 17:29, Ed Schouten e...@80386.nl wrote: On 10 July 2014 17:23, Pietro Cerutti g...@freebsd.org wrote: It's because of the standard library. Examples: /usr/include/c++/v1/memory:1454:47: error: unused

svn commit: r268576 - in head/bin/sh: . tests/parameters

2014-07-12 Thread Jilles Tjoelker
Author: jilles Date: Sat Jul 12 21:54:11 2014 New Revision: 268576 URL: http://svnweb.freebsd.org/changeset/base/268576 Log: sh: Correctly handle positional parameters beyond INT_MAX on 64-bit systems. Currently, there can be no more than INT_MAX positional parameters. Make sure to treat