git.net

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

[jira] [Commented] (DAFFODIL-830) Performance: Remove unnecessary SeqCombParsers from generated parser


    [ https://issues.apache.org/jira/browse/DAFFODIL-830?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16458654#comment-16458654 ] 

Michael Beckerle commented on DAFFODIL-830:
-------------------------------------------

Assigning to Josh as he has sequence-combinator changes that definitely revisited this whole issue, so such time as that gets committed that will reflect closing out this matter as well.

> Performance: Remove unnecessary SeqCombParsers from generated parser
> --------------------------------------------------------------------
>
>                 Key: DAFFODIL-830
>                 URL: https://issues.apache.org/jira/browse/DAFFODIL-830
>             Project: Daffodil
>          Issue Type: Improvement
>          Components: Middle "End", Performance
>    Affects Versions: s10
>            Reporter: Michael Beckerle
>            Priority: Minor
>             Fix For: deferred
>
>
> This is another missing optimization.
> We often have parsers that have nested <seq>....</seq>
> This is never needed as these can be merged. This improves tracing and debugging by avoiding having these excess things there.
> It also simply reduces the number of parser 'actions' a bit, so can't hurt for performance reasons. Every parsed element seems to have at least one excess <seq>...</seq> created around it.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)