This is the mail archive of the
ecos-patches@sourceware.org
mailing list for the eCos project.
Re: [PATCH] Speed-up sprintf() family of functions.
- From: Sergei Organov <osv at javad dot com>
- To: ecos-patches at ecos dot sourceware dot org
- Date: Thu, 04 Jan 2007 01:48:20 +0300
- Subject: Re: [PATCH] Speed-up sprintf() family of functions.
- References: <87fyb3x070.fsf@javad.com> <459529B0.8080407@eCosCentric.com> <87wt4allsq.fsf@javad.com> <4596F7EA.4090605@eCosCentric.com> <877iw5kw4k.fsf@javad.com> <459BCA9A.1030007@eCosCentric.com>
Jonathan Larmour <jifl@eCosCentric.com> writes:
> Sergei Organov wrote:
[...]
> The only potential problem could be extra footprint caused by pulling
> in the various versions of the write() and get_error() methods, along
> with the desstructor. But in this *particular* case, this isn't so bad
> as those functions do not themselves impose extra requirements of any
> note, and people are always going to want to write() a stream. So it's
> not a free-for-all for virtual functions by any means - the same logic
> would not apply to all methods, but this particular case is ok.
Yes, I agree that an application that wants to read() but not to write()
ought to be almost impossible to meet in practice.
>
> If you could repost the patch with the changes we did agree on, that
> would be great, thanks. Although I won't be able to commit it before
> your copyright assignment arrives.
That's fine with me. I then defer the effort of changing the patch till
then.
-- Sergei.