You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@daffodil.apache.org by "Mike Beckerle (Jira)" <ji...@apache.org> on 2022/03/24 17:26:00 UTC
[jira] [Commented] (DAFFODIL-2295) priorAlignmentApprox incorrect if term is preceded by separator with text bit alignment
[ https://issues.apache.org/jira/browse/DAFFODIL-2295?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17511964#comment-17511964 ]
Mike Beckerle commented on DAFFODIL-2295:
-----------------------------------------
Related to DAFFODIL-2626
> priorAlignmentApprox incorrect if term is preceded by separator with text bit alignment
> ---------------------------------------------------------------------------------------
>
> Key: DAFFODIL-2295
> URL: https://issues.apache.org/jira/browse/DAFFODIL-2295
> Project: Daffodil
> Issue Type: Bug
> Components: Middle "End"
> Affects Versions: 2.5.0
> Reporter: Mike Beckerle
> Priority: Minor
>
> The AlignedMixin trait has a member priorAlignmentApprox which for a term, computes an approximation of the alignment prior to it.
> One of the things that could be before a term is a separator of a separated sequence (prefix or infix position), and in that case, the alignment will be the mandatory-text-alignment of that separator's encoding.
> The code of priorAlignmentApprox takes a number of things into consideration, but does not take the possibility of a separator into account.
> The design note [https://daffodil.apache.org/dev/design-notes/term-sharing-in-schema-compiler/] shows numerous things in the "unique before" of a term including the separator text regions, but the code doesn't implement this fully.
> This is a bug, because the schema compiler may not insert an alignment region when one is required to meet the term's alignment property requirement.
>
--
This message was sent by Atlassian Jira
(v8.20.1#820001)