more Lexer;

also move tex2lyx specific code to tex2lyx


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

src/Layout.cpp

@@ -131,7 +131,7 @@ Layout::Layout()
 }
 
 
-bool Layout::read(Lexer & lexrc, TextClass const & tclass)
+bool Layout::read(Lexer & lex, TextClass const & tclass)
 {
 	// This table is sorted alphabetically [asierra 30March96]
 	LexerKeyword layoutTags[] = {
@@ -191,17 +191,17 @@ bool Layout::read(Lexer & lexrc, TextClass const & tclass)
 
 	bool error = false;
 	bool finished = false;
-	lexrc.pushTable(layoutTags);
+	lex.pushTable(layoutTags);
 	// parse style section
-	while (!finished && lexrc.isOK() && !error) {
+	while (!finished && lex.isOK() && !error) {
-		int le = lexrc.lex();
+		int le = lex.lex();
 		// See comment in LyXRC.cpp.
 		switch (le) {
 		case Lexer::LEX_FEOF:
 			continue;
 
 		case Lexer::LEX_UNDEF:		// parse error
-			lexrc.printError("Unknown layout tag `$$Token'");
+			lex.printError("Unknown layout tag `$$Token'");
 			error = true;
 			continue;
 		default: break;
@@ -212,293 +212,264 @@ bool Layout::read(Lexer & lexrc, TextClass const & tclass)
 			break;
 
 		case LT_CATEGORY:
-			if (lexrc.next())
+			lex >> category_;
-				category_ = lexrc.getDocString();
 			break;
 
-		case LT_COPYSTYLE:     // initialize with a known style
+		case LT_COPYSTYLE: {     // initialize with a known style
-			if (lexrc.next()) {
+			docstring style;
-				docstring const style = subst(lexrc.getDocString(),
+			lex >> style;
-								'_', ' ');
+			style = subst(style, '_', ' ');
 
-				if (tclass.hasLayout(style)) {
+			if (tclass.hasLayout(style)) {
-					docstring const tmpname = name_;
+				docstring const tmpname = name_;
-					this->operator=(tclass[style]);
+				this->operator=(tclass[style]);
-					name_ = tmpname;
+				name_ = tmpname;
-				} else {
+			} else {
-					lyxerr << "Cannot copy unknown style `"
+				lyxerr << "Cannot copy unknown style `"
-					       << to_utf8(style) << "'\n"
+							 << to_utf8(style) << "'\n"
-					       << "All layouts so far:"
+							 << "All layouts so far:"
-					       << endl;
+							 << endl;
-					DocumentClass::const_iterator lit = tclass.begin();
+				DocumentClass::const_iterator lit = tclass.begin();
-					DocumentClass::const_iterator len = tclass.end();
+				DocumentClass::const_iterator len = tclass.end();
-					for (; lit != len; ++lit)
+				for (; lit != len; ++lit)
-						lyxerr << to_utf8(lit->name()) << endl;
+					lyxerr << to_utf8(lit->name()) << endl;
 
-					//lexrc.printError("Cannot copy known "
+				//lex.printError("Cannot copy known "
-					//		 "style `$$Token'");
+				//		 "style `$$Token'");
-				}
 			}
 			break;
+			}
 
-		case LT_OBSOLETEDBY:     // replace with a known style
+		case LT_OBSOLETEDBY: {   // replace with a known style
-			if (lexrc.next()) {
+			docstring style;
-				docstring const style = 
+			lex >> style;
-					subst(lexrc.getDocString(), '_', ' ');
+			style = subst(style, '_', ' ');
 
-				if (tclass.hasLayout(style)) {
+			if (tclass.hasLayout(style)) {
-					docstring const tmpname = name_;
+				docstring const tmpname = name_;
-					this->operator=(tclass[style]);
+				this->operator=(tclass[style]);
-					name_ = tmpname;
+				name_ = tmpname;
-					if (obsoleted_by().empty())
+				if (obsoleted_by().empty())
-						obsoleted_by_ = style;
+					obsoleted_by_ = style;
-				} else {
+			} else {
-					lyxerr << "Cannot replace with unknown style `" 
+				lyxerr << "Cannot replace with unknown style `" 
-						<< to_utf8(style) << '\'' << endl;
+					<< to_utf8(style) << '\'' << endl;
 
-					//lexrc.printError("Cannot replace with"
+				//lex.printError("Cannot replace with"
-					//		 " unknown style "
+				//		 " unknown style "
-					//		 "`$$Token'");
+				//		 "`$$Token'");
-				}
 			}
 			break;
+		}
 
 		case LT_DEPENDSON:
-			if (lexrc.next())
+			lex >> depends_on_;
-			depends_on_ = subst(lexrc.getDocString(), '_', ' ');
+			depends_on_ = subst(depends_on_, '_', ' ');
 			break;
 
 		case LT_MARGIN:		// margin style definition.
-			readMargin(lexrc);
+			readMargin(lex);
 			break;
 
 		case LT_LATEXTYPE:	// LaTeX style definition.
-			readLatexType(lexrc);
+			readLatexType(lex);
 			break;
 
 		case LT_LATEXHEADER:	// header for environments
-			lexrc.next();
+			lex >> latexheader;
-			latexheader = lexrc.getString();
 			break;
 
 		case LT_LATEXFOOTER:	// footer for environments
-			lexrc.next();
+			lex >> latexfooter;
-			latexfooter = lexrc.getString();
 			break;
 
 		case LT_LATEXPARAGRAPH:
-			lexrc.next();
+			lex >> latexparagraph;
-			latexparagraph = lexrc.getString();
 			break;
 
 		case LT_INTITLE:
-			intitle = lexrc.next() && lexrc.getInteger();
+			lex >> intitle;
 			break;
 
 		case LT_TOCLEVEL:
-			lexrc.next();
+			lex >> toclevel;
-			toclevel = lexrc.getInteger();
 			break;
 
 		case LT_OPTARGS:
-			if (lexrc.next())
+			lex >> optionalargs ;
-				optionalargs = lexrc.getInteger();
 			break;
 
 		case LT_NEED_PROTECT:
-			needprotect = lexrc.next() && lexrc.getInteger();
+			lex >> needprotect;
 			break;
 
 		case LT_KEEPEMPTY:
-			keepempty = lexrc.next() && lexrc.getInteger();
+			lex >> keepempty;
 			break;
 
 		case LT_FONT:
-			font = lyxRead(lexrc, font);
+			font = lyxRead(lex, font);
 			labelfont = font;
 			break;
 
 		case LT_TEXTFONT:
-			font = lyxRead(lexrc, font);
+			font = lyxRead(lex, font);
 			break;
 
 		case LT_LABELFONT:
-			labelfont = lyxRead(lexrc, labelfont);
+			labelfont = lyxRead(lex, labelfont);
 			break;
 
 		case LT_NEXTNOINDENT:	// Indent next paragraph?
-			if (lexrc.next() && lexrc.getInteger())
+			lex >> nextnoindent;
-				nextnoindent = true;
-			else
-				nextnoindent = false;
 			break;
 
 		case LT_COMMANDDEPTH:
-			lexrc.next();
+			lex >> commanddepth;
-			commanddepth = lexrc.getInteger();
 			break;
 
 		case LT_LATEXNAME:
-			if (lexrc.next())
+			lex >> latexname_;
-				latexname_ = lexrc.getString();
 			break;
 
 		case LT_LATEXPARAM:
-			if (lexrc.next())
+			lex >> latexparam_;
-				latexparam_ = subst(lexrc.getString(), "&quot;", "\"");
+			latexparam_ = subst(latexparam_, "&quot;", "\"");
 			break;
 
 		case LT_INNERTAG:
-			if (lexrc.next())
+			lex >> innertag_;
-				innertag_ = lexrc.getString();
 			break;
 
 		case LT_LABELTAG:
-			if (lexrc.next())
+			lex >> labeltag_;
-				labeltag_ = lexrc.getString();
 			break;
 
 		case LT_ITEMTAG:
-			if (lexrc.next())
+			lex >> itemtag_;
-				itemtag_ = lexrc.getString();
 			break;
 
 		case LT_PREAMBLE:
-			preamble_ = from_utf8(lexrc.getLongString("EndPreamble"));
+			preamble_ = from_utf8(lex.getLongString("EndPreamble"));
 			break;
 
 		case LT_LABELTYPE:
-			readLabelType(lexrc);
+			readLabelType(lex);
 			break;
 
 		case LT_ENDLABELTYPE:
-			readEndLabelType(lexrc);
+			readEndLabelType(lex);
 			break;
 
 		case LT_LEFTMARGIN:	// left margin type
-			if (lexrc.next())
+			lex >> leftmargin;
-				leftmargin = lexrc.getDocString();
 			break;
 
 		case LT_RIGHTMARGIN:	// right margin type
-			if (lexrc.next())
+			lex >> rightmargin;
-				rightmargin = lexrc.getDocString();
 			break;
 
 		case LT_LABELINDENT:	// label indenting flag
-			if (lexrc.next())
+			lex >> labelindent;
-				labelindent = lexrc.getDocString();
 			break;
 
 		case LT_PARINDENT:	// paragraph indent. flag
-			if (lexrc.next())
+			lex >> parindent;
-				parindent = lexrc.getDocString();
 			break;
 
 		case LT_PARSKIP:	// paragraph skip size
-			if (lexrc.next())
+			lex >> parskip;
-				parskip = lexrc.getFloat();
 			break;
 
 		case LT_ITEMSEP:	// item separation size
-			if (lexrc.next())
+			lex >> itemsep;
-				itemsep = lexrc.getFloat();
 			break;
 
 		case LT_TOPSEP:		// top separation size
-			if (lexrc.next())
+			lex >> topsep;
-				topsep = lexrc.getFloat();
 			break;
 
 		case LT_BOTTOMSEP:	// bottom separation size
-			if (lexrc.next())
+			lex >> bottomsep;
-				bottomsep = lexrc.getFloat();
 			break;
 
 		case LT_LABEL_BOTTOMSEP: // label bottom separation size
-			if (lexrc.next())
+			lex >> labelbottomsep;
-				labelbottomsep = lexrc.getFloat();
 			break;
 
 		case LT_LABELSEP:	// label separator
-			if (lexrc.next()) {
+			lex >> labelsep;
-				labelsep = from_utf8(subst(lexrc.getString(), 'x', ' '));
+			labelsep = subst(labelsep, 'x', ' ');
-			}
 			break;
 
 		case LT_PARSEP:		// par. separation size
-			if (lexrc.next())
+			lex >> parsep;
-				parsep = lexrc.getFloat();
 			break;
 
 		case LT_FILL_TOP:	// fill top flag
-			if (lexrc.next())
+			lex >> fill_top;
-				fill_top = lexrc.getInteger();
 			break;
 
 		case LT_FILL_BOTTOM:	// fill bottom flag
-			if (lexrc.next())
+			lex >> fill_bottom;
-				fill_bottom = lexrc.getInteger();
 			break;
 
 		case LT_NEWLINE:	// newlines allowed?
-			if (lexrc.next())
+			lex >> newline_allowed;
-				newline_allowed = lexrc.getInteger();
 			break;
 
 		case LT_ALIGN:		// paragraph align
-			readAlign(lexrc);
+			readAlign(lex);
 			break;
 		case LT_ALIGNPOSSIBLE:	// paragraph allowed align
-			readAlignPossible(lexrc);
+			readAlignPossible(lex);
 			break;
 
 		case LT_LABELSTRING:	// label string definition
-			if (lexrc.next()) {
+			// FIXME: this means LT_ENDLABELSTRING may only
-				labelstring_ = trim(lexrc.getDocString());
+			// occur after LT_LABELSTRING
-				labelstring_appendix_ = labelstring_;
+			lex >> labelstring_;	
-			}
+			labelstring_ = trim(labelstring_);
+			labelstring_appendix_ = labelstring_;
 			break;
 
 		case LT_ENDLABELSTRING:	// endlabel string definition
-			if (lexrc.next())
+			lex >> endlabelstring_;	
-				endlabelstring_ = trim(lexrc.getDocString());
+			endlabelstring_ = trim(endlabelstring_);
 			break;
 
 		case LT_LABELSTRING_APPENDIX: // label string appendix definition
-			if (lexrc.next())
+			lex >> labelstring_appendix_;	
-				labelstring_appendix_ = trim(lexrc.getDocString());
+			labelstring_appendix_ = trim(labelstring_appendix_);
 			break;
 
 		case LT_LABELCOUNTER: // name of counter to use
-			if (lexrc.next())
+			lex >> counter;	
-				counter = lyx::from_ascii(trim(lexrc.getString()));
+			counter = trim(counter);
 			break;
 
 		case LT_FREE_SPACING:	// Allow for free spacing.
-			if (lexrc.next())
+			lex >> free_spacing;
-				free_spacing = lexrc.getInteger();
 			break;
 
 		case LT_PASS_THRU:	// Allow for pass thru.
-			if (lexrc.next())
+			lex >> pass_thru;
-				pass_thru = lexrc.getInteger();
 			break;
 
 		case LT_SPACING: // setspace.sty
-			readSpacing(lexrc);
+			readSpacing(lex);
 			break;
 
 		case LT_REQUIRES:
-			lexrc.eatLine();
+			lex.eatLine();
 			vector<string> const req = 
-				getVectorFromString(lexrc.getString());
+				getVectorFromString(lex.getString());
 			requires_.insert(req.begin(), req.end());
 			break;
 
 		}
 	}
-	lexrc.popTable();
+	lex.popTable();
 
 	return !error;
 }
@@ -512,21 +483,23 @@ enum {
 	AT_LAYOUT
 };
 
-void Layout::readAlign(Lexer & lexrc)
-{
-	LexerKeyword alignTags[] = {
-		{ "block",  AT_BLOCK },
-		{ "center", AT_CENTER },
-		{ "layout", AT_LAYOUT },
-		{ "left",   AT_LEFT },
-		{ "right",  AT_RIGHT }
-	};
 
-	PushPopHelper pph(lexrc, alignTags);
+LexerKeyword alignTags[] = {
-	int le = lexrc.lex();
+	{ "block",  AT_BLOCK },
+	{ "center", AT_CENTER },
+	{ "layout", AT_LAYOUT },
+	{ "left",   AT_LEFT },
+	{ "right",  AT_RIGHT }
+};
+
+
+void Layout::readAlign(Lexer & lex)
+{
+	PushPopHelper pph(lex, alignTags);
+	int le = lex.lex();
 	switch (le) {
 	case Lexer::LEX_UNDEF:
-		lexrc.printError("Unknown alignment `$$Token'");
+		lex.printError("Unknown alignment `$$Token'");
 		return;
 	default: break;
 	};
@@ -550,24 +523,16 @@ void Layout::readAlign(Lexer & lexrc)
 }
 
 
-void Layout::readAlignPossible(Lexer & lexrc)
+void Layout::readAlignPossible(Lexer & lex)
 {
-	LexerKeyword alignTags[] = {
+	lex.pushTable(alignTags);
-		{ "block",  AT_BLOCK },
-		{ "center", AT_CENTER },
-		{ "layout", AT_LAYOUT },
-		{ "left",   AT_LEFT },
-		{ "right",  AT_RIGHT }
-	};
-
-	lexrc.pushTable(alignTags);
 	alignpossible = LYX_ALIGN_NONE | LYX_ALIGN_LAYOUT;
-	int lineno = lexrc.lineNumber();
+	int lineno = lex.lineNumber();
 	do {
-		int le = lexrc.lex();
+		int le = lex.lex();
 		switch (le) {
 		case Lexer::LEX_UNDEF:
-			lexrc.printError("Unknown alignment `$$Token'");
+			lex.printError("Unknown alignment `$$Token'");
 			continue;
 		default: break;
 		};
@@ -588,12 +553,12 @@ void Layout::readAlignPossible(Lexer & lexrc)
 			alignpossible |= LYX_ALIGN_LAYOUT;
 			break;
 		}
-	} while (lineno == lexrc.lineNumber());
+	} while (lineno == lex.lineNumber());
-	lexrc.popTable();
+	lex.popTable();
 }
 
 
-void Layout::readLabelType(Lexer & lexrc)
+void Layout::readLabelType(Lexer & lex)
 {
 	enum {
 		LA_NO_LABEL = 1,
@@ -622,11 +587,11 @@ void Layout::readLabelType(Lexer & lexrc)
 		{ "top_environment",          LA_TOP_ENVIRONMENT }
 	};
 
-	PushPopHelper pph(lexrc, labelTypeTags);
+	PushPopHelper pph(lex, labelTypeTags);
-	int le = lexrc.lex();
+	int le = lex.lex();
 	switch (le) {
 	case Lexer::LEX_UNDEF:
-		lexrc.printError("Unknown labeltype tag `$$Token'");
+		lex.printError("Unknown labeltype tag `$$Token'");
 		return;
 	default: break;
 	}
@@ -665,7 +630,7 @@ void Layout::readLabelType(Lexer & lexrc)
 }
 
 
-void Layout::readEndLabelType(Lexer & lexrc)
+void Layout::readEndLabelType(Lexer & lex)
 {
 	static LexerKeyword endlabelTypeTags[] = {
 		{ "box",	      END_LABEL_BOX },
@@ -674,11 +639,11 @@ void Layout::readEndLabelType(Lexer & lexrc)
 		{ "static",     END_LABEL_STATIC }
 	};
 
-	PushPopHelper pph(lexrc, endlabelTypeTags);
+	PushPopHelper pph(lex, endlabelTypeTags);
-	int le = lexrc.lex();
+	int le = lex.lex();
 	switch (le) {
 	case Lexer::LEX_UNDEF:
-		lexrc.printError("Unknown labeltype tag `$$Token'");
+		lex.printError("Unknown labeltype tag `$$Token'");
 		break;
 	case END_LABEL_STATIC:
 	case END_LABEL_BOX:
@@ -687,14 +652,14 @@ void Layout::readEndLabelType(Lexer & lexrc)
 		endlabeltype = static_cast<EndLabelType>(le);
 		break;
 	default:
-		lyxerr << "Unhandled value " << le
+		LYXERR0("Unhandled value " << le
-		       << " in Layout::readEndLabelType." << endl;
+		       << " in Layout::readEndLabelType.");
 		break;
 	}
 }
 
 
-void Layout::readMargin(Lexer & lexrc)
+void Layout::readMargin(Lexer & lex)
 {
 	LexerKeyword marginTags[] = {
 		{ "dynamic",           MARGIN_DYNAMIC },
@@ -704,12 +669,12 @@ void Layout::readMargin(Lexer & lexrc)
 		{ "static",            MARGIN_STATIC }
 	};
 
-	PushPopHelper pph(lexrc, marginTags);
+	PushPopHelper pph(lex, marginTags);
 
-	int le = lexrc.lex();
+	int le = lex.lex();
 	switch (le) {
 	case Lexer::LEX_UNDEF:
-		lexrc.printError("Unknown margin type tag `$$Token'");
+		lex.printError("Unknown margin type tag `$$Token'");
 		return;
 	case MARGIN_STATIC:
 	case MARGIN_MANUAL:
@@ -726,7 +691,7 @@ void Layout::readMargin(Lexer & lexrc)
 }
 
 
-void Layout::readLatexType(Lexer & lexrc)
+void Layout::readLatexType(Lexer & lex)
 {
 	LexerKeyword latexTypeTags[] = {
 		{ "bib_environment",  LATEX_BIB_ENVIRONMENT },
@@ -737,11 +702,11 @@ void Layout::readLatexType(Lexer & lexrc)
 		{ "paragraph",        LATEX_PARAGRAPH }
 	};
 
-	PushPopHelper pph(lexrc, latexTypeTags);
+	PushPopHelper pph(lex, latexTypeTags);
-	int le = lexrc.lex();
+	int le = lex.lex();
 	switch (le) {
 	case Lexer::LEX_UNDEF:
-		lexrc.printError("Unknown latextype tag `$$Token'");
+		lex.printError("Unknown latextype tag `$$Token'");
 		return;
 	case LATEX_PARAGRAPH:
 	case LATEX_COMMAND:
@@ -759,7 +724,7 @@ void Layout::readLatexType(Lexer & lexrc)
 }
 
 
-void Layout::readSpacing(Lexer & lexrc)
+void Layout::readSpacing(Lexer & lex)
 {
 	enum {
 		ST_SPACING_SINGLE = 1,
@@ -775,11 +740,11 @@ void Layout::readSpacing(Lexer & lexrc)
 		{"single",  ST_SPACING_SINGLE }
 	};
 
-	PushPopHelper pph(lexrc, spacingTags);
+	PushPopHelper pph(lex, spacingTags);
-	int le = lexrc.lex();
+	int le = lex.lex();
 	switch (le) {
 	case Lexer::LEX_UNDEF:
-		lexrc.printError("Unknown spacing token `$$Token'");
+		lex.printError("Unknown spacing token `$$Token'");
 		return;
 	default: break;
 	}
@@ -794,8 +759,8 @@ void Layout::readSpacing(Lexer & lexrc)
 		spacing.set(Spacing::Double);
 		break;
 	case ST_OTHER:
-		lexrc.next();
+		lex.next();
-		spacing.set(Spacing::Other, lexrc.getString());
+		spacing.set(Spacing::Other, lex.getString());
 		break;
 	}
 }
@@ -835,14 +800,4 @@ bool Layout::operator==(Layout const & rhs) const
 }
 
 
-Layout * Layout::forCaption()
-{
-	Layout * lay = new Layout();
-	lay->name_ = from_ascii("Caption");
-	lay->latexname_ = "caption";
-	lay->latextype = LATEX_COMMAND;
-	lay->optionalargs = 1;
-	return lay;
-}
-
 } // namespace lyx

src/Layout.h

@@ -234,10 +234,12 @@ public:
 	/// until it has proper support for the caption inset (JMarc)
 	static Layout * forCaption();
 
-private:
 	/// Name of the layout/paragraph environment
 	docstring name_;
+	/// LaTeX name for environment
+	std::string latexname_;
 
+private:
 	/** Name of an layout that has replaced this layout.
 	    This is used to rename a layout, while keeping backward
 	    compatibility
@@ -250,8 +252,6 @@ private:
 	 */
 	docstring depends_on_;
 
-	/// LaTeX name for environment
-	std::string latexname_;
 	/// Label string. "Abstract", "Reference", "Caption"...
 	docstring labelstring_;
 	///

src/tex2lyx/Context.cpp

@@ -24,7 +24,7 @@ namespace lyx {
 
 namespace {
 
-void begin_layout(ostream & os, LayoutPtr const & layout, TeXFont const & font,
+void begin_layout(ostream & os, Layout const * const & layout, TeXFont const & font,
 		  TeXFont const & normalfont)
 {
 	os << "\n\\begin_layout " << to_utf8(layout->name()) << "\n";
@@ -84,7 +84,7 @@ bool Context::empty = true;
 
 Context::Context(bool need_layout_,
 		 TeX2LyXDocClass const & textclass_,
-		 LayoutPtr layout_, LayoutPtr parent_layout_,
+		 Layout const * layout_, Layout const * parent_layout_,
 		 TeXFont font_)
 	: need_layout(need_layout_),
 	  need_end_layout(false), need_end_deeper(false),

src/tex2lyx/Context.h

@@ -78,8 +78,8 @@ class Context {
 public:
 	Context(bool need_layout_,
 		TeX2LyXDocClass const & textclass_,
-		LayoutPtr layout_ = LayoutPtr(),
+		Layout const * layout_ = 0,
-		LayoutPtr parent_layout_= LayoutPtr(),
+		Layout const * parent_layout_= 0,
 		TeXFont font_ = TeXFont());
 	~Context();
 
@@ -142,9 +142,9 @@ public:
 	/// The textclass of the document. Could actually be a global variable
 	TeX2LyXDocClass const & textclass;
 	/// The layout of the current paragraph
-	LayoutPtr layout;
+	Layout const * layout;
 	/// The layout of the outer paragraph (for environment layouts)
-	LayoutPtr parent_layout;
+	Layout const * parent_layout;
 	/// font attributes of this context
 	TeXFont font;
 	/// font attributes of normal text

src/tex2lyx/tex2lyx.cpp

@@ -40,10 +40,6 @@ using namespace lyx::support::os;
 
 namespace lyx {
 
-// Hacks to allow the thing to link in the lyxlayout stuff
-LayoutPtr captionlayout;
-
-
 string const trim(string const & a, char const * p)
 {
 	// BOOST_ASSERT(p);
@@ -400,7 +396,6 @@ void tex2lyx(istream & is, ostream & os)
 	stringstream ss;
 	TeX2LyXDocClass textclass;
 	parse_preamble(p, ss, documentclass, textclass);
-	captionlayout = LayoutPtr(Layout::forCaption());
 
 	active_environments.push_back("document");
 	Context context(true, textclass);

src/tex2lyx/tex2lyx.h

@@ -26,8 +26,6 @@ namespace lyx {
 
 namespace support { class FileName; }
 
-typedef Layout const * LayoutPtr;
-
 class Context;
 
 /// A trivial subclass, just to give us a public default constructor
@@ -40,7 +38,6 @@ void parse_preamble(Parser & p, std::ostream & os,
 
 /// used packages with options
 extern std::map<std::string, std::vector<std::string> > used_packages;
-extern LayoutPtr captionlayout;
 
 /// in text.cpp
 std::string translate_len(std::string const &);

src/tex2lyx/text.cpp

@@ -424,23 +424,37 @@ void handle_comment(ostream & os, string const & s, Context & context)
 }
 
 
-LayoutPtr findLayout(TextClass const & textclass, string const & name)
+Layout const * findLayout(TextClass const & textclass, string const & name)
 {
 	DocumentClass::const_iterator lit = textclass.begin();
 	DocumentClass::const_iterator len = textclass.end();
 	for (; lit != len; ++lit)
 		if (lit->latexname() == name)
 			return &*lit;
-	return LayoutPtr();
+	return 0;
 }
 
 
 void eat_whitespace(Parser &, ostream &, Context &, bool);
 
 
+Layout * captionlayout()
+{
+	static Layout * lay = 0;
+	if (!lay) {
+		lay = new Layout;
+		lay->name_ = from_ascii("Caption");
+		lay->latexname_ = "caption";
+		lay->latextype = LATEX_COMMAND;
+		lay->optionalargs = 1;
+	}
+	return lay;
+}
+
+
 void output_command_layout(ostream & os, Parser & p, bool outer,
 			   Context & parent_context,
-			   LayoutPtr newlayout)
+			   Layout const * newlayout)
 {
 	parent_context.check_end_layout(os);
 	Context context(true, parent_context.textclass, newlayout,
@@ -696,7 +710,7 @@ void parse_unknown_environment(Parser & p, string const & name, ostream & os,
 void parse_environment(Parser & p, ostream & os, bool outer,
 		       Context & parent_context)
 {
-	LayoutPtr newlayout;
+	Layout const * newlayout;
 	string const name = p.getArg('{', '}');
 	const bool is_starred = suffixIs(name, '*');
 	string const unstarred_name = rtrim(name, "*");
@@ -1105,7 +1119,7 @@ void parse_noweb(Parser & p, ostream & os, Context & context)
 void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
 		Context & context)
 {
-	LayoutPtr newlayout;
+	Layout const * newlayout = 0;
 	// store the current selectlanguage to be used after \foreignlanguage
 	string selectlang;
 	// Store the latest bibliographystyle (needed for bibtex inset)
@@ -1539,9 +1553,9 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
 		// Special handling for \caption
 		// FIXME: remove this when InsetCaption is supported.
 		else if (context.new_layout_allowed &&
-			 t.cs() == captionlayout->latexname()) {
+			 t.cs() == captionlayout()->latexname()) {
 			output_command_layout(os, p, outer, context, 
-					      captionlayout);
+					      captionlayout());
 			p.skip_spaces();
 		}