feature requests from ochafik

5 files changed, 79 insertions, 12 deletions

diff --git a/etc/build.properties b/etc/build.properties
index 4d465d7..0afa8f0 100644
--- a/etc/build.properties
+++ b/etc/build.properties
@@ -1,4 +1,4 @@
-global.version				= 1.2.5
+global.version				= 1.2.6
 global.name					= anarres-cpp
 
 global.dir.arch				= ${global.dir.root}/arch
diff --git a/src/java/org/anarres/cpp/Feature.java b/src/java/org/anarres/cpp/Feature.java
index 649f135..04a68b7 100644
--- a/src/java/org/anarres/cpp/Feature.java
+++ b/src/java/org/anarres/cpp/Feature.java
@@ -31,9 +31,12 @@ public enum Feature {
 	CSYNTAX,
 	/** Preserves comments in the lexed output. */
 	KEEPCOMMENTS,
+	/** Preserves comments in the lexed output, even when inactive. */
+	KEEPALLCOMMENTS,
 	VERBOSE,
 	DEBUG,
 
 	/** Supports lexing of objective-C. */
-	OBJCSYNTAX
+	OBJCSYNTAX,
+	INCLUDENEXT
 }
diff --git a/src/java/org/anarres/cpp/MacroTokenSource.java b/src/java/org/anarres/cpp/MacroTokenSource.java
index 53b0932..0759875 100644
--- a/src/java/org/anarres/cpp/MacroTokenSource.java
+++ b/src/java/org/anarres/cpp/MacroTokenSource.java
@@ -56,7 +56,8 @@ import static org.anarres.cpp.Token.*;
 		return super.isExpanding(m);
 	}
 
-	private static void escape(StringBuilder buf, CharSequence cs) {
+	/* XXX Called from Preprocessor [ugly]. */
+	/* pp */ static void escape(StringBuilder buf, CharSequence cs) {
 		for (int i = 0; i < cs.length(); i++) {
 			char	c = cs.charAt(i);
 			switch (c) {
diff --git a/src/java/org/anarres/cpp/Main.java b/src/java/org/anarres/cpp/Main.java
index 4fa17f1..00f47f0 100644
--- a/src/java/org/anarres/cpp/Main.java
+++ b/src/java/org/anarres/cpp/Main.java
@@ -109,6 +109,9 @@ public class Main {
 		pp.addMacro("__JCPP__");
 		pp.getSystemIncludePath().add("/usr/local/include");
 		pp.getSystemIncludePath().add("/usr/include");
+		pp.getFrameworksPath().add("/System/Library/Frameworks");
+		pp.getFrameworksPath().add("/Library/Frameworks");
+		pp.getFrameworksPath().add("/Local/Library/Frameworks");
 
         GETOPT: while ((c = g.getopt()) != -1) {
             switch (c) {
diff --git a/src/java/org/anarres/cpp/Preprocessor.java b/src/java/org/anarres/cpp/Preprocessor.java
index 282c432..9e8ce89 100644
--- a/src/java/org/anarres/cpp/Preprocessor.java
+++ b/src/java/org/anarres/cpp/Preprocessor.java
@@ -92,6 +92,7 @@ public class Preprocessor implements Closeable {
 	};
 	private static final Macro		__LINE__ = new Macro(INTERNAL, "__LINE__");
 	private static final Macro		__FILE__ = new Macro(INTERNAL, "__FILE__");
+	private static final Macro		__COUNTER__ = new Macro(INTERNAL, "__COUNTER__");
 
 	private List<Source>			inputs;
 
@@ -100,6 +101,9 @@ public class Preprocessor implements Closeable {
 	private Stack<State>			states;
 	private Source					source;
 
+	/* Miscellaneous support. */
+	private int						counter;
+
 	/* Support junk to make it work like cpp */
 	private List<String>			quoteincludepath;	/* -iquote */
 	private List<String>			sysincludepath;		/* -I */
@@ -111,14 +115,20 @@ public class Preprocessor implements Closeable {
 
 	public Preprocessor() {
 		this.inputs = new ArrayList<Source>();
+
 		this.macros = new HashMap<String,Macro>();
 		macros.put(__LINE__.getName(), __LINE__);
 		macros.put(__FILE__.getName(), __FILE__);
+		macros.put(__COUNTER__.getName(), __COUNTER__);
 		this.states = new Stack<State>();
 		states.push(new State());
 		this.source = null;
+
+		this.counter = 0;
+
 		this.quoteincludepath = new ArrayList<String>();
 		this.sysincludepath = new ArrayList<String>();
+		this.frameworkspath = new ArrayList<String>();
 		this.features = EnumSet.noneOf(Feature.class);
 		this.warnings = EnumSet.noneOf(Warning.class);
 		this.filesystem = new JavaFileSystem();
@@ -514,10 +524,13 @@ public class Preprocessor implements Closeable {
 	/* XXX Make this include the NL, and make all cpp directives eat
 	 * their own NL. */
 	private Token line_token(int line, String name, String extra) {
-		return new Token(P_LINE, line, 0,
-			"#line " + line + " \"" + name + "\"" + extra + "\n",
-			null
-				);
+		StringBuilder	buf = new StringBuilder();
+		buf.append("#line ").append(line)
+			.append(" \"");
+		/* XXX This call to escape(name) is correct but ugly. */
+		MacroTokenSource.escape(buf, name);
+		buf.append("\"").append(extra).append("\n");
+		return new Token(P_LINE, line, 0, buf.toString(), null);
 	}
 
 	private Token source_token()
@@ -543,6 +556,7 @@ public class Preprocessor implements Closeable {
 				continue;
 			}
 			Token	tok = s.token();
+			/* XXX Refactor with skipline() */
 			if (tok.getType() == EOF && s.isAutopop()) {
 				// System.out.println("Autopop " + s);
 				pop_source();
@@ -591,12 +605,30 @@ public class Preprocessor implements Closeable {
 	 *
 	 * The metadata on the token will be correct, which is better
 	 * than generating a new one.
+	 *
+	 * This method can, as of recent patches, return a P_LINE token.
 	 */
 	private Token source_skipline(boolean white)
 						throws IOException,
 								LexerException {
 		// (new Exception("skipping line")).printStackTrace(System.out);
-		return source.skipline(white);
+		Source	s = getSource();
+		Token	tok = s.skipline(white);
+		/* XXX Refactor with source_token() */
+		if (tok.getType() == EOF && s.isAutopop()) {
+			// System.out.println("Autopop " + s);
+			pop_source();
+			Source	t = getSource();
+			if (getFeature(Feature.LINEMARKERS)
+					&& s.isNumbered()
+					&& t != null) {
+				/* We actually want 'did the nested source
+				 * contain a newline token', which isNumbered()
+				 * approximates. This is not perfect, but works. */
+				return line_token(t.getLine() + 1, t.getName(), " 2");
+			}
+		}
+		return tok;
 	}
 
 	/* processes and expands a macro. */
@@ -773,6 +805,17 @@ public class Preprocessor implements Closeable {
 							text, text) }
 						), true);
 		}
+		else if (m == __COUNTER__) {
+			/* This could equivalently have been done by adding
+			 * a special Macro subclass which overrides getTokens(). */
+			int	value = this.counter++;
+			push_source(new FixedTokenSource(
+					new Token[] { new Token(INTEGER,
+							orig.getLine(), orig.getColumn(),
+							String.valueOf(value),
+							Integer.valueOf(value)) }
+						), true);
+		}
 		else {
 			push_source(new MacroTokenSource(m, args), true);
 		}
@@ -1018,6 +1061,8 @@ public class Preprocessor implements Closeable {
 		// System.out.println("Try to include " + file);
 		if (!file.isFile())
 			return false;
+		if (getFeature(Feature.DEBUG))
+			System.err.println("pp: including " + file);
 		push_source(file.getSource(), true);
 		return true;
 	}
@@ -1070,7 +1115,7 @@ public class Preprocessor implements Closeable {
 		error(line, 0, buf.toString());
 	}
 
-	private Token include()
+	private Token include(boolean next)
 						throws IOException,
 								LexerException {
 		LexerSource	lexer = (LexerSource)source;
@@ -1127,7 +1172,7 @@ public class Preprocessor implements Closeable {
 			/* 'tok' is the 'nl' after the include. We use it after the
 			 * #line directive. */
 			if (getFeature(Feature.LINEMARKERS))
-				return line_token(1, name, " 1");
+				return line_token(1, source.getName(), " 1");
 			return tok;
 		}
 		finally {
@@ -1546,6 +1591,10 @@ public class Preprocessor implements Closeable {
 					case CCOMMENT:
 					case CPPCOMMENT:
 						// Patch up to preserve whitespace.
+						if (getFeature(Feature.KEEPALLCOMMENTS))
+							return tok;
+						if (!isActive())
+							return toWhitespace(tok);
 						if (getFeature(Feature.KEEPCOMMENTS))
 							return tok;
 						return toWhitespace(tok);
@@ -1686,7 +1735,18 @@ public class Preprocessor implements Closeable {
 							if (!isActive())
 								return source_skipline(false);
 							else
-								return include();
+								return include(false);
+							// break;
+						case PP_INCLUDE_NEXT:
+							if (!isActive())
+								return source_skipline(false);
+							if (!getFeature(Feature.INCLUDENEXT)) {
+								error(tok,
+									"Directive include_next not enabled"
+									);
+								return source_skipline(false);
+							}
+							return include(true);
 							// break;
 
 						case PP_WARNING:
@@ -1865,7 +1925,7 @@ public class Preprocessor implements Closeable {
 	}
 
 #set ($i = 1)	/* First ppcmd is 1, not 0. */
-#set ($ppcmds = [ "define", "elif", "else", "endif", "error", "if", "ifdef", "ifndef", "include", "line", "pragma", "undef", "warning" ])
+#set ($ppcmds = [ "define", "elif", "else", "endif", "error", "if", "ifdef", "ifndef", "include", "line", "pragma", "undef", "warning", "include_next", "import" ])
 #foreach ($ppcmd in $ppcmds)
 	private static final int PP_$ppcmd.toUpperCase() = $i;
 #set ($i = $i + 1)