pkg update - geometry warnings / errors on 5.2.0

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

pkg update - geometry warnings / errors on 5.2.0

nrjank
updating a fresh 5.2.0 install on windows produced the following warnings. obviously not catastrophic, but worth noting:

octave:2> pkg update
error: the following dependencies were unsatisfied:
   geometry needs matgeom >= 1.0.0

octave:2> pkg install -forge matgeom
For information about changes from previous versions of the matgeom package, run 'news matgeom'.

octave:3> pkg update
clipper.cpp: In function 'void ClipperLib::InitEdge(ClipperLib::TEdge*, ClipperLib::TEdge*, ClipperLib::TEdge*, const ClipperLib::IntPoint&)':
clipper.cpp:750:34: warning: 'void* memset(void*, int, size_t)' clearing an object of non-trivial type 'struct ClipperLib::TEdge'; use assignment or value-initialization instead [-Wclass-memaccess]
  750 |   std::memset(e, 0, sizeof(TEdge));
      |                                  ^
clipper.cpp:95:8: note: 'struct ClipperLib::TEdge' declared here
   95 | struct TEdge {
      |        ^~~~~
warning: directory __OH__/share/octave/packages/geometry-3.0.0 previously lost
For information about changes from previous versions of the geometry package, run 'news geometry'.


Reply | Threaded
Open this post in threaded view
|

Re: pkg update - geometry warnings / errors on 5.2.0

Juan Pablo Carbajal-2
> octave:3> pkg update
> clipper.cpp: In function 'void ClipperLib::InitEdge(ClipperLib::TEdge*, ClipperLib::TEdge*, ClipperLib::TEdge*, const ClipperLib::IntPoint&)':
> clipper.cpp:750:34: warning: 'void* memset(void*, int, size_t)' clearing an object of non-trivial type 'struct ClipperLib::TEdge'; use assignment or value-initialization instead [-Wclass-memaccess]
>   750 |   std::memset(e, 0, sizeof(TEdge));

This is the only relevant warning, Yes the dev used memset, we have to
fix this in future releases. Olaf laready pointed it out at release
time. Thanks!


Reply | Threaded
Open this post in threaded view
|

Re: pkg update - geometry warnings / errors on 5.2.0

nrjank


This is the only relevant warning, Yes the dev used memset, we have to
fix this in future releases. Olaf laready pointed it out at release
time. Thanks!


Got it. 

warning: directory __OH__/share/octave/packages/geometry-3.0.0 previously lost

I noticed a bunch of these as well for other packages. What exactly is that warning about?