MantisBT - Community
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0023348Community[OCCT] OCCT:DRAWpublic2012-07-25 19:472012-11-16 13:17
ReporterPawel 
Assigned ToPawel 
PrioritynormalSeverityminor 
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target Version[OCCT] 6.5.4Fixed in Version[OCCT] 6.5.4 
Test case numberNot needed
Summary0023348: Expression 'ii >= 0' is always true. Unsigned type value is always >= 0.
DescriptionIn Draw_Interpretor.cxx, line 391
Problem introduced after changing some variable to 'Standard_Size'
Steps To Reproduce
Additional information
and documentation updates
TagsNo tags attached.
Relationships
related to 0023072closed dbv Open CASCADE Eliminate compiler warnings (level 3) on Windows / MSVC++ 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2012-07-25 19:47PawelNew Issue
2012-07-25 19:47PawelAssigned To => Pawel
2012-07-25 19:48PawelRelationship addedrelated to 0023072
2012-07-26 12:59PawelNote Added: 0021159
2012-07-26 12:59PawelAssigned ToPawel => bugmaster
2012-07-26 12:59PawelStatusnew => resolved
2012-07-27 08:48abvAssigned Tobugmaster => Pawel
2012-07-27 08:51abvNote Added: 0021170
2012-07-27 11:55PawelNote Added: 0021172
2012-07-27 11:55PawelAssigned ToPawel => bugmaster
2012-07-27 11:55PawelStatusresolved => reviewed
2012-07-31 10:03apnNote Added: 0021188
2012-07-31 10:04apnTest case number => Not needed
2012-07-31 10:04apnStatusreviewed => tested
2012-08-06 16:35PawelChangeset attached => occt master d33dea30
2012-08-06 16:35PawelAssigned Tobugmaster => Pawel
2012-08-06 16:35PawelStatustested => verified
2012-08-06 16:35PawelResolutionopen => fixed
2012-11-16 13:14bugmasterFixed in Version => 6.5.4
2012-11-16 13:17bugmasterStatusverified => closed

Notes
(0021159)
Pawel   
2012-07-26 12:59   
Corresponding git branch pushed. Please review.
(0021170)
abv   
2012-07-27 08:51   
I have modified the code to use OSD_Path instead of parsing C strings; this makes the code more clear and also makes it working on Windows (where original path contains reversed slashes which were not considered).

The new test case added: demo draw getsource

Please review
(0021172)
Pawel   
2012-07-27 11:55   
No remarks. Please forward for testing.
(0021188)
apn   
2012-07-31 10:03   
Dear BugMaster,
Branch CR23348 (and products from GIT master) was compiled on Linux and Windows platforms and tested.

Regression:
Not detected

Improvements:
Not detected

Testing case:
Not needed