DiscussionsIssue archiveOCCT:Foundation Classes

Archived issue #0028550

Foundation Classes - fix empty message passed to thrown exception

Open CASCADEOCCT:Foundation Classesclosed10 public notes

Search issues

Description

gp_Dir throws exceptions in many cases, however message is always empty:
Standard_ConstructionError_Raise_if (D <= gp::Resolution(), "");

This makes description of unhandled exceptions (e.g. using Standard_Failure::GetMessageString()) non-informative.

Public activity

10 archived notes

Participants are labeled by their role within this record.

01Commenter 2
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Tue Apr 18 18:58:12 2017 +0300

    0028550: Foundation Classes - fix empty message passed to thrown exception
02Author
Patch is ready for review.
03Commenter 3
Please remove exclamation signs at the end of exception messages. Also please replace "IsDone() should be checked before" by "no result" to indicate the reason of exception instead of referring to assumed way of misuse.
04Commenter 2
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Tue Apr 18 18:58:12 2017 +0300

    0028550: Foundation Classes - fix empty message passed to thrown exception
05Commenter 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
06Author
Remarks have been applied.
07Commenter 7
No remarks, please test
08Commenter 8
Dear Commenter 1,

Branch CR28550_1 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested.
[revision removed]

Number of compiler warnings:
occt component:
   Linux: 0 (0 on master)
   Windows: 0 (0 on master)
   MasOS: 0 (0 on master)
products component:
   Linux: 64
   Windows: 0
   MacOS: 1198

Regressions/Differences:
Not detected

Testing cases:
Not needed

Testing on Linux:
Total MEMORY difference: 92277963 / 90727282 [+1.71%]
Total CPU difference: 19707.110000000313 / 19728.57000000033 [-0.11%]

Testing on Windows:
Total MEMORY difference: 57802591 / 57804189 [-0.00%]
Total CPU difference: 18331.989511998592 / 18136.473458698543 [+1.08%]
09Commenter 2
Branch [archived branch] has been deleted by Author.

[revision removed]
10Commenter 2
Branch [archived branch] has been deleted by Author.

[revision removed]

Related records