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.

Sign up for GitHub

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

Jump to bottom

Fix the handling of certain alias types in CDL files. #1984

Merged
merged 2 commits into from
Apr 14, 2021
Merged

Fix the handling of certain alias types in CDL files. #1984

merged 2 commits into from
Apr 14, 2021

Commits on Apr 13, 2021

  1. Fix the handling of certain alias types on CDL files. 

    re: Unidata#1977

PR Unidata#1753, changed ncgen
to allows certain type names to be used as identifiers in
selected situations.

An unwanted side effect was that existing type aliases no longer
were accepted by ncgen. Specifically, using the "long" type
caused an error.

I was able to figure out a better solution to the original
problem (Unidata#1750)
that also fixes this problem as well.

This PR fixes that problem in ncgen/ncgen.l,
and adds tests to ncdump/test_keywords.sh
    @DennisHeimbigner
    DennisHeimbigner committed Apr 13, 2021
    Configuration menu
    Copy the full SHA
    ac42162 View commit details
    Browse the repository at this point in the history

Commits on Apr 14, 2021

  1. Update release notes

    @DennisHeimbigner
    DennisHeimbigner committed Apr 14, 2021
    Configuration menu
    Copy the full SHA
    30ee991 View commit details
    Browse the repository at this point in the history