restore comment about boost::shared_ptr and dtor.

[lyx.git] / src / TextClass.h

diff --git a/src/TextClass.h b/src/TextClass.h
index e4067ffd20eae613f11a764d3f49cd123b39646f..51c42b06a226ef53b1ea35f292cb766bf72f31f8 100644 (file)
--- a/src/TextClass.h
+++ b/src/TextClass.h
@@ -11,16 +11,18 @@
 #define TEXTCLASS_H
 
 #include "ColorCode.h"
+#include "Counters.h"
+#include "FloatList.h"
 #include "FontInfo.h"
+#include "Layout.h"
 #include "LayoutEnums.h"
-#include "LayoutPtr.h"
 
 #include "insets/InsetLayout.h"
 
 #include "support/docstring.h"
 #include "support/types.h"
 
-#include <boost/shared_ptr.hpp>
+#include <boost/noncopyable.hpp>
 
 #include <list>
 #include <map>
@@ -37,6 +39,19 @@ class Layout;
 class LayoutFile;
 class Lexer;
 
+/// Based upon ideas in boost::noncopyable, inheriting from this
+/// class effectively makes the copy constructor protected but the
+/// assignment constructor private.
+class ProtectCopy
+{
+protected:
+       ProtectCopy() {}
+       ~ProtectCopy() {}
+       ProtectCopy(const ProtectCopy &) {};
+private:
+       const ProtectCopy & operator=(const ProtectCopy &);
+};
+
 
 /// A TextClass represents a collection of layout information: At the 
 /// moment, this includes Layout's and InsetLayout's.
@@ -46,7 +61,7 @@ class Lexer;
 /// Simple TextClass objects are not directly constructed in the main 
 /// LyX code---the constructor is protected. (That said, in tex2lyx
 /// there are what amount to simple TextClass objects.)
-class TextClass {
+class TextClass : protected ProtectCopy {
 public:
        ///
        virtual ~TextClass() {};
@@ -54,7 +69,7 @@ public:
        // typedefs
        ///////////////////////////////////////////////////////////////////
        /// The individual paragraph layouts comprising the document class
-       typedef std::vector<LayoutPtr> LayoutList;
+       typedef std::vector<Layout> LayoutList;
        /// The inset layouts available to this class
        typedef std::map<docstring, InsetLayout> InsetLayouts;
        ///
@@ -63,8 +78,7 @@ public:
        ///////////////////////////////////////////////////////////////////
        // Iterators
        ///////////////////////////////////////////////////////////////////
-       /// Note that this returns a (LayoutPtr *). We really need a custom
-       /// iterator here.
+       ///
        const_iterator begin() const { return layoutlist_.begin(); }
        ///
        const_iterator end() const { return layoutlist_.end(); }
@@ -74,12 +88,16 @@ public:
        // Layout Info
        ///////////////////////////////////////////////////////////////////
        ///
-       LayoutPtr const & defaultLayout() const;
+       Layout const & defaultLayout() const;
        ///
        docstring const & defaultLayoutName() const;
+       ///
+       bool isDefaultLayout(Layout const &) const;
+       /// 
+       bool isEmptyLayout(Layout const &) const;
        /// returns a special layout for use when we don't really want one,
        /// e.g., in table cells
-       LayoutPtr const & emptyLayout() const 
+       Layout const & emptyLayout() const 
                        { return operator[](emptylayout_); };
        /// the name of the empty layout
        docstring const & emptyLayoutName() const 
@@ -89,7 +107,7 @@ public:
        ///
        bool hasLayout(docstring const & name) const;
        ///
-       LayoutPtr const & operator[](docstring const & vname) const;
+       Layout const & operator[](docstring const & vname) const;
 
        ///////////////////////////////////////////////////////////////////
        // reading routines
@@ -125,6 +143,19 @@ public:
 protected:
        /// Protect construction
        TextClass();
+       ///
+       Layout & operator[](docstring const & vname);
+
+       ///////////////////////////////////////////////////////////////////
+       // non-const iterators
+       ///////////////////////////////////////////////////////////////////
+       ///
+       typedef LayoutList::iterator iterator;
+       ///
+       iterator begin() { return layoutlist_.begin(); }
+       ///
+       iterator end() { return layoutlist_.end(); }
+
        ///////////////////////////////////////////////////////////////////
        // members
        ///////////////////////////////////////////////////////////////////
@@ -137,9 +168,9 @@ protected:
        /// document class description
        std::string description_;
        /// available types of float, eg. figure, algorithm.
-       boost::shared_ptr<FloatList> floatlist_;
+       mutable FloatList floatlist_;
        /// Types of counters, eg. sections, eqns, figures, avail. in document class.
-       boost::shared_ptr<Counters> counters_;
+       mutable Counters counters_;
        /// Has this layout file been loaded yet?
        mutable bool loaded_;
        /// Is the TeX class available?
@@ -189,7 +220,7 @@ protected:
        /// The name of the title command
        std::string titlename_;
        /// Input layouts available to this layout
-       mutable InsetLayouts insetlayoutlist_;
+       InsetLayouts insetlayoutlist_;
        /// The minimal TocLevel of sectioning layouts
        int min_toclevel_;
        /// The maximal TocLevel of sectioning layouts
@@ -226,7 +257,7 @@ private:
 ///
 /// In the main LyX code, DocumentClass objects are created only by
 /// DocumentClassBundle, for which see below.
-class DocumentClass : public TextClass {
+class DocumentClass : public TextClass, boost::noncopyable {
 public:
        ///
        virtual ~DocumentClass() {}
@@ -239,7 +270,7 @@ public:
        /// A DocumentClass nevers count as loaded, since it is dynamic
        virtual bool loaded() { return false; }
        /// Inset layouts of this doc class
-       InsetLayouts & insetLayouts() const { return insetlayoutlist_; };
+       InsetLayouts const & insetLayouts() const { return insetlayoutlist_; };
        /// \return the layout object of an inset given by name. If the name
        /// is not found as such, the part after the ':' is stripped off, and
        /// searched again. In this way, an error fallback can be provided:
@@ -256,11 +287,10 @@ public:
        // accessors
        ///////////////////////////////////////////////////////////////////
        /// the list of floats defined in the document class
-       FloatList & floats() { return *floatlist_.get(); }
        /// the list of floats defined in the document class
-       FloatList const & floats() const { return *floatlist_.get(); }
-       /// The Counters present in this document class.
-       Counters & counters() const { return *counters_.get(); }
+       FloatList const & floats() const { return floatlist_; }
+       ///
+       Counters & counters() const { return counters_; }
        ///
        std::string const & opt_fontsize() const { return opt_fontsize_; }
        ///
@@ -312,7 +342,7 @@ protected:
        DocumentClass() {};
 private:
        /// The only class that can create a DocumentClass is
-       /// DocumentClassBundle, which calls the private constructor.
+       /// DocumentClassBundle, which calls the protected constructor.
        friend class DocumentClassBundle;
        ///
        static InsetLayout empty_insetlayout_;
@@ -329,7 +359,7 @@ private:
 ///
 /// This is a singleton class. Its sole instance is accessed via 
 /// DocumentClassBundle::get().
-class DocumentClassBundle {
+class DocumentClassBundle : boost::noncopyable {
 public:
        /// \return Pointer to a new class equal to baseClass
        DocumentClass & newClass(LayoutFile const & baseClass);
@@ -338,10 +368,8 @@ public:
 private:
        /// control instantiation
        DocumentClassBundle() {}
-       /// noncopyable
-       DocumentClassBundle(DocumentClassBundle const &);
        ///
-       std::list<DocumentClass> tc_list_;
+       std::list<DocumentClass *> tc_list_;
 };