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

Clean up minor issues in DAP2+DAP4 code #3092

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

DennisHeimbigner
Copy link
Collaborator

re: PR #3068

Part of splitting PR 3068

  1. Libdap2+Libdap4: Update to change "_FillValue" to NC_FillValue constant.
  2. Libdap4: Update the algorithm for sizing the packet buffer.
  3. Libdap4: Dynamically sort the ATOMICTYPEINFO table so it can be used with binary search in lookupAtomicType function
  4. Mark some unused function arguments.
  5. Subsume the Hyrax changes in PR Step 2 in splitting PR 3068 #3089.

re: PR Unidata#3068

Part of splitting PR 3068

1. Libdap2+Libdap4: Update to change "_FillValue" to NC_FillValue constant.
2. Libdap4: Update the algorithm for sizing the packet buffer.
3. Libdap4: Dynamically sort the ATOMICTYPEINFO table so it can be used
   with binary search in lookupAtomicType function
4. Mark some unused function arguments.
5. Subsume the Hyrax changes in PR Unidata#3089.
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.

1 participant