View Issue Details

IDProjectCategoryView StatusLast Update
0000443Far Manager_Commonpublic2008-11-25 09:42
ReporterSten Assigned Toalexy  
PrioritynormalSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Fixed in Version1.80 alpha 1 
Summary0000443: LocalStricmp() is too slow
DescriptionВсе то же, что и в Mantis#442, но теперь речь идет о LocalStricmp() - ее реализация жутко неэффективна, а вызывается она очень часто. Предлагаю такой варинат:

int __cdecl LocalStricmp(const char *s1,const char *s2)
{
    return CompareStringA(
        LOCALE_SYSTEM_DEFAULT,
        NORM_IGNORECASE,
        s1,
        -1,
        s2,
        -1
        )-2;
}

Так же как и в Mantis#442 не очень представляю, зачем там был нужен OEM_CP, так что не гарантирую, что эта реализация 100% эквивалентна тому, что было.
TagsNo tags attached.
Build592

Activities

alexy

2007-12-25 05:00

administrator   bugnote:0001073

Ничего лучше тут не сделаешь, это нужно для Враппера старых плагинов, а там всё в OEM.

alexy

2007-12-25 05:01

administrator   bugnote:0001074

Можно конечно сделать как было в 1.70, но как то не хочется.

alexy

2008-11-20 14:00

administrator   bugnote:0001998

fixed. 592.

Issue History

Date Modified Username Field Change
2007-12-15 09:09 Sten New Issue
2007-12-25 05:00 alexy Note Added: 0001073
2007-12-25 05:01 alexy Note Added: 0001074
2008-06-04 12:16 alexy Status new => assigned
2008-06-04 12:16 alexy Assigned To => alexy
2008-11-20 14:00 alexy Note Added: 0001998
2008-11-20 14:00 alexy Status assigned => feedback
2008-11-25 09:42 alexy Build => 592
2008-11-25 09:42 alexy Status feedback => closed
2008-11-25 09:42 alexy Resolution open => fixed
2008-11-25 09:42 alexy Fixed in Version => 1.80 alpha 1