Mailing-List:	contact cygwin-apps-help AT cygwin DOT com; run by ezmlm
Sender:	cygwin-apps-owner AT cygwin DOT com
List-Subscribe:	<mailto:cygwin-apps-subscribe AT cygwin DOT com>
List-Archive:	<http://sources.redhat.com/ml/cygwin-apps/>
List-Post:	<mailto:cygwin-apps AT cygwin DOT com>
List-Help:	<mailto:cygwin-apps-help AT cygwin DOT com>, <http://sources.redhat.com/lists.html#faqs>
Delivered-To:	mailing list cygwin-apps AT cygwin DOT com
From:	"Gary R. Van Sickle" <g DOT r DOT vansickle AT worldnet DOT att DOT net>
To:	<cygwin-apps AT sources DOT redhat DOT com>
Subject:	RE: setup w/char* eliminated is big
Date:	Sat, 16 Feb 2002 01:20:47 -0600
Message-ID:	<NCBBIHCHBLCMLBLOBONKGEKFCKAA.g.r.vansickle@worldnet.att.net>
MIME-Version:	1.0
X-Priority:	3 (Normal)
X-MSMail-Priority:	Normal
X-Mailer:	Microsoft Outlook IMO, Build 9.0.2416 (9.0.2911.0)
In-reply-to:	<3C6D31C4.2010800@syntrex.com>
X-MimeOLE:	Produced By Microsoft MimeOLE V6.00.2600.0000
Importance:	Normal

> -----Original Message-----
> From: cygwin-apps-owner AT cygwin DOT com
> [mailto:cygwin-apps-owner AT cygwin DOT com]On Behalf Of Pavel Tsekov
> Sent: Friday, February 15, 2002 10:05 AM
>
> Robert Collins wrote:
>
> > Ok,
> >  finally got some breathing time.
> >
> > Setup with char * eliminated is ~350K. Ouch.
> >
> > This is why I've not committed my patch yet (I've been trying to see
> > *where* the extra 100K appeared from).
>
>
> You have four 'inline' - I know they're small in size, but three of them
> are the most commonly used methods (the default and the copy constructor
> and also the 'operator ='). Remove the 'inline' modifier and see if the
> executable gets smaller.
>

Done and done:

CVS + "For the curious" patch + Two subsequent patches from Michael Chase ==
355840 bytes.
Above with all inlines "un-inlined" == 344576 bytes.

So a bit over 11KB saved.  In my judgement that's enough to warrant removing the
inlines; if string-handling speed is a significant factor for setup.exe I'd say
there's something wrong somewhere that no amount of inlines could remedy.

--
Gary R. Van Sickle
Brewer.  Patriot.

- Raw text -