Archived issue #0024076

New warning during compilation OCCT on WNT platform (release and debug)

Open CASCADEOCCT:Codingclosed3 public notes

Description

Warning in file src/Adaptor3d/Adaptor3d_IsoCurve.cxx

Current master includes two fixes 24058 and 23947. In both issues operator return gp_Elips() was deleted.

24058:
gp_Elips Adaptor3d_IsoCurve::Ellipse() const
{
  ...
- // portage WNT
- return gp_Elips();
  ...
}

23947:
gp_Elips Adaptor3d_IsoCurve::Ellipse() const
{
  ...
  {
    Standard_NoSuchObject::Raise("Adaptor3d_IsoCurve:Ellipse");
- return gp_Elips();
  }
  ...
}

Regarding merge this two changes there is 1 warning in current state of master because function doesn't have any return value. 'Adaptor3d_IsoCurve::Ellipse' : not all control paths return a value

Public activity

3 archived notes

Participants are labeled by their role within this record.

01Commenter 12013-07-16 14:35

Dear apn,
Please, test.

02Author2013-07-17 10:13

Reviewed, branch CR24076 will be tested.

03Author2013-07-17 10:56

Dear Commenter 1,

Branch CR24076 (and products from GIT master) was compiled on Linux and Windows platforms and tested.
[revision removed]

Number of compiler warnings:

occt component :
Linux: 2 (2 on master)
Windows: 7 (8 on master)

products component :
Linux: 0 (0 on master)
Windows: 63 (63 on master)

Regressions:
No regressions

Improvements:
No improvements

Testing cases:
Not needed

Testing on Linux:
Total MEMORY difference: 365563336 / 365734040
Total CPU difference: 46073.060000000754 / 45333.95000000106

Testing on Windows:
Total MEMORY difference: 423974928 / 425278288
Total CPU difference: 33724.75 / 34925.53125

There are not differences in images found by testdiff.