From f4417e8125d6f2c1137977b328a421b163525229 Mon Sep 17 00:00:00 2001
From: Georg Baum <baum@lyx.org>
Date: Fri, 26 Dec 2014 22:13:09 +0100
Subject: [PATCH] Remove InsetArgument 999 workaround

Since we iterate through all arguments we can as well compute the correct id
directly.
---
 src/tex2lyx/test/test-structure.lyx.lyx |  2 +-
 src/tex2lyx/test/test.lyx.lyx           |  2 +-
 src/tex2lyx/text.cpp                    | 20 ++++++--------------
 3 files changed, 8 insertions(+), 16 deletions(-)

diff --git a/src/tex2lyx/test/test-structure.lyx.lyx b/src/tex2lyx/test/test-structure.lyx.lyx
index 6a52a847ee..107bb669e1 100644
--- a/src/tex2lyx/test/test-structure.lyx.lyx
+++ b/src/tex2lyx/test/test-structure.lyx.lyx
@@ -188,7 +188,7 @@ A section
 
 \begin_layout Section
 
-\begin_inset Argument 999
+\begin_inset Argument 1
 status collapsed
 
 
diff --git a/src/tex2lyx/test/test.lyx.lyx b/src/tex2lyx/test/test.lyx.lyx
index c7b6dd5e69..d1da45bafa 100644
--- a/src/tex2lyx/test/test.lyx.lyx
+++ b/src/tex2lyx/test/test.lyx.lyx
@@ -349,7 +349,7 @@ Final text
 \end_deeper
 \begin_layout Section
 
-\begin_inset Argument 999
+\begin_inset Argument 1
 status collapsed
 
 
diff --git a/src/tex2lyx/text.cpp b/src/tex2lyx/text.cpp
index 6ab1549702..721db0ce6b 100644
--- a/src/tex2lyx/text.cpp
+++ b/src/tex2lyx/text.cpp
@@ -637,21 +637,18 @@ void output_command_layout(ostream & os, Parser & p, bool outer,
 	}
 	context.check_deeper(os);
 	context.check_layout(os);
+	int i = 0;
 	Layout::LaTeXArgMap::const_iterator lait = context.layout->latexargs().begin();
 	Layout::LaTeXArgMap::const_iterator const laend = context.layout->latexargs().end();
 	for (; lait != laend; ++lait) {
+		++i;
 		eat_whitespace(p, os, context, false);
 		if (lait->second.mandatory) {
 			if (p.next_token().cat() != catBegin)
 				break;
 			p.get_token(); // eat '{'
-			// FIXME: Just a workaround. InsetArgument::updateBuffer
-			//        will compute a proper ID for all "999" Arguments
-			//        (which is also what lyx2lyx produces).
-			//        However, tex2lyx should be able to output proper IDs
-			//        itself.
-			begin_inset(os, "Argument 999\n");
-			os << "status collapsed\n\n";
+			begin_inset(os, "Argument ");
+			os << i << "\nstatus collapsed\n\n";
 			parse_text_in_inset(p, os, FLAG_BRACE_LAST, outer, context);
 			end_inset(os);
 		} else {
@@ -659,13 +656,8 @@ void output_command_layout(ostream & os, Parser & p, bool outer,
 			    p.next_token().character() != '[')
 				break;
 			p.get_token(); // eat '['
-			// FIXME: Just a workaround. InsetArgument::updateBuffer
-			//        will compute a proper ID for all "999" Arguments
-			//        (which is also what lyx2lyx produces).
-			//        However, tex2lyx should be able to output proper IDs
-			//        itself.
-			begin_inset(os, "Argument 999\n");
-			os << "status collapsed\n\n";
+			begin_inset(os, "Argument ");
+			os << i << "\nstatus collapsed\n\n";
 			parse_text_in_inset(p, os, FLAG_BRACK_LAST, outer, context);
 			end_inset(os);
 		}
-- 
2.39.2