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

Ncdump fails under visual studio for certain dap urls #366

Merged
merged 8 commits into from
Feb 23, 2017
Merged

Commits on Feb 20, 2017

  1. re: #365

    Following command fails under visual studio.
        ncdump -h http://thredds.ucar.edu/thredds/dodsC/nexrad/composite/gini/n0r/1km/20170216/Level3_Composite_n0r_1km_20170216_1635.gini
    
    The problem is that sscanf for windows does not appear to support
    scanning 8bit integers: it appears to only allow scanning of characters.
    
    Solution:
    Scan the input as an integer (for type Byte) or unsigned int (for type UByte)
    and then recast the result as a char or unsigned char.
    
    Primary code fix is in libdap2/dapcvt.c#dapcvtattrval
    DennisHeimbigner committed Feb 20, 2017
    Configuration menu
    Copy the full SHA
    96bd037 View commit details
    Browse the repository at this point in the history
  2. Followon to

    re: #365
    
    1. Added to RELEASENOTES.md
    2. Add a range check to more closely
       mimic unix sscanf
    3. locate and fix same sscanf problems in ncgen/cvt.c
    
    Still need a stable url for a test case.
    DennisHeimbigner committed Feb 20, 2017
    Configuration menu
    Copy the full SHA
    f71b695 View commit details
    Browse the repository at this point in the history

Commits on Feb 22, 2017

  1. Configuration menu
    Copy the full SHA
    f96e0e8 View commit details
    Browse the repository at this point in the history
  2. Updated generated ncgen files.

    WardF committed Feb 22, 2017
    Configuration menu
    Copy the full SHA
    4208a87 View commit details
    Browse the repository at this point in the history
  3. 1. Added a restriction note on ncgen/Makefile.am.makeparser task.

    2. invoked makeparser using bison 3.0.4 and flex 2.5.35
    DennisHeimbigner committed Feb 22, 2017
    Configuration menu
    Copy the full SHA
    d0fb4a4 View commit details
    Browse the repository at this point in the history

Commits on Feb 23, 2017

  1. Updated release notes

    WardF committed Feb 23, 2017
    Configuration menu
    Copy the full SHA
    cda06db View commit details
    Browse the repository at this point in the history
  2. Invoked makeparser from OSX.

    WardF committed Feb 23, 2017
    Configuration menu
    Copy the full SHA
    1cdde8b View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    7cb0350 View commit details
    Browse the repository at this point in the history