diff options
| -rw-r--r-- | base/inifile.cc | 91 | ||||
| -rw-r--r-- | base/inifile.hh | 110 | ||||
| -rw-r--r-- | base/statistics.cc | 69 | ||||
| -rw-r--r-- | base/statistics.hh | 53 | ||||
| -rw-r--r-- | cpu/base_cpu.cc | 1 | ||||
| -rw-r--r-- | cpu/exetrace.cc | 1 | ||||
| -rw-r--r-- | dev/etherint.cc | 1 | ||||
| -rw-r--r-- | sim/sim_object.hh | 1 | ||||
| -rw-r--r-- | sim/system.cc | 1 | ||||
| -rw-r--r-- | test/Makefile | 2 | ||||
| -rw-r--r-- | test/foo.ini | 3 | ||||
| -rw-r--r-- | test/initest.cc | 135 |
12 files changed, 362 insertions, 106 deletions
diff --git a/base/inifile.cc b/base/inifile.cc index d5436fba8..7fd2f5568 100644 --- a/base/inifile.cc +++ b/base/inifile.cc @@ -59,8 +59,8 @@ IniFile::IniFile() IniFile::~IniFile() { - ConfigTable::iterator i = table.begin(); - ConfigTable::iterator end = table.end(); + SectionTable::iterator i = table.begin(); + SectionTable::iterator end = table.end(); while (i != end) { delete (*i).second; @@ -75,6 +75,16 @@ IniFile::loadCPP(const string &file, vector<char *> &cppArgs) { int fd[2]; + // Open the file just to verify that we can. Otherwise if the + // file doesn't exist or has bad permissions the user will get + // confusing errors from cpp/g++. + ifstream tmpf(file.c_str()); + + if (!tmpf.is_open()) + return false; + + tmpf.close(); + #ifdef CPP_PIPE if (pipe(fd) == -1) return false; @@ -183,7 +193,8 @@ IniFile::Entry::getValue() const void IniFile::Section::addEntry(const std::string &entryName, - const std::string &value) + const std::string &value, + bool append) { EntryTable::iterator ei = table.find(entryName); @@ -191,6 +202,10 @@ IniFile::Section::addEntry(const std::string &entryName, // new entry table[entryName] = new Entry(value); } + else if (append) { + // append new reult to old entry + ei->second->appendValue(value); + } else { // override old entry ei->second->setValue(value); @@ -198,6 +213,27 @@ IniFile::Section::addEntry(const std::string &entryName, } +bool +IniFile::Section::add(const std::string &assignment) +{ + string::size_type offset = assignment.find('='); + if (offset == string::npos) // no '=' found + return false; + + // if "+=" rather than just "=" then append value + bool append = (assignment[offset-1] == '+'); + + string entryName = assignment.substr(0, append ? offset-1 : offset); + string value = assignment.substr(offset + 1); + + eat_white(entryName); + eat_white(value); + + addEntry(entryName, value, append); + return true; +} + + IniFile::Entry * IniFile::Section::findEntry(const std::string &entryName) const { @@ -212,10 +248,10 @@ IniFile::Section::findEntry(const std::string &entryName) const IniFile::Section * IniFile::addSection(const string §ionName) { - ConfigTable::iterator ci = table.find(sectionName); + SectionTable::iterator i = table.find(sectionName); - if (ci != table.end()) { - return ci->second; + if (i != table.end()) { + return i->second; } else { // new entry @@ -229,9 +265,9 @@ IniFile::addSection(const string §ionName) IniFile::Section * IniFile::findSection(const string §ionName) const { - ConfigTable::const_iterator ci = table.find(sectionName); + SectionTable::const_iterator i = table.find(sectionName); - return (ci == table.end()) ? NULL : ci->second; + return (i == table.end()) ? NULL : i->second; } @@ -248,21 +284,10 @@ IniFile::add(const string &str) string sectionName = str.substr(0, offset); string rest = str.substr(offset + 1); - offset = rest.find('='); - if (offset == string::npos) // no '='found - return false; - - string entryName = rest.substr(0, offset); - string value = rest.substr(offset + 1); - eat_white(sectionName); - eat_white(entryName); - eat_white(value); - Section *s = addSection(sectionName); - s->addEntry(entryName, value); - return true; + return s->add(rest); } bool @@ -294,14 +319,8 @@ IniFile::load(istream &f) if (section == NULL) continue; - string::size_type offset = line.find('='); - string entryName = line.substr(0, offset); - string value = line.substr(offset + 1); - - eat_white(entryName); - eat_white(value); - - section->addEntry(entryName, value); + if (!section->add(line)) + return false; } return true; @@ -387,10 +406,10 @@ IniFile::printUnreferenced() { bool unref = false; - for (ConfigTable::iterator ci = table.begin(); - ci != table.end(); ++ci) { - const string §ionName = ci->first; - Section *section = ci->second; + for (SectionTable::iterator i = table.begin(); + i != table.end(); ++i) { + const string §ionName = i->first; + Section *section = i->second; if (!section->isReferenced()) { if (section->findEntry("unref_section_ok") == NULL) { @@ -416,15 +435,15 @@ IniFile::Section::dump(const string §ionName) for (EntryTable::iterator ei = table.begin(); ei != table.end(); ++ei) { cout << sectionName << ": " << (*ei).first << " => " - << (*ei).second << "\n"; + << (*ei).second->getValue() << "\n"; } } void IniFile::dump() { - for (ConfigTable::iterator ci = table.begin(); - ci != table.end(); ++ci) { - ci->second->dump(ci->first); + for (SectionTable::iterator i = table.begin(); + i != table.end(); ++i) { + i->second->dump(i->first); } } diff --git a/base/inifile.hh b/base/inifile.hh index 919732e1e..3a82f2d4d 100644 --- a/base/inifile.hh +++ b/base/inifile.hh @@ -36,75 +36,171 @@ #include "base/hashmap.hh" +/// +/// @file +/// Declaration of IniFile object. +/// + +/// +/// This class represents the contents of a ".ini" file. +/// +/// It's basically a two level lookup table: a set of named sections, +/// where each section is a set of key/value pairs. Section names, +/// keys, and values are all uninterpreted strings. +/// class IniFile { protected: + + /// + /// A single key/value pair. + /// class Entry { - std::string value; - mutable bool referenced; + std::string value; ///< The entry value. + mutable bool referenced; ///< Has this entry been used? public: + /// Constructor. Entry(const std::string &v) : value(v), referenced(false) { } + /// Has this entry been used? bool isReferenced() { return referenced; } + /// Fetch the value. const std::string &getValue() const; + /// Set the value. void setValue(const std::string &v) { value = v; } + + /// Append the given string to the value. A space is inserted + /// between the existing value and the new value. Since this + /// operation is typically used with values that are + /// space-separated lists of tokens, this keeps the tokens + /// separate. + void appendValue(const std::string &v) { value += " "; value += v; } }; + /// + /// A section. + /// class Section { + /// EntryTable type. Map of strings to Entry object pointers. typedef m5::hash_map<std::string, Entry *> EntryTable; - EntryTable table; - mutable bool referenced; + EntryTable table; ///< Table of entries. + mutable bool referenced; ///< Has this section been used? public: + /// Constructor. Section() : table(), referenced(false) { } + /// Has this section been used? bool isReferenced() { return referenced; } - void addEntry(const std::string &entryName, const std::string &value); + /// Add an entry to the table. If an entry with the same name + /// already exists, the 'append' parameter is checked If true, + /// the new value will be appended to the existing entry. If + /// false, the new value will replace the existing entry. + void addEntry(const std::string &entryName, const std::string &value, + bool append); + + /// Add an entry to the table given a string assigment. + /// Assignment should be of the form "param=value" or + /// "param+=value" (for append). This funciton parses the + /// assignment statment and calls addEntry(). + /// @retval True for success, false if parse error. + bool add(const std::string &assignment); + + /// Find the entry with the given name. + /// @retval Pointer to the entry object, or NULL if none. Entry *findEntry(const std::string &entryName) const; + /// Print the unreferenced entries in this section to cerr. + /// Messages can be suppressed using "unref_section_ok" and + /// "unref_entries_ok". + /// @param sectionName Name of this section, for use in output message. + /// @retval True if any entries were printed. bool printUnreferenced(const std::string §ionName); + + /// Print the contents of this section to cout (for debugging). void dump(const std::string §ionName); }; - typedef m5::hash_map<std::string, Section *> ConfigTable; + /// SectionTable type. Map of strings to Section object pointers. + typedef m5::hash_map<std::string, Section *> SectionTable; protected: - ConfigTable table; + /// Hash of section names to Section object pointers. + SectionTable table; + /// Look up section with the given name, creating a new section if + /// not found. + /// @retval Pointer to section object. Section *addSection(const std::string §ionName); + + /// Look up section with the given name. + /// @retval Pointer to section object, or NULL if not found. Section *findSection(const std::string §ionName) const; + /// Load parameter settings from given istream. This is a helper + /// function for load(string) and loadCPP(), which open a file + /// and then pass it here. + /// @retval True if successful, false if errors were encountered. bool load(std::istream &f); public: + /// Constructor. IniFile(); + + /// Destructor. ~IniFile(); + /// Load the specified file, passing it through the C preprocessor. + /// Parameter settings found in the file will be merged with any + /// already defined in this object. + /// @param file The path of the file to load. + /// @param cppFlags Vector of extra flags to pass to cpp. + /// @retval True if successful, false if errors were encountered. bool loadCPP(const std::string &file, std::vector<char *> &cppFlags); + + /// Load the specified file. + /// Parameter settings found in the file will be merged with any + /// already defined in this object. + /// @param file The path of the file to load. + /// @retval True if successful, false if errors were encountered. bool load(const std::string &file); + /// Take string of the form "<section>:<parameter>=<value>" or + /// "<section>:<parameter>+=<value>" and add to database. + /// @retval True if successful, false if parse error. bool add(const std::string &s); + /// Find value corresponding to given section and entry names. + /// Value is returned by reference in 'value' param. + /// @retval True if found, false if not. bool find(const std::string §ion, const std::string &entry, std::string &value) const; + + /// Find value corresponding to given section and entry names, + /// following "default" links to other sections where possible. + /// Value is returned by reference in 'value' param. + /// @retval True if found, false if not. bool findDefault(const std::string §ion, const std::string &entry, std::string &value) const; + /// Print unreferenced entries in object. Iteratively calls + /// printUnreferend() on all the constituent sections. bool printUnreferenced(); + /// Dump contents to cout. For debugging. void dump(); }; diff --git a/base/statistics.cc b/base/statistics.cc index 2f52314b9..a2734cf37 100644 --- a/base/statistics.cc +++ b/base/statistics.cc @@ -132,6 +132,10 @@ class Database typedef list<Stat *> list_t; typedef map<const Stat *, StatData *> map_t; + list<BinBase *> bins; + map<const BinBase *, std::string > bin_names; + list_t binnedStats; + list_t allStats; list_t printStats; map_t map; @@ -147,6 +151,7 @@ class Database void reset(); void regStat(Stat *stat); StatData *print(Stat *stat); + void regBin(BinBase *bin, std::string name); }; Database::Database() @@ -158,15 +163,51 @@ Database::~Database() void Database::dump(ostream &stream) { + list_t::iterator i = printStats.begin(); list_t::iterator end = printStats.end(); - while (i != end) { Stat *stat = *i; - if (stat->dodisplay()) - stat->display(stream); + if (stat->binned()) + binnedStats.push_back(stat); ++i; } + + list<BinBase *>::iterator j = bins.begin(); + list<BinBase *>::iterator bins_end=bins.end(); + + if (!bins.empty()) { + ccprintf(stream, "PRINTING BINNED STATS\n"); + while (j != bins_end) { + (*j)->activate(); + ::map<const BinBase *, std::string>::const_iterator iter; + iter = bin_names.find(*j); + if (iter == bin_names.end()) + panic("a binned stat not found in names map!"); + ccprintf(stream,"---%s Bin------------\n", (*iter).second); + + list_t::iterator i = binnedStats.begin(); + list_t::iterator end = binnedStats.end(); + while (i != end) { + Stat *stat = *i; + if (stat->dodisplay()) + stat->display(stream); + ++i; + } + ++j; + ccprintf(stream, "---------------------------------\n"); + } + } + + list_t::iterator k = printStats.begin(); + list_t::iterator endprint = printStats.end(); + ccprintf(stream, "*****ALL STATS*****\n"); + while (k != endprint) { + Stat *stat = *k; + if (stat->dodisplay() && !stat->binned()) + stat->display(stream); + ++k; + } } StatData * @@ -235,6 +276,21 @@ Database::regStat(Stat *stat) assert(success && "this should never fail"); } +void +Database::regBin(BinBase *bin, std::string name) +{ + if (bin_names.find(bin) != bin_names.end()) + panic("shouldn't register bin twice"); + + bins.push_back(bin); + + bool success = (bin_names.insert(make_pair(bin,name))).second; + assert(bin_names.find(bin) != bin_names.end()); + assert(success && "this should not fail"); + + cprintf("registering %s\n", name); +} + bool Stat::less(Stat *stat1, Stat *stat2) { @@ -288,6 +344,7 @@ Stat::Stat(bool reg) if (reg) StatDB().regStat(this); + #ifdef STAT_DEBUG number = ++total_stats; cprintf("I'm stat number %d\n",number); @@ -842,6 +899,12 @@ BinBase::memory() return mem; } +void +BinBase::regBin(BinBase *bin, std::string name) +{ + StatDB().regBin(bin, name); +} + } // namespace Detail void diff --git a/base/statistics.hh b/base/statistics.hh index a5fc5c6b5..00f103919 100644 --- a/base/statistics.hh +++ b/base/statistics.hh @@ -60,7 +60,7 @@ #include "sim/host.hh" // -// Un-comment this to enable wierdo-stat debugging +// Un-comment this to enable weirdo-stat debugging // // #define STAT_DEBUG @@ -232,6 +232,8 @@ class Stat */ virtual bool zero() const = 0; + //need to document + virtual bool binned() const = 0; /** * Set the name and marks this stat to print at the end of simulation. @@ -326,6 +328,9 @@ class ScalarStat : public Stat * @param stream The output stream. */ virtual void display(std::ostream &stream) const; + + //need to document + virtual bool binned() const = 0; }; void @@ -369,6 +374,9 @@ class VectorStat : public Stat * @param stream The output stream. */ virtual void display(std::ostream &stream) const; + + //need to document + virtual bool binned() const = 0; }; ////////////////////////////////////////////////////////////////////// @@ -627,6 +635,7 @@ class ScalarBase : public ScalarStat * @return 1. */ virtual size_t size() const { return 1; } + virtual bool binned() const { return bin_t::binned; } /** * Reset stat value to default @@ -756,6 +765,7 @@ class VectorBase : public VectorStat * @return The size of the vector. */ virtual size_t size() const { return bin.size(); } + virtual bool binned() const { return bin_t::binned; } /** * Reset stat value to default */ @@ -883,6 +893,7 @@ class ScalarProxy : public ScalarStat * @return 1. */ virtual size_t size() const { return 1; } + virtual bool binned() const { return false; } /** * This stat has no state. Nothing to reset */ @@ -963,6 +974,7 @@ class Vector2dBase : public Stat virtual size_t size() const { return bin.size(); } virtual bool zero() const { return data(0)->value(params) == 0.0; } + virtual bool binned() const { return bin_t::binned; } virtual void display(std::ostream &out) const @@ -1094,6 +1106,7 @@ class VectorProxy : public VectorStat assert (index >= 0 && index < size()); return ScalarProxy<T, Storage, Bin>(*bin, *params, offset + index); } + virtual bool binned() const { return false; } /** * This stat has no state. Nothing to reset. @@ -1509,6 +1522,8 @@ class DistBase : public Stat data()->display(stream, myname(), mydesc(), myprecision(), myflags(), params); } + + virtual bool binned() const { return bin_t::binned; } /** * Reset stat value to default */ @@ -1555,6 +1570,7 @@ class VectorDistBase : public Stat virtual size_t size() const { return bin.size(); } virtual bool zero() const { return false; } virtual void display(std::ostream &stream) const; + virtual bool binned() const { return bin_t::binned; } /** * Reset stat value to default */ @@ -1618,6 +1634,8 @@ class DistProxy : public Stat data()->display(stream, name.str(), desc.str(), cstat->myprecision(), cstat->myflags(), cstat->params); } + + virtual bool binned() const { return false; } /** * Proxy has no state. Nothing to reset. */ @@ -1692,6 +1710,8 @@ class Node : public RefCounted * @return The total of the result vector. */ virtual result_t total() const = 0; + + virtual bool binned() const = 0; }; /** Reference counting pointer to a function Node. */ @@ -1709,6 +1729,8 @@ class ScalarStatNode : public Node virtual result_t total() const { return stat.val(); }; virtual size_t size() const { return 1; } + + virtual bool binned() const { return stat.binned(); } }; template <typename T, template <typename T> class Storage, class Bin> @@ -1725,6 +1747,8 @@ class ScalarProxyNode : public Node virtual result_t total() const { return proxy.val(); }; virtual size_t size() const { return 1; } + + virtual bool binned() const { return proxy.binned(); } }; class VectorStatNode : public Node @@ -1738,6 +1762,8 @@ class VectorStatNode : public Node virtual result_t total() const { return stat.total(); }; virtual size_t size() const { return stat.size(); } + + virtual bool binned() const { return stat.binned(); } }; template <typename T> @@ -1752,6 +1778,7 @@ class ConstNode : public Node virtual result_t total() const { return data[0]; }; virtual size_t size() const { return 1; } + virtual bool binned() const { return false; } }; template <typename T> @@ -1770,6 +1797,7 @@ class FunctorNode : public Node virtual result_t total() const { return (result_t)functor(); }; virtual size_t size() const { return 1; } + virtual bool binned() const { return false; } }; template <typename T> @@ -1788,6 +1816,7 @@ class ScalarNode : public Node virtual result_t total() const { return (result_t)scalar; }; virtual size_t size() const { return 1; } + virtual bool binned() const { return false; } }; template <class Op> @@ -1820,6 +1849,7 @@ class UnaryNode : public Node } virtual size_t size() const { return l->size(); } + virtual bool binned() const { return l->binned(); } }; template <class Op> @@ -1880,6 +1910,8 @@ class BinaryNode : public Node return ls; } } + + virtual bool binned() const { return (l->binned() || r->binned()); } }; template <class Op> @@ -1921,6 +1953,7 @@ class SumNode : public Node } virtual size_t size() const { return 1; } + virtual bool binned() const { return l->binned(); } }; /** @@ -2047,7 +2080,9 @@ class BinBase public: BinBase(size_t size); - ~BinBase(); + virtual ~BinBase(); + virtual void activate() = 0; + void regBin(BinBase *bin, std::string name); }; } // namespace Detail @@ -2055,6 +2090,12 @@ class BinBase template <class BinType> struct StatBin : public Detail::BinBase { + private: + std::string _name; + + public: + std::string name() const { return _name;} + static StatBin *&curBin() { static StatBin *current = NULL; return current; @@ -2078,13 +2119,14 @@ struct StatBin : public Detail::BinBase return off; } - explicit StatBin(size_t size = 1024) : Detail::BinBase(size) {} + explicit StatBin(std::string name, size_t size = 1024) : Detail::BinBase(size) { _name = name; this->regBin(this, name); } char *memory(off_t off) { assert(offset() <= size()); return Detail::BinBase::memory() + off; } + virtual void activate() { setCurBin(this); } static void activate(StatBin &bin) { setCurBin(&bin); } class BinBase @@ -2110,6 +2152,7 @@ struct StatBin : public Detail::BinBase typedef typename Storage::Params Params; public: + enum { binned = true }; Bin() { allocate(sizeof(Storage)); } bool initialized() const { return true; } void init(const Params ¶ms) { } @@ -2198,6 +2241,7 @@ struct NoBin { public: typedef typename Storage::Params Params; + enum { binned = false }; private: char ptr[sizeof(Storage)]; @@ -2229,6 +2273,7 @@ struct NoBin { public: typedef typename Storage::Params Params; + enum { binned = false }; private: char *ptr; @@ -2606,6 +2651,8 @@ class Formula : public Detail::VectorStat return root->size(); } + virtual bool binned() const { return root->binned(); } + /** * Formulas don't need to be reset */ diff --git a/cpu/base_cpu.cc b/cpu/base_cpu.cc index 2e1d95d88..90e090d5e 100644 --- a/cpu/base_cpu.cc +++ b/cpu/base_cpu.cc @@ -34,6 +34,7 @@ #include "base/cprintf.hh" #include "cpu/exec_context.hh" #include "base/misc.hh" +#include "sim/param.hh" #include "sim/sim_events.hh" using namespace std; diff --git a/cpu/exetrace.cc b/cpu/exetrace.cc index 01f50e675..c350288bc 100644 --- a/cpu/exetrace.cc +++ b/cpu/exetrace.cc @@ -29,6 +29,7 @@ #include <fstream> #include <iomanip> +#include "sim/param.hh" #include "cpu/full_cpu/dyn_inst.hh" #include "cpu/full_cpu/spec_state.hh" #include "cpu/full_cpu/issue.hh" diff --git a/dev/etherint.cc b/dev/etherint.cc index 2845ce729..cfffb3a87 100644 --- a/dev/etherint.cc +++ b/dev/etherint.cc @@ -28,6 +28,7 @@ #include "dev/etherint.hh" #include "base/misc.hh" +#include "sim/param.hh" #include "sim/sim_object.hh" void diff --git a/sim/sim_object.hh b/sim/sim_object.hh index 6c15fc88f..20da07164 100644 --- a/sim/sim_object.hh +++ b/sim/sim_object.hh @@ -38,7 +38,6 @@ #include <vector> #include <iostream> -#include "sim/param.hh" #include "sim/serialize.hh" /* diff --git a/sim/system.cc b/sim/system.cc index e1e293c90..0e0b83332 100644 --- a/sim/system.cc +++ b/sim/system.cc @@ -28,6 +28,7 @@ #include "cpu/exec_context.hh" #include "targetarch/vtophys.hh" +#include "sim/param.hh" #include "sim/system.hh" using namespace std; diff --git a/test/Makefile b/test/Makefile index c95f8cb8d..1502bca3d 100644 --- a/test/Makefile +++ b/test/Makefile @@ -37,7 +37,7 @@ circletest: circletest.o circlebuf.o cprintftest: cprintftest.o cprintf.o $(CXX) $(LFLAGS) -o $@ $^ -initest: initest.o str.o inifile.o +initest: initest.o str.o inifile.o cprintf.o $(CXX) $(LFLAGS) -o $@ $^ lrutest: lru_test.o diff --git a/test/foo.ini b/test/foo.ini index aa4305f12..534a4e001 100644 --- a/test/foo.ini +++ b/test/foo.ini @@ -5,3 +5,6 @@ Foo2=384 [General] Test3=89 + +[Junk] +Test4+=mia diff --git a/test/initest.cc b/test/initest.cc index 51089a46b..818c64da7 100644 --- a/test/initest.cc +++ b/test/initest.cc @@ -26,92 +26,117 @@ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. */ -#include <iostream.h> -#include <fstream.h> -#include <unistd.h> - +#include <iostream> +#include <fstream> #include <string> #include <vector> #include "base/inifile.hh" +#include "base/cprintf.hh" + +using namespace std; char *progname; void usage() { - cout << "Usage: " << progname << " <ini file>\n"; - exit(1); + cout << "Usage: " << progname << " <ini file>\n"; + exit(1); } #if 0 - char *defines = getenv("CONFIG_DEFINES"); - if (defines) { - char *c = defines; - while ((c = strchr(c, ' ')) != NULL) { +char *defines = getenv("CONFIG_DEFINES"); +if (defines) { + char *c = defines; + while ((c = strchr(c, ' ')) != NULL) { *c++ = '\0'; count++; - } - count++; } + count++; +} #endif int main(int argc, char *argv[]) { - IniFile config; - - progname = argv[0]; - - int cpp_options_count = 0; - char **cpp_options = new char*[argc * 2]; - char **cur_opt = cpp_options; - - int ch; - while ((ch = getopt(argc, argv, "D:")) != -1) { - switch (ch) { - case 'D': - *cur_opt++ = "-D"; - *cur_opt++ = optarg; - cpp_options_count += 2; - break; - - default: - usage(); + IniFile simConfigDB; + + progname = argv[0]; + + vector<char *> cppArgs; + + vector<char *> cpp_options; + cpp_options.reserve(argc * 2); + + for (int i = 1; i < argc; ++i) { + char *arg_str = argv[i]; + + // if arg starts with '-', parse as option, + // else treat it as a configuration file name and load it + if (arg_str[0] == '-') { + + // switch on second char + switch (arg_str[1]) { + case 'D': + case 'U': + case 'I': + // cpp options: record & pass to cpp. Note that these + // cannot have spaces, i.e., '-Dname=val' is OK, but + // '-D name=val' is not. I don't consider this a + // problem, since even though gnu cpp accepts the + // latter, other cpp implementations do not (Tru64, + // for one). + cppArgs.push_back(arg_str); + break; + + case '-': + // command-line configuration parameter: + // '--<section>:<parameter>=<value>' + + if (!simConfigDB.add(arg_str + 2)) { + // parse error + ccprintf(cerr, + "Could not parse configuration argument '%s'\n" + "Expecting --<section>:<parameter>=<value>\n", + arg_str); + exit(0); + } + break; + + default: + usage(); + } + } + else { + // no '-', treat as config file name + + if (!simConfigDB.loadCPP(arg_str, cppArgs)) { + cprintf("Error processing file %s\n", arg_str); + exit(1); + } + } } - } - - argc -= optind; - argv += optind; - if (argc != 1) - usage(); - - string file = argv[0]; - - if (!config.loadCPP(file, cpp_options, cpp_options_count)) { - cout << "File not found!\n"; - exit(1); - } + string value; - string value; #define FIND(C, E) \ - if (config.find(C, E, value)) \ + if (simConfigDB.find(C, E, value)) \ cout << ">" << value << "<\n"; \ else \ cout << "Not Found!\n" - FIND("General", "Test2"); - FIND("Junk", "Test3"); - FIND("Junk", "Test4"); - FIND("General", "Test1"); - FIND("Junk2", "test3"); - FIND("General", "Test3"); + FIND("General", "Test2"); + FIND("Junk", "Test3"); + FIND("Junk", "Test4"); + FIND("General", "Test1"); + FIND("Junk2", "test3"); + FIND("General", "Test3"); - cout << "\n"; + cout << "\n"; - config.dump(); + simConfigDB.dump(); - return 0; + return 0; } |