Alfredo Braunstein <[EMAIL PROTECTED]> writes:

| +Buffer * newFile(string const & name, string tname, bool isNamed)

This should take "string const &" not just "string"

| +/// Make a new file (buffer) using a template
| +Buffer * newFile(string const &, string, bool isNamed = false);

and here.
Also please add variables on the args.

| Index: frontends/controllers/tex_helpers.C
| ===================================================================
| RCS file:
| /usr/local/lyx/cvsroot/lyx-devel/src/frontends/controllers/tex_helpers.C,v
| retrieving revision 1.10
| diff -u -p -u -r1.10 tex_helpers.C
| --- frontends/controllers/tex_helpers.C 21 Feb 2003 15:36:29 -0000      1.10
| +++ frontends/controllers/tex_helpers.C 19 Jun 2003 22:55:42 -0000
| @@ -85,7 +85,6 @@ string const getTexFileList(string const
|                 return str_out;
|         }
|         vector<string> dbaseWP = listWithoutPath(dbase);
| -       lyx::eliminate_duplicates(dbaseWP);
|         string const str_out =
|                 getStringFromVector(dbaseWP, "\n");
|         return str_out;

Is it intentional to have that here?

Looks good.

-- 
        Lgb

Reply via email to