View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0023351 | Community | OCCT:DRAW | public | 2012-07-26 13:26 | 2012-11-16 13:17 |
Reporter | Pawel | Assigned To | |||
Priority | normal | Severity | minor | ||
Status | closed | Resolution | fixed | ||
Platform | A | OS | L | ||
Product Version | 6.5.3 | ||||
Target Version | 6.5.4 | Fixed in Version | 6.5.4 | ||
Summary | 0023351: The use of 'if (A) {...} else if (A) {...}' pattern was detected. | ||||
Description | In test TestTopOpe_BOOPCommands.cxx lines 90 and 110 lines 137 and 157 Was this intended? | ||||
Tags | No tags attached. | ||||
Test case number | Not needed | ||||
|
Fix is integrated to branch CR23351 (see commit message for details). Please review |
|
I have no comments, please test |
|
In the last commit, I suppose that "if (typ != TopAbs_SHAPE)" could be replaced by "else" |
|
I suppose there's no need to replace "if (typ != TopAbs_SHAPE)" by "else" because it will be necessary to change function logic a little and may cause some additional problems. |
|
Dear BugMaster, Branch CR23351 (and master from occt-products) was compiled on Linux and Windows platforms and tested. Regressions: Not detected Improvements: Not detected Testing cases: Not needed |
occt: master 7da2a6ab 2012-10-19 14:09:57
|
0023351: The use of 'if (A) {...} else if (A) {...}' pattern was detected. Got rid of duplicated code. Got rid of redundant conditions' checks in Standard_Integer TOPOC(Draw_Interpretor& interpretor,Standard_Integer na,const char** a) function. |
Affected Issues 0023351 |
|
mod - src/TestTopOpe/TestTopOpe_BOOPCommands.cxx | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2012-07-26 13:26 | Pawel | New Issue | |
2012-07-26 13:26 | Pawel | Assigned To | => bugmaster |
2012-08-17 16:42 | bugmaster | Assigned To | bugmaster => emv |
2012-08-17 16:42 | bugmaster | Status | new => assigned |
2012-10-09 10:17 | oan | Assigned To | emv => omy |
2012-10-11 14:58 |
|
Note Added: 0021724 | |
2012-10-11 15:00 |
|
Assigned To | omy => oan |
2012-10-11 15:15 |
|
Status | assigned => resolved |
2012-10-12 11:40 | oan | Note Added: 0021744 | |
2012-10-12 11:40 | oan | Status | resolved => reviewed |
2012-10-12 12:58 |
|
Note Added: 0021749 | |
2012-10-12 17:47 |
|
Assigned To | oan => mkv |
2012-10-16 09:28 |
|
Note Added: 0021777 | |
2012-10-16 10:21 |
|
Note Added: 0021779 | |
2012-10-16 10:21 |
|
Note Edited: 0021779 | |
2012-10-16 10:22 |
|
Test case number | => Not needed |
2012-10-16 10:22 |
|
Assigned To | mkv => bugmaster |
2012-10-16 10:22 |
|
Status | reviewed => tested |
2012-10-23 20:41 |
|
Changeset attached | => occt master 7da2a6ab |
2012-10-23 20:45 |
|
Assigned To | bugmaster => omy |
2012-10-23 20:45 |
|
Status | tested => verified |
2012-10-23 20:45 |
|
Resolution | open => fixed |
2012-11-16 13:14 | bugmaster | Fixed in Version | => 6.5.4 |
2012-11-16 13:17 | bugmaster | Status | verified => closed |