On Fri, Apr 05, 2024 at 11:02:14PM +0200, Patrice Dumas wrote:
> > I've committed a change to remove the $no_build parameter but not the
> > $no_store parameter.
> 
> It seems to me that the changes are no complete, to me the _get_parser_info
> $no_build argument should be removed too, and the build_document should
> only be called if $no_store is set.  It seems to me that is would be
> better for parse_texi_line, it seems to me that if $no_store is not set,
> the tree should not be built in _get_parser_info, but in
> parse_texi_line, such that the decision to get a tree or only a handler
> is possible in future changes of parse_texi_line.

I've made this change.  I agree that it is better this way.

Reply via email to