OCCT3D
DiscussionsIssue archiveOCCT:DRAW

Archived issue #0031731

Draw Harness - colorize errors and exception messages

Open CASCADEOCCT:DRAWclosed12 public notes

Search issues

Description

It would be nice printing unhandled exceptions and error messages in red.

Steps to reproduce

pload VISUALIZATION XDE
XGetColor
XAddShape
# error put into theDI

pload MODELING
box b 0 0 0
# Standard_DomainError exception

Public activity

12 archived notes

Participants are labeled by their role within this record.

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

[revision removed]


Detailed log of new commits:

Author: Author
Date: Mon Aug 24 19:13:08 2020 +0300

    0031731: Draw Harness - colorize errors and exception messages
    
    Draw_Interpretor, CommandCmd() - catched exceptions and messages put into Tcl string result
    before throwing a Tcl exception (return 1) are now print in intense red (using Message::SendFail()).
    Duplication of exception message in std::cout and Tcl output has been removed.
    
    Draw Harness plugins have been updated to use either Message::SendFail() or theDI instead of std::cout/std::cerr
    for printing colored error message before throwing a Tcl exception.
02Commenter 3
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
03Commenter 3
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Mon Aug 24 19:13:08 2020 +0300

    0031731: Draw Harness - colorize errors and exception messages
    
    Draw_Interpretor, CommandCmd() - catched exceptions and messages put into Tcl string result
    before throwing a Tcl exception (return 1) are now print in intense red (using Message::SendFail()).
    Duplication of exception message in std::cout and Tcl output has been removed.
    
    Draw Harness plugins have been updated to use either Message::SendFail() or theDI instead of std::cout/std::cerr
    for printing colored error message before throwing a Tcl exception.
04Commenter 3
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
05Commenter 3
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
06Commenter 3
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
07Commenter 3
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
08Author
Patch is ready for review:
- OCCT branch CR31731_1
- OCC Products branch CR31731_1

http://jenkins-test-12.nnov.opencascade.com:8080/view/CR31731_1-CR31731_1-KGV
09Commenter 9
No remarks, please integrate
10Commenter 1
Combination -
OCCT branch : [archived branch]
master SHA - [revision removed]
[revision removed]
Products branch : [archived branch] SHA - [revision removed]
was compiled on Linux, MacOS and Windows platforms and tested in optimize mode.

Number of compiler warnings:
No new/fixed warnings

Regressions/Differences/Improvements:
No regressions/differences

CPU differences:
Debian80-64:
OCCT
Total CPU difference: 17249.00000000021 / 17322.48000000014 [-0.42%]
Products
Total CPU difference: 12007.590000000067 / 11882.730000000085 [+1.05%]
Windows-64-VC14:
OCCT
Total CPU difference: 18729.984375 / 18716.078125 [+0.07%]
Products
Total CPU difference: 13222.90625 / 13242.09375 [-0.14%]


Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention
11Commenter 3
Branch [archived branch] has been deleted by Participant.

[revision removed]
12Commenter 3
Branch [archived branch] has been deleted by Participant.

[revision removed]

Related records