bison-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: FYI: lalr1.cc: Remove the traits


From: Hans Aberg
Subject: Re: FYI: lalr1.cc: Remove the traits
Date: Mon, 19 Sep 2005 11:03:25 +0200


On 19 Sep 2005, at 09:35, Akim Demaille wrote:

Index: ChangeLog
from  Akim Demaille  <address@hidden>

    Remove the traits, failed experiment.
    It never proved useful, and anyway because of the current
    definition, it was not possible to have several specialization of
    this traits, making it useless.
    * data/lalr1.cc (yy:traits): Remove.
    Inline its definitions in the parser class.

Also, just use the std::deque for stack. I think that std::stack will not be able to look deeper than top, as is required by the parser. And std::deque is in the latest C++ revision written so that it does not invoke copy constructors when expanding the stack, as it just allocates a new chunk, leaving the old one intact.

  Hans Aberg






reply via email to

[Prev in Thread] Current Thread [Next in Thread]