[ wxwindows-Patches-1861348 ] wxStreamBuffer::m_buffer_size FIXME

[SourceForge.net]

Patches item #1861348, was opened at 2007-12-31 13:33
Message generated for change (Settings changed) made by vadz
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=309863&aid=1861348&group_id=9863

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Common
Group: optimization
>Status: Closed
>Resolution: Accepted
Priority: 5
Private: No
Submitted By: Steven Van Ingelgem (g00fy)
Assigned to: Nobody/Anonymous (nobody)
Summary: wxStreamBuffer::m_buffer_size FIXME

Initial Comment:
Checking wxStreamBuffer I saw that there was a fixme for the size.

In the attached patch I removed this variable and made all the functions which made use of it compile again. No function signature was changed and a small optimalisation was added in the case that there was already more than enough space available in the buffer (PutToBuffer function).


Added into the patch is an optimization (in my opinion) for the wxStreamBuffer latex file as right now it seems to be just put together as it goes. I made it in alphabetic order.

BTW, "licence" seems to be misspelled everywhere? Or is it a difference between en_GB and en_US?

----------------------------------------------------------------------

>Comment By: Vadim Zeitlin (vadz)
Date: 2008-01-02 17:53

Message:
Logged In: YES 
user_id=71618
Originator: NO

Thanks for cleaning this up!

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=309863&aid=1861348&group_id=9863