Skip to content

Add dfdlx:currentPosition function#1443

Merged
mbeckerle merged 1 commit intoapache:mainfrom
mbeckerle:daf-2974
Oct 17, 2025
Merged

Add dfdlx:currentPosition function#1443
mbeckerle merged 1 commit intoapache:mainfrom
mbeckerle:daf-2974

Conversation

@mbeckerle
Copy link
Contributor

This is for some DFDL Workgroup experiments with data formats that DFDL can't currently represent.

This is not to be merged (we'll use a local build) for experiments.

DAFFODIL-2974

Copy link
Member

@stevedlawrence stevedlawrence left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

Copy link
Contributor

@olabusayoT olabusayoT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@jadams-tresys
Copy link
Contributor

Just to clarify, are we still not intending to merge these into the next release @mbeckerle ?

@mbeckerle
Copy link
Contributor Author

Just to clarify, are we still not intending to merge these into the next release @mbeckerle ?

I think this could be merged but I would do it for Daffodil 4.0.0 and not until I or someone has time to actually try out using it.

@mbeckerle
Copy link
Contributor Author

I've changed views on this. I do believe we should merge this, even if it's just for experiments it's too hard to do that with local builds etc.

This is for some DFDL experiments with data formats
that DFDL can't currently represent.

The function is only meaningful at parse time.

DAFFODIL-2974
Copy link
Member

@stevedlawrence stevedlawrence left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 agreed this is easier to test and experiment with once merged

@mbeckerle mbeckerle merged commit 25391d8 into apache:main Oct 17, 2025
11 checks passed
@mbeckerle mbeckerle deleted the daf-2974 branch October 17, 2025 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants