View Issue Details

IDProjectCategoryView StatusLast Update
0023266CommunityOCCT:DRAWpublic2012-11-16 13:17
ReporterPawel Assigned Toika  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
PlatformWindowsOSVC++ 2005 
Product Version6.5.3 
Target Version6.5.4Fixed in Version6.5.4 
Summary0023266: cppcheck warning: Logical disjunction always evaluates to true
DescriptionIn QANewDBRepNaming_FeatureCommands.cxx the line (104):

if (nb != 6 || nb != 7)

always yields 'true' and so the method always returns.
TagsNo tags attached.
Test case numberNot needed

Relationships

duplicate of 0023098 closeddbv Cppcheck warnings/errors in QA files 

Activities

ika

2012-10-09 18:12

developer   ~0021699

Git branch CR23266 is ready to be reviewed.

Dear Oleg,
Please review.

oan

2012-10-09 19:19

developer   ~0021702

No comments, please test.

mkv

2012-10-11 19:16

tester   ~0021735

Dear BugMaster,
Branch CR23266 (and products from GIT master) was compiled on Linux and Windows platforms and tested.

Regressions:
Not detected

Improvements:
Not detected

Testing cases:
Not needed

See results in /QADisk/occttests/results/KAS/dev/CR23266-master-products_11102012/lin
See reference results in /QADisk/occttests/results/KAS/dev/IR-2012-10-05-master-products_05102012/lin
See test cases in /QADisk/occttests/tests/ED

Related Changesets

occt: master a7ae5c81

2012-10-09 14:00:27

ika

Details Diff
0023266: cppcheck warning: Logical disjunction always evaluates to true

Logical disjunction changed to Logical conjunction, it evaluates true only then number of parameters isn't right.
Affected Issues
0023266
mod - src/QANewDBRepNaming/QANewDBRepNaming_FeatureCommands.cxx Diff File

Issue History

Date Modified Username Field Change
2012-07-12 13:59 Pawel New Issue
2012-07-12 13:59 Pawel Assigned To => bugmaster
2012-10-04 11:59 Pawel Relationship added duplicate of 0023098
2012-10-09 10:16 oan Assigned To bugmaster => ika
2012-10-09 10:19 oan Status new => assigned
2012-10-09 18:10 ika Assigned To ika => oan
2012-10-09 18:12 ika Note Added: 0021699
2012-10-09 19:17 oan Status assigned => resolved
2012-10-09 19:19 oan Note Added: 0021702
2012-10-09 19:19 oan Status resolved => reviewed
2012-10-10 18:51 mkv Assigned To oan => mkv
2012-10-11 19:16 mkv Note Added: 0021735
2012-10-11 19:18 mkv Test case number => Not needed
2012-10-11 19:18 mkv Assigned To mkv => bugmaster
2012-10-11 19:18 mkv Status reviewed => tested
2012-10-16 15:21 ika Changeset attached => occt master a7ae5c81
2012-10-16 15:21 ika Assigned To bugmaster => ika
2012-10-16 15:21 ika Status tested => verified
2012-10-16 15:21 ika Resolution open => fixed
2012-11-16 13:14 bugmaster Fixed in Version => 6.5.4
2012-11-16 13:17 bugmaster Status verified => closed