[LyX/master] Remove InsetArgument 999 workaround

[Georg Baum]

commit f4417e8125d6f2c1137977b328a421b163525229
Author: Georg Baum <b...@lyx.org>
Date:   Fri Dec 26 22:13:09 2014 +0100

    Remove InsetArgument 999 workaround
    
    Since we iterate through all arguments we can as well compute the correct id
    directly.

diff --git a/src/tex2lyx/test/test-structure.lyx.lyx 
b/src/tex2lyx/test/test-structure.lyx.lyx
index 6a52a84..107bb66 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 c7b6dd5..d1da45b 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 6ab1549..721db0c 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);
                }