[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[netCDF #RAQ-461561]: NC_FILL_STRING definition suggestion

This archive contains answers to questions sent to Unidata support through mid-2025. Note that the archive is no longer being updated. We provide the archive for reference; many of the answers presented here remain technically correct, even if somewhat outdated. For the most up-to-date information on the use of NSF Unidata software and data services, please consult the Software Documentation first.

  • Subject: [netCDF #RAQ-461561]: NC_FILL_STRING definition suggestion
  • Date: Tue, 27 May 2014 11:04:08 -0600
Charlie,

> I suggest that instead of the netcdf.h definition
> 
> #define NC_FILL_STRING  ""
> 
> instead the following be used
> 
> #define NC_FILL_STRING  (char *)""
> 
> This would be more consistent with the other fill value definitions,
> and would prevent clang compiler warning like this
> 
> ../src/nco/nco_mss_val.c:39:32: warning: assigning to 'nco_string' (aka
> 'char *') from 'const char [1]' discards
> qualifiers [-Wincompatible-pointer-types-discards-qualifiers]
> 
> It does not cause problems on the other compilers I have checked.
> However, my checks were not thorough and may I overlooked a
> downside consequence.

Great suggestion, thanks!  It's committed on GitHub.

--Russ

Russ Rew                                         UCAR Unidata Program
address@hidden                      http://www.unidata.ucar.edu



Ticket Details
===================
Ticket ID: RAQ-461561
Department: Support netCDF
Priority: Normal
Status: Closed