From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: [RFC][PATCH 1/2] PM: Reduce code duplication in the core suspend code Date: Sun, 10 Jun 2007 18:42:54 +0200 Message-ID: <200706101842.54871.rjw@sisk.pl> References: Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Content-Disposition: inline List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-pm-bounces@lists.linux-foundation.org Errors-To: linux-pm-bounces@lists.linux-foundation.org To: Alan Stern Cc: linux-pm , Pavel Machek List-Id: linux-pm@vger.kernel.org On Sunday, 10 June 2007 16:55, Alan Stern wrote: > On Sun, 10 Jun 2007, Rafael J. Wysocki wrote: > > > From: Rafael J. Wysocki > > > > Reduce code duplication in drivers/base/suspend.c by introducing an inline > > function for printing diagnostic messages. > > Could you save object-code space by making the function non-inline? > It looks complicated enough. I thought I would do that, but then I thought I'd want it to be compiled out if dev_dbg() were a noop (like when the debugging is off). > Or is the compiler supposed to figure such things out for itself? I think it is. I'll remove this 'inline'. Greetings, Rafael -- "Premature optimization is the root of all evil." - Donald Knuth