]> git.lyx.org Git - features.git/commit
Move the call to Buffer::errors("Parse") from BufferView::insertLyXFile to GuiView...
authorVincent van Ravesteijn <vfr@lyx.org>
Fri, 29 Oct 2010 16:51:50 +0000 (16:51 +0000)
committerVincent van Ravesteijn <vfr@lyx.org>
Fri, 29 Oct 2010 16:51:50 +0000 (16:51 +0000)
commitef7ecea2b3d78a88ee160c583c2cc017d980f4bc
tree7ba47922fd11783b87aa7711da3189b7530c9f2b
parentc9a91c54745d080726e1955809b10b7f414a27e5
Move the call to Buffer::errors("Parse") from BufferView::insertLyXFile to GuiView::insertLyXFile.

Buffer::errors("Parse") is called 7 times in the whole project. 4 times from GuiView and three times from functions in other classes, but which are (almost) only called from the GuiView.

Buffer::errors is used to signal the GUI that there might be an error occuring, but what sense does it make if it is only called from the Gui ?

Isn't it better to let the function return wether it succeeded or not and let the GuiView take action in doing something with the possible errors.

git-svn-id: svn://svn.lyx.org/lyx/lyx-devel/trunk@35911 a592a061-630c-0410-9148-cb99ea01b6c8
src/BufferView.cpp
src/frontends/qt4/GuiView.cpp