Coverity musings
Jean-Marc Lasgouttes
lasgouttes at lyx.org
Thu Sep 26 13:48:09 UTC 2024
Le 19/09/2024 à 16:27, Jürgen Spitzmüller a écrit :
> Am Donnerstag, dem 19.09.2024 um 16:16 +0200 schrieb Jean-Marc
> Lasgouttes:
>> What I have in mind as a simpler alternative is this (like the last
>> patch, with vector instead of map).
>
> Seems to make sense. I have no time to look closer now, unfortunately.
It turns out that I forgot to change the variable type in TeXBottomHLine
and that a vector<bool> is not convenient for our use. Indeed, it is
specialized in the C++ library to use a birfield so that one cannot do
things like:
vec1[c] |= vec2[c];
The possible solutions make the code less readable. For now I just stuck
to the existing data structure (a map).
Let's see whether Coverity is happy with that.
JMarc
More information about the lyx-devel
mailing list