[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5923 Use a hash table for the lexer keywords

[Auto mailings of changes to Lily Issues via Testlilyissues-auto]

• Needs: -->
• Patch: new --> needs_work
• Type: -->
• Comment:

Fails make

...
Making lily/out/lily-lexer.o < cc
Making lily/out/metronome-engraver.o < cc
Making lily/out/warn-scheme.o < cc
/home/james/lilypond-git/lily/lily-lexer.cc:47:13: error: 'unordered_map' in namespace 'std' does not name a template type
 static std::unordered_map<std::string, int> keytable{
             ^~~~~~~~~~~~~
/home/james/lilypond-git/lily/lily-lexer.cc: In member function 'int Lily_lexer::lookup_keyword(const string&)':
/home/james/lilypond-git/lily/lily-lexer.cc:183:20: error: 'keytable' was not declared in this scope
   auto const &it = keytable.find (s);
                    ^~~~~~~~
/home/james/lilypond-git/lily/lily-lexer.cc:183:20: note: suggested alternative: 'mutable'
   auto const &it = keytable.find (s);
                    ^~~~~~~~
                    mutable
Making lily/out/lexer.o < cc
/home/james/lilypond-git/build/../stepmake/stepmake/c++-rules.make:4: recipe for target 'out/lily-lexer.o' failed
make[1]: *** [out/lily-lexer.o] Error 1
make[1]: *** Waiting for unfinished jobs....
out/lexer.cc: In member function 'virtual int yyFlexLexer::LexerInput(char*, int)':
out/lexer.cc:6476:21: warning: conversion to 'int' from 'std::streamsize {aka long int}' may alter its value [-Wconversion]
   return yyin.gcount();
          ~~~~~~~~~~~^~
/home/james/lilypond-git/build/../stepmake/stepmake/generic-targets.make:6: recipe for target 'all' failed
make: *** [all] Error 2

---

[issues:#5923] Use a hash table for the lexer keywords

Status: Started
Created: Mon Apr 20, 2020 08:10 AM UTC by Han-Wen Nienhuys
Last Updated: Mon Apr 27, 2020 09:00 AM UTC
Owner: Han-Wen Nienhuys

Get rid of unused function ly:lexer-keywords.

https://codereview.appspot.com/549920043

---

Sent from sourceforge.net because address@hidden is subscribed to https://sourceforge.net/p/testlilyissues/issues/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.

_______________________________________________
Testlilyissues-auto mailing list
address@hidden
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto