Hi, Here is the second revision of this patchset.
Changelog: * use a new class TokenValue to hold what was in the token value hash in v1. This improves a little bit the performance and is easier to read. * Implemented the other few review comments Please review, Thanks, Brice Brice Figureau (4): Fix #2366 - puppetdoc was parsing classes in the wrong order Make sure the parser sees the correct line number Fix #2364 - Associates the correct comment to the right statement Allow boolean value for boolean cli parameter lib/puppet/parser/grammar.ra | 117 ++++++----- lib/puppet/parser/lexer.rb | 71 +++++-- lib/puppet/parser/parser.rb | 389 ++++++++++++++++++++--------------- lib/puppet/parser/parser_support.rb | 9 +- lib/puppet/util/rdoc/parser.rb | 3 +- lib/puppet/util/settings.rb | 4 +- spec/unit/parser/lexer.rb | 48 ++++- spec/unit/util/settings.rb | 7 + 8 files changed, 390 insertions(+), 258 deletions(-) --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Puppet Developers" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/puppet-dev?hl=en -~----------~----~----~----~------~----~------~--~---
