Menu

netcdf Merge Request #1: Use size_t for array index (merged)

Merging...

Merged

Something went wrong. Please, merge manually

Checking if merge is possible...

Something went wrong. Please, merge manually

Orion Poplawski wants to merge 1 commit from /u/opoplawski/octave/ to master, 2023-09-20

Getting the following warning:

__netcdf__.cc: In function octave_value_list Fnetcdf_getAtt(const octave_value_list&, int):
__netcdf__.cc:1617:29: warning: comparison of integer expressions of different signedness: int and size_t {aka long unsigned int} [-Wsign-compare]
 1617 |            for (int vi=0; vi<len;vi++) {                                        \
      |                           ~~^~~~
__netcdf__.cc:1626:7: note: in expansion of macro OV_NETCDF_GETVLEN_ATT_CASE
Commit Date  
[aebb41] (size_t) by Orion Poplawski Orion Poplawski

Use size_t for array index

2023-09-20 03:49:48 Tree

Discussion

  • lostbard

    lostbard - 2023-09-20
    • Status: open --> merged
     
  • lostbard

    lostbard - 2023-09-20

    thanks for the patch - i merged it.
    I see a few other warnings unrelated to this, that I will take a look at

     

Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.