[Libreoffice] [REVIEW] request for review for patch for fdo#32742# regression
eagles051387 at gmail.com
Wed Jan 12 23:50:40 PST 2011
Not trying to hijack, but i'm still a novice developer still learning
the ways of development, but is there such a thing as over optimization?
On 1/12/11 7:36 PM, Noel Power wrote:
> On 12/01/11 18:03, Norbert Thiebaud wrote:
>> On Wed, Jan 12, 2011 at 10:35 AM, Octavio Alvarez
>> <alvarezp at alvarezp.ods.org> wrote:
>>> On Wed, 12 Jan 2011 03:18:01 -0800, Noel Power<nopower at novell.com>
>>>> Hi there
>>>> fix for fdo#32742#, its a simple fix ( but was a bugger to find )
>>>> imo is
>>>> riskless etc.
>>>> is ripe for signoff and cherrypicking, be grateful if someone could
>>>> with that
>>> Is suggest to make ::std::sort on the next line use s_nCount also and
>>> prevent a useless division operation.
>> Actually the 'useless' division' is resolved at compile time. so it
>> is free.
>> on the other hand passing s_nCount instead of a constant _could_
>> deprive the template code of some possible optimizations.
>> (I don't know if std::sort is smart enough to take avantage of it..
>> but it could, and eventually it will)
> the other thing to remember is that this is some static
> initialisation, it is done just once, so even thinking about
> optimising that would seem to be overkill :-)
> LibreOffice mailing list
> LibreOffice at lists.freedesktop.org
More information about the LibreOffice