Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor and Typographical Errors #28

Closed
mbeckerle opened this issue Feb 14, 2022 · 7 comments
Closed

Minor and Typographical Errors #28

mbeckerle opened this issue Feb 14, 2022 · 7 comments

Comments

@mbeckerle
Copy link
Collaborator

mbeckerle commented Feb 14, 2022

Build up list of minor/typo fixes needed in this single bug.

  1. In description of lengthKind 'prefixed' unparsing: "bits is always be a multiple of 8 bits"
@mbeckerle
Copy link
Collaborator Author

Description of representation property in Section 13.1 you will find "text, (no closing quotation mark)

@smhdfdl
Copy link
Contributor

smhdfdl commented Jul 14, 2022

Clarify the meaning of 'base simple type' for the property dfdl:binaryDecimalVirtualPoint to make clear it does not include any of the set of built-in integer simple types.

@mbeckerle
Copy link
Collaborator Author

String "whenthe" in 6.3.1.1

@mbeckerle
Copy link
Collaborator Author

mbeckerle commented Jan 11, 2023

This in 13.6.1.1 Table 32

Used to quote special characters in a prefix or suffix, for example, "'#'#" formats 123 to "#123". To create a single quote itself, use two in a row: "# o''clock".

The examples need to use fonts better. Probably monospaced, because in the basic font the double quotes and single quotes are too close together and so you can't tell what is going on. Something more like '#'# leaving off the surrounding double quotes would work better.

@mbeckerle
Copy link
Collaborator Author

Table 2 in Section 6.2 has a cross reference about dfdl:assert that is incorrect.

For dfdl:assert it should say See Section 7.5 (not 7.2.1)

@mbeckerle
Copy link
Collaborator Author

NOTE: Mike Beckerle has edited all prior to this comment into a new version of the spec.

@mbeckerle
Copy link
Collaborator Author

Fixed by #45

A new ticket should be opened for future such typo/minor errors. Closing this ticket as these specific ones are all fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants