Some comments and cosmetics.

git-svn-id: svn://svn.lyx.org/lyx/lyx-devel/trunk@21151 a592a061-630c-0410-9148-cb99ea01b6c8

diff --git a/src/frontends/qt4/GuiInclude.h b/src/frontends/qt4/GuiInclude.h
index 8230860dd12719120737de673a257ec1b92b091b..3f46796e266862ed9c8096be5845008b1dba9e83 100644 (file)
--- a/src/frontends/qt4/GuiInclude.h
+++ b/src/frontends/qt4/GuiInclude.h
@@ -25,6 +25,8 @@
 namespace lyx {
 namespace frontend {
 
+//FIXME This could, and therefore probably should, inherit from
+//GuiCommand.
 class GuiInclude : public GuiDialog, public Ui::IncludeUi
 {
        Q_OBJECT

diff --git a/src/frontends/qt4/GuiRef.cpp b/src/frontends/qt4/GuiRef.cpp
index 1bb852685abba7466beab0c086f59cf9ffa1df46..6de11331efe3006af2facbcc7a26818497d2faf1 100644 (file)
--- a/src/frontends/qt4/GuiRef.cpp
+++ b/src/frontends/qt4/GuiRef.cpp
@@ -42,9 +42,6 @@ namespace frontend {
 using support::makeAbsPath;
 using support::makeDisplayPath;
 
-//FIXME It should be possible to eliminate lfun_name_
-//now and recover that information from params().insetType().
-//But let's not do that quite yet.
 /// Flags what action is taken by Kernel::dispatch()
 static std::string const lfun_name_ = "ref";
 

diff --git a/src/frontends/qt4/GuiRef.h b/src/frontends/qt4/GuiRef.h
index c7945f3af438cb9a4633b791d7130120b0c443ca..96285e3640ff218bce8442253e1ebe89d7b54309 100644 (file)
--- a/src/frontends/qt4/GuiRef.h
+++ b/src/frontends/qt4/GuiRef.h
@@ -24,6 +24,9 @@ class QListWidgetItem;
 namespace lyx {
 namespace frontend {
 
+// FIXME This could, and therefore, should inherit from
+// GuiCommand. Note, e.g., that the definitions of the first
+// three private functions all just replicate what's there.
 class GuiRef : public GuiDialog, public Ui::RefUi
 {
        Q_OBJECT