Skip to content

ncgen -lc is incorrectly handling _Format#1506

Merged
WardF merged 6 commits intoUnidata:masterfrom
DennisHeimbigner:cformat.dmh
Nov 6, 2019
Merged

ncgen -lc is incorrectly handling _Format#1506
WardF merged 6 commits intoUnidata:masterfrom
DennisHeimbigner:cformat.dmh

Conversation

@DennisHeimbigner
Copy link
Copy Markdown
Collaborator

re: #1504 (partial)

The ncgen c-code generator is incorrectly generating an attribute
for _Format.
Soln: remove code.

re: Unidata#1504 (partial)

The ncgen c-code generator is incorrectly generating an attribute
for _Format.
Soln: remove code.
Comment thread nc_test4/Makefile.am

# Un comment to use a more verbose test driver
#SH_LOG_DRIVER = $(SHELL) $(top_srcdir)/test-driver-verbose
#sh_LOG_DRIVER = $(SHELL) $(top_srcdir)/test-driver-verbose
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Added duplicate line?

Comment thread nc_test4/tst_filter.sh Outdated
if test "x$srcdir" = x ; then srcdir=`pwd`; fi
. ../test_common.sh

set -x
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Probably don't want to leave set -x on, as it will cause a lot of output on make check.

@edhartnett
Copy link
Copy Markdown
Contributor

Good tests!

@WardF WardF self-assigned this Nov 6, 2019
@WardF WardF added this to the 4.7.3 milestone Nov 6, 2019
@WardF WardF merged commit b5673f5 into Unidata:master Nov 6, 2019
@DennisHeimbigner DennisHeimbigner deleted the cformat.dmh branch March 29, 2020 21:17
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.

3 participants