Hi all, I removed some uses of SvStringsSortDtor, which is listed in http://wiki.documentfoundation.org/Easy_Hacks#Easy_Hacks
There's uses of SvStringsSortDtor which I couldn't understand, so I left
them untouched.
They are sc/source/filter/html/htmlexp.cxx and
sw/source/filter/writer/writer.cxx,
which are very similar; two SvStringsSortDtors (pSrcArr and pDstArr) are
used in pair.
Looking at htmlexp.cxx:1320, pSrcArr is used to lookup for an index of a
string, and then the index is applied to pDstArr.
In my understanding indices of two arrays are unrelated, because arrays are
sorted by their respective contents.
if( pSrcArr->Seek_Entry( &rFileNm, &nPos ))
> {
> rFileNm = *(*pDestArr)[ nPos ];
> return sal_True;
> }
>
where rFileNm is of type String&.
I guess what is intended here is mapping of strings, from pSrcArr to
pDstArr (with incorrect implementation).
If so, I'll replace them with std::map<String, String>.
How do you think?
Cheers,
--
Daisuke Nishino
0001-Removed-some-SvStringsSortDtor.core.patch
Description: Binary data
_______________________________________________ LibreOffice mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/libreoffice
