Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes:
| >>>>> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes:
|
| Angus> Compiling the latest CVS, I get the following warning message.
| Angus> Perhaps this is useful to someone. Angus
|
| Angus> cxx -std strict_ansi -nocleanup -ptr
| Angus> /usr/users/aleem/lyx/devel/lyx_cxx_repository -DHAVE_CONFIG_H
| Angus> -I. -I. -I../../src -I./../ -I../.. -I../..
| Angus> -I../../src/cheaders -I/usr/local/include -g -MD -c insetfoot.C
| Angus> -o insetfoot.o
|
| Angus> cxx: Warning: ../../src/insets/insettext.h, line 156: "void
| Angus> InsetText::deleteLyXText(BufferView *) const" does not match
| Angus> "UpdatableInset::deleteLyXText" -- virtual function override
| Angus> intended? void deleteLyXText(BufferView *) const; ---------^
|
| Strangely enough, I do not see this with cxx 6.2 (which I am testing
| since today). However, UpdatableInset::deleteLyXText is not 'const'.
| Should it be?
Yes I think it safely can be const.
The inset is not changed by deleting the internal LyXText.
(it only looses the screen/breaking cache)
Lgb
