diff options
Diffstat (limited to 'cpp/src')
| -rw-r--r-- | cpp/src/Slice/GrammarUtil.h | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/cpp/src/Slice/GrammarUtil.h b/cpp/src/Slice/GrammarUtil.h index 642815afc7b..6e70b9f2eeb 100644 --- a/cpp/src/Slice/GrammarUtil.h +++ b/cpp/src/Slice/GrammarUtil.h @@ -16,11 +16,14 @@ namespace Slice { + +// TODO: ML: Consider moving to Scanner.l, as it is only used there. // // Function object to do case-insensitive string comparison. // struct CICompare : public std::binary_function<std::string, std::string, bool> { + // TODO: ML: Make non-inline. bool operator()(const std::string& s1, const std::string& s2) const { std::string::const_iterator p1 = s1.begin(); @@ -52,12 +55,15 @@ struct CICompare : public std::binary_function<std::string, std::string, bool> // // Definitions for the case-insensitive keyword-token map. // +// TODO: ML: Naming conventions for types. (Should be StringTokenMap.) +// TODO: ML: Consider moving to Scanner.l, as it is only used there. Then keywordMap can also be static. typedef std::map<std::string, int, CICompare> stringTokenMap; extern stringTokenMap keywordMap; // // initialize() fills the keyword map with all keyword-token pairs. // +// TODO: ML: Perhaps initializeKeywordMap() would be a better name? void initialize(); class StringTok; |