Ticket #412 (new defect)

Opened 8 years ago

nonstandard trailing comma in src/data_sources/grf2ew/ustime.h

Reported by: baker Owned by: somebody
Priority: minor Milestone: All Platforms
Component: ALL modules Version: 7.8
Keywords: grf2ew Cc:

Description

The definition of typedef enum UST_DFL_DATE_FMT in ustime.h has an extra comma. It causes warnings like:

icc -c -I../../../include -g -D_BSD_SOURCE -m32 -Dlinux -D__i386 -D_LINUX -D_INTEL -D_USE_SCHED  -D_USE_PTHREADS -D_USE_TERMIOS -I/opt/earthworm/earthworm-7.6-5880/include -no-gcc -g -O0 -Wcheck -Wformat -check=stack,uninit -traceback -D_DEBUG  main.c

ustime.h(49): warning #271: trailing comma is nonstandard
        UST_CALENDAR = 1,                                       /* yyyy-mm-dd, two digit month and day-of-month */
                        ^

The fix is:

--- src/data_sources/grf2ew/ustime.h
+++ src/data_sources/grf2ew/ustime.h    
@@ -48,3 +48,3 @@
        UST_ORDINAL = 0,                                        /* yyyy-ddd, three digit ordinal day-of-year */
-       UST_CALENDAR = 1,                                       /* yyyy-mm-dd, two digit month and day-of-month */
+       UST_CALENDAR = 1                                        /* yyyy-mm-dd, two digit month and day-of-month */
 } UST_DFL_DATE_FMT;
Note: See TracTickets for help on using tickets.