]> git.lyx.org Git - features.git/commitdiff
Clarify that the Dialog::lyxview_ pointer is in fact a reference
authorGuillaume Munch <gm@lyx.org>
Wed, 28 Sep 2016 21:04:10 +0000 (23:04 +0200)
committerGuillaume Munch <gm@lyx.org>
Thu, 29 Sep 2016 10:37:19 +0000 (12:37 +0200)
src/frontends/qt4/Dialog.cpp
src/frontends/qt4/Dialog.h

index a9c1250b6eb5276660816435c370d06f2b8eae11..e1fddaaa7e0f2660b4e3380d61c3e73fb3205e03 100644 (file)
@@ -45,7 +45,7 @@ namespace lyx {
 namespace frontend {
 
 Dialog::Dialog(GuiView & lv, QString const & name, QString const & title)
-       : name_(name), title_(title), lyxview_(&lv)
+       : name_(name), title_(title), lyxview_(lv)
 {}
 
 
@@ -75,21 +75,21 @@ void Dialog::updateDialog() const
 
 void Dialog::disconnect() const
 {
-       lyxview_->disconnectDialog(fromqstr(name_));
+       lyxview_.disconnectDialog(fromqstr(name_));
 }
 
 
 bool Dialog::isBufferAvailable() const
 {
-       return lyxview_->currentBufferView() != 0;
+       return lyxview_.currentBufferView() != 0;
 }
 
 
 bool Dialog::isBufferReadonly() const
 {
-       if (!lyxview_->documentBufferView())
+       if (!lyxview_.documentBufferView())
                return true;
-       return lyxview_->documentBufferView()->buffer().isReadonly();
+       return lyxview_.documentBufferView()->buffer().isReadonly();
 }
 
 
@@ -112,21 +112,21 @@ KernelDocType Dialog::docType() const
 
 BufferView const * Dialog::bufferview() const
 {
-       return lyxview_->currentBufferView();
+       return lyxview_.currentBufferView();
 }
 
 
 Buffer const & Dialog::buffer() const
 {
-       LAPPERR(lyxview_->currentBufferView());
-       return lyxview_->currentBufferView()->buffer();
+       LAPPERR(lyxview_.currentBufferView());
+       return lyxview_.currentBufferView()->buffer();
 }
 
 
 Buffer const & Dialog::documentBuffer() const
 {
-       LAPPERR(lyxview_->documentBufferView());
-       return lyxview_->documentBufferView()->buffer();
+       LAPPERR(lyxview_.documentBufferView());
+       return lyxview_.documentBufferView()->buffer();
 }
 
 
@@ -193,9 +193,9 @@ void Dialog::showView()
        if (wantInitialFocus())
                w->setFocus();
        else {
-               lyxview_->raise();
-               lyxview_->activateWindow();
-               lyxview_->setFocus();
+               lyxview_.raise();
+               lyxview_.activateWindow();
+               lyxview_.setFocus();
        }
 }
 
@@ -270,7 +270,7 @@ void Dialog::checkStatus()
 
 QString Dialog::sessionKey() const
 {
-       return "views/" + QString::number(lyxview_->id())
+       return "views/" + QString::number(lyxview_.id())
                + "/" + name();
 }
 
index a9769ff4abe30cb686a7d88e102575492368791c..f3f9c1d47422606cee77943c6e66aedf451a03d2 100644 (file)
@@ -250,7 +250,7 @@ public:
         *  We should aim to reduce/remove these from the interface.
         */
        //@{
-       GuiView const & lyxview() const { return *lyxview_; }
+       GuiView const & lyxview() const { return lyxview_; }
        /// Current buffer
        Buffer const & buffer() const;
        /// Main document buffer
@@ -273,7 +273,7 @@ private:
        ///
        QString title_;
        ///
-       GuiView * lyxview_;
+       GuiView & lyxview_;
 
        /// intentionally unimplemented, therefore uncopiable
        Dialog(Dialog const &);