diff options
Diffstat (limited to 'src')
-rw-r--r-- | src/input/test0.c (renamed from src/input/test.c) | 0 | ||||
-rw-r--r-- | src/input/test0.h | 2 | ||||
-rw-r--r-- | src/input/test1.c | 2 | ||||
-rw-r--r-- | src/java/org/anarres/cpp/Main.java | 61 | ||||
-rw-r--r-- | src/java/org/anarres/cpp/Preprocessor.java | 79 | ||||
-rw-r--r-- | src/tests/org/anarres/cpp/CppReaderTestCase.java | 2 | ||||
-rw-r--r-- | src/tests/org/anarres/cpp/MainTestCase.java | 2 |
7 files changed, 86 insertions, 62 deletions
diff --git a/src/input/test.c b/src/input/test0.c index 150b759..150b759 100644 --- a/src/input/test.c +++ b/src/input/test0.c diff --git a/src/input/test0.h b/src/input/test0.h index 72db7b7..b6697c6 100644 --- a/src/input/test0.h +++ b/src/input/test0.h @@ -1,7 +1,7 @@ test0start_2 -#include <test1.h> +#include "test1.h" test0end___6 diff --git a/src/input/test1.c b/src/input/test1.c new file mode 100644 index 0000000..3e6fbda --- /dev/null +++ b/src/input/test1.c @@ -0,0 +1,2 @@ +#include "./test0.h" +#include <test1.h> diff --git a/src/java/org/anarres/cpp/Main.java b/src/java/org/anarres/cpp/Main.java index 253b9ad..e62ca83 100644 --- a/src/java/org/anarres/cpp/Main.java +++ b/src/java/org/anarres/cpp/Main.java @@ -53,15 +53,6 @@ public class Main { } } - private List<String> i_default; - private List<String> i_system; - private List<String> i_user; - private List<String> i_quote; - private Map<String,String> d_default; - private Map<String,String> d_user; - private Set<Warning> warnings; - private List<String> f_include; - private static final Option[] OPTS = new Option[] { new Option("help", LongOpt.NO_ARGUMENT, 'h', null, "Displays help and usage information."), @@ -69,15 +60,17 @@ public class Main { "Defines the given macro."), new Option("undefine", LongOpt.REQUIRED_ARGUMENT, 'U', "name", "Undefines the given macro, previously either builtin or defined using -D."), - new Option("include", LongOpt.REQUIRED_ARGUMENT, 'i', "file", + new Option("include", LongOpt.REQUIRED_ARGUMENT, 1, "file", "Process file as if \"#" + "include \"file\"\" appeared as the first line of the primary source file."), new Option("incdir", LongOpt.REQUIRED_ARGUMENT, 'I', "dir", "Adds the directory dir to the list of directories to be searched for header files."), + new Option("iquote", LongOpt.REQUIRED_ARGUMENT, 0, "dir", + "Adds the directory dir to the list of directories to be searched for header files included using \"\"."), new Option("warning", LongOpt.REQUIRED_ARGUMENT, 'W', "type", "Enables the named warning class (" + getWarnings() + ")."), new Option("no-warnings", LongOpt.NO_ARGUMENT, 'w', null, "Disables ALL warnings."), - new Option("version", LongOpt.NO_ARGUMENT, 'V', null, + new Option("version", LongOpt.NO_ARGUMENT, 2, null, "Prints jcpp's version number (" + Version.getVersion() + ")"), }; @@ -96,15 +89,6 @@ public class Main { (new Main()).run(OPTS, args); } - public Main() { - i_default = new ArrayList<String>(); - i_system = new ArrayList<String>(); - i_user = new ArrayList<String>(); - i_quote = new ArrayList<String>(); - d_default = new HashMap<String,String>(); - d_user = new HashMap<String,String>(); - } - public void run(Option[] opts, String[] args) throws Exception { String sopts = getShortOpts(opts); Getopt g = new Getopt("jcpp", args, sopts, opts); @@ -115,23 +99,27 @@ public class Main { Preprocessor pp = new Preprocessor(); pp.addFeature(Feature.LINEMARKERS); + pp.addMacro("__JCPP__"); + GETOPT: while ((c = g.getopt()) != -1) { switch (c) { case 'D': arg = g.getOptarg(); idx = arg.indexOf('='); if (idx == -1) - d_user.put(arg, "1"); + pp.addMacro(arg); else - d_user.put(arg.substring(0, idx), + pp.addMacro(arg.substring(0, idx), arg.substring(idx + 1)); break; case 'U': - d_default.remove(g.getOptarg()); - d_user.remove(g.getOptarg()); + pp.getMacros().remove(g.getOptarg()); break; case 'I': - i_user.add(g.getOptarg()); + pp.getSystemIncludePath().add(g.getOptarg()); + break; + case 0: // --iquote= + pp.getQuoteIncludePath().add(g.getOptarg()); break; case 'W': arg = g.getOptarg().toUpperCase(); @@ -144,14 +132,14 @@ public class Main { case 'w': pp.getWarnings().clear(); break; - case 'i': + case 1: // --include= // pp.addInput(new File(g.getOptarg())); // Comply exactly with spec. pp.addInput(new StringLexerSource( "#" + "include \"" + g.getOptarg() + "\"\n" )); break; - case 'V': + case 2: // --version System.out.println("Anarres Java C Preprocessor version " + Version.getVersion()); System.out.println("Copyright (C) 2008 Shevek (http://www.anarres.org/)."); System.out.println("This is free software; see the source for copying conditions. There is NO"); @@ -166,14 +154,10 @@ public class Main { } } - /* XXX include-path, include-files. */ - - for (Map.Entry<String,String> e : d_default.entrySet()) - pp.addMacro(e.getKey(), e.getValue()); - for (Map.Entry<String,String> e : d_user.entrySet()) - pp.addMacro(e.getKey(), e.getValue()); - - /* XXX Include paths. */ + List<String> path = pp.getSystemIncludePath(); + path.add("/usr/local/include"); + path.add("/usr/include"); + path.add("/usr/lib/gcc/i686-pc-linux-gnu/4.1.2/include"); for (int i = g.getOptind(); i < args.length; i++) pp.addInput(new FileLexerSource(new File(args[i]))); @@ -203,6 +187,8 @@ public class Main { StringBuilder buf = new StringBuilder(); for (int i = 0; i < opts.length; i++) { char c = (char)opts[i].getVal(); + if (!Character.isLetterOrDigit(c)) + continue; for (int j = 0; j < buf.length(); j++) if (buf.charAt(j) == c) throw new Exception( @@ -294,7 +280,8 @@ public class Main { line.append('=').append(opt.eg); break; } - line.append(" (-").append((char)opt.getVal()).append(")"); + if (Character.isLetterOrDigit(opt.getVal())) + line.append(" (-").append((char)opt.getVal()).append(")"); if (line.length() < 30) { while (line.length() < 30) line.append(' '); @@ -323,7 +310,7 @@ public class Main { Source source = new FileLexerSource(new File(args[0])); Preprocessor pp = new Preprocessor(source); - pp.setIncludePath(path); + pp.setSystemIncludePath(path); for (int i = 1; i < args.length; i++) { pp.push_source(new FileLexerSource(new File(args[i])),true); diff --git a/src/java/org/anarres/cpp/Preprocessor.java b/src/java/org/anarres/cpp/Preprocessor.java index 63304b0..b3b1169 100644 --- a/src/java/org/anarres/cpp/Preprocessor.java +++ b/src/java/org/anarres/cpp/Preprocessor.java @@ -54,7 +54,8 @@ public class Preprocessor { private Stack<State> states; private Source source; - private List<String> path; + private List<String> quoteincludepath; /* -iquote */ + private List<String> sysincludepath; /* -I */ private Set<Feature> features; private Set<Warning> warnings; private PreprocessorListener listener; @@ -66,7 +67,8 @@ public class Preprocessor { this.states = new Stack<State>(); states.push(new State()); this.source = null; - this.path = null; + this.quoteincludepath = new ArrayList<String>(); + this.sysincludepath = new ArrayList<String>(); this.features = EnumSet.noneOf(Feature.class); features.add(Feature.DIGRAPHS); features.add(Feature.TRIGRAPHS); @@ -230,11 +232,27 @@ public class Preprocessor { } /** - * Sets the include path used by this Preprocessor. + * Sets the user include path used by this Preprocessor. */ /* Note for future: Create an IncludeHandler? */ - public void setIncludePath(List<String> path) { - this.path = path; + public void setQuoteIncludePath(List<String> path) { + this.quoteincludepath = path; + } + + public List<String> getQuoteIncludePath() { + return quoteincludepath; + } + + /** + * Sets the system include path used by this Preprocessor. + */ + /* Note for future: Create an IncludeHandler? */ + public void setSystemIncludePath(List<String> path) { + this.sysincludepath = path; + } + + public List<String> getSystemIncludePath() { + return sysincludepath; } /** @@ -804,6 +822,28 @@ public class Preprocessor { return source_skipline(true); } + protected boolean include(File file) + throws IOException, + LexerException { + if (!file.exists()) + return false; + if (!file.isFile()) + return false; + push_source(new FileLexerSource(file), true); + return true; + } + + protected boolean include(Iterable<String> path, String name) + throws IOException, + LexerException { + for (String dir : path) { + File file = new File(dir + File.separator + name); + if (include(file)) + return true; + } + return false; + } + /** * Handles a include directive. * @@ -820,27 +860,22 @@ public class Preprocessor { if (dir == null) dir = new File("/"); File file = new File(dir, name); - // System.err.println("Include: " + file); - if (file.exists() && file.isFile()) { - push_source(new FileLexerSource(file), true); + if (include(file)) + return; + if (include(quoteincludepath, name)) return; - } } - if (path != null) { - for (int i = 0; i < path.size(); i++) { - File file = new File( - path.get(i) + File.separator + name - ); - if (file.exists() && file.isFile()) { - // System.err.println("Include: " + file); - push_source(new FileLexerSource(file), true); - return; - } - } - } + if (include(sysincludepath, name)) + return; - error(line, 0, "File not found: " + name + " in " + path); + StringBuilder buf = new StringBuilder(); + if (quoted) + for (String dir : quoteincludepath) + buf.append(" ").append(dir); + for (String dir : sysincludepath) + buf.append(" ").append(dir); + error(line, 0, "File not found: " + name + " in " + buf); } private Token include() diff --git a/src/tests/org/anarres/cpp/CppReaderTestCase.java b/src/tests/org/anarres/cpp/CppReaderTestCase.java index 6097436..92ade21 100644 --- a/src/tests/org/anarres/cpp/CppReaderTestCase.java +++ b/src/tests/org/anarres/cpp/CppReaderTestCase.java @@ -14,7 +14,7 @@ public class CppReaderTestCase extends BaseTestCase implements Test { System.out.println("Testing " + in + " => " + out); StringReader r = new StringReader(in); CppReader p = new CppReader(r); - p.getPreprocessor().setIncludePath( + p.getPreprocessor().setSystemIncludePath( Collections.singletonList("src/input") ); p.getPreprocessor().getFeatures().add(Feature.LINEMARKERS); diff --git a/src/tests/org/anarres/cpp/MainTestCase.java b/src/tests/org/anarres/cpp/MainTestCase.java index a42f3d2..313a463 100644 --- a/src/tests/org/anarres/cpp/MainTestCase.java +++ b/src/tests/org/anarres/cpp/MainTestCase.java @@ -9,7 +9,7 @@ import static org.anarres.cpp.Token.*; public class MainTestCase extends BaseTestCase { public void testMain() throws Exception { - Main.main(new String[] { "-V" }); + Main.main(new String[] { "--version" }); } } |