On 04/09/2012 09:09 AM, Vincent van Ravesteijn wrote:
Op 7-12-2011 17:03, kuem...@lyx.org schreef:
Author: kuemmel
Date: Wed Dec 7 17:03:37 2011
New Revision: 40417
URL: http://www.lyx.org/trac/changeset/40417
Log:
clang: std::string is used here, fwd is not enough
Modified:
lyx-devel/trunk/src/Lexer.h
Modified: lyx-devel/trunk/src/Lexer.h
==============================================================================
--- lyx-devel/trunk/src/Lexer.h Wed Dec 7 16:17:06 2011 (r40416)
+++ lyx-devel/trunk/src/Lexer.h Wed Dec 7 17:03:37 2011 (r40417)
@@ -18,7 +18,7 @@
#define LEXER_H
#include "support/strfwd.h"
-
+#include<string>
namespace lyx {
Also for branch ?
There are multiple (2) reports that clang can't compile LyX:
- https://trac.macports.org/ticket/32109
- http://thread.gmane.org/gmane.editors.lyx.devel/141322
It'd be nice if we had more information. Lexer.h is included in a lot of
other files,
and this will include <string> now in all of them.
Is there any way to test for clang? If not, does it make any sense to
include
strfwd.h if we're also including <string>?
Richard
