From owner-cvs-src@FreeBSD.ORG Mon May 16 10:17:13 2005 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 5BB9716A4CE; Mon, 16 May 2005 10:17:13 +0000 (GMT) Received: from aiolos.otenet.gr (aiolos.otenet.gr [195.170.0.23]) by mx1.FreeBSD.org (Postfix) with ESMTP id 76EB343D80; Mon, 16 May 2005 10:17:12 +0000 (GMT) (envelope-from keramida@freebsd.org) Received: from orion.daedalusnetworks.priv (aris.bedc.ondsl.gr [62.103.39.226])j4GAFSjT010612; Mon, 16 May 2005 13:15:30 +0300 Received: from orion.daedalusnetworks.priv (orion [127.0.0.1]) j4GAH6eP000760; Mon, 16 May 2005 13:17:06 +0300 (EEST) (envelope-from keramida@freebsd.org) Received: (from keramida@localhost)j4GAH4X4000759; Mon, 16 May 2005 13:17:04 +0300 (EEST) (envelope-from keramida@freebsd.org) Date: Mon, 16 May 2005 13:17:04 +0300 From: Giorgos Keramidas To: Bruce Evans , "David O'Brien" Message-ID: <20050516101704.GB676@orion.daedalusnetworks.priv> References: <200504161543.j3GFhclO075103@repoman.freebsd.org> <20050516001837.GA65506@dragon.NUXI.org> <20050516004059.GA15002@gothmog.gr> <20050516032743.GA89903@dragon.NUXI.org> <20050516143822.D42751@delplex.bde.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20050516143822.D42751@delplex.bde.org> cc: cvs-src@freebsd.org cc: src-committers@freebsd.org cc: cvs-all@freebsd.org Subject: Re: cvs commit: src/usr.bin/top machine.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 16 May 2005 10:17:13 -0000 On 2005-05-16 14:57, Bruce Evans wrote: >On Sun, 15 May 2005, David O'Brien wrote: >>On Mon, May 16, 2005 at 03:40:59AM +0300, Giorgos Keramidas wrote: >>>On 2005-05-15 17:18, David O'Brien wrote: >>>> >>>>Please back machine.c back to revsion 1.69, ASAP. >>>>The current top is redicious: >>..snip.. >>>Your USERNAME column is too wide (12 columns). Do you happen to have >>>very long names? >> >> The dovecot port adds a the 'dovecot-auth' user. The top(1) code >> supports 13 character usernames on SMP systems and 15 on UP systems. >> Your rev 1.70-1.71 changes failed to take that into account. > > Another bug in top is that it expands the USERNAME column to have enough > space for the longest user name (clipped to 13 or 15 characters), even > when the user(s) with the longest name have never run any processes. > > Another bug in at last old versions of top (machine.c 1.51) is that this > support is broken if there is actually a user name of maximal length, due > to essentially the same bug that brought this thread back to life: the > header is not properly truncated; I have a local patch that already does this. Care to have a look and tell me if it's ok to commit the header timming diff? >>> If that is true, would it be ok to wait 1-2 days until I have a >>> machine.c version that prints USERNAME using only 8 columns instead >>> of using arbitrarily long columns, like now? >> >> I don't know why to wait 1-2 days. I made the change in 1 minute. >> Or is there debate if that change will be acceptable? If there isn't >> a debate about it, let me know and I'll commit this diff. > > It is easy to add more bugs in 1 minute. Exactly :-)