On Sat, Jun 10, 2017 at 06:35:13PM +0200, Jean-Pierre Chrétien wrote:

> > Jean-Pierre, do you still get these warnings on current master?
> 
> Those in insets/InsetSpecialChar.cpp disappeared, but these remain :
> 
> ../../../master/3rdparty/mythes/1.2.5/mythes.cxx: In member function ‘int
> MyThes::thInitialize(const char*, const char*)’:
> ../../../master/3rdparty/mythes/1.2.5/mythes.cxx:51:29: warning: comparison
> between signed and unsigned integer expressions [-Wsign-compare]
>      if (idxsz <= 0 || idxsz > std::numeric_limits<int>::max() /
> sizeof(sizeof(char*))) {
>                              ^
> ../../../master/3rdparty/mythes/1.2.5/mythes.cxx: In member function ‘int
> MyThes::Lookup(const char*, int, mentry**)’:
> ../../../master/3rdparty/mythes/1.2.5/mythes.cxx:179:36: warning: comparison
> between signed and unsigned integer expressions [-Wsign-compare]
>      if (nmeanings < 0 || nmeanings > std::numeric_limits<int>::max() /
> sizeof(mentry))

I'm not sure we should change this for the included 3rd party libraries
so we might just have to live with them.

>                                     ^
> ../../master/src/insets/InsetListingsParams.cpp: In constructor
> ‘lyx::{anonymous}::ParValidator::ParValidator()’:
> ../../master/src/insets/InsetListingsParams.cpp:297:1: note: variable
> tracking size limit exceeded with -fvar-tracking-assignments, retrying
> without
>  ParValidator::ParValidator()
>  ^

>From what I understand [1], this is not serious.

Thanks for checking, Jean-Pierre.

Scott

[1]
https://stackoverflow.com/questions/23499909/adjust-variable-tracking-assignment-length

Reply via email to