View Issue Details

Related ChangesetsJump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0023289CommunityOCCT:Modeling Algorithmspublic2012-07-16 16:242012-11-16 13:18
ReporterPawel Assigned Toaba 
PrioritynormalSeverityminor 
Status closedResolutionfixed 
PlatformWindowsOSVC++ 2008 
Product Version6.5.3 
Target Version6.5.4Fixed in Version6.5.4 
Summary0023289: IntCurve_IntPolyPolyGen.gxx, suspicious else
Descriptionin lines 805:

if ((D1.LastParameter()-D1.FirstParameter()) > 0.5*(DeltaU))
    nbsamplesOnC1=(5*(nbsamplesOnC1*NbIter))/4;
      else
    nbsamplesOnC1=(5*(nbsamplesOnC1*NbIter))/4;

and 822:

 if ((D2.LastParameter()-D2.FirstParameter()) > 0.5*(DeltaV))
    nbsamplesOnC2=(5*(nbsamplesOnC2*NbIter))/4;
      else
    nbsamplesOnC2=(5*(nbsamplesOnC2*NbIter))/4;
TagsNo tags attached.
Test case numberNot needed

Relationships

child of 0023132 closedomy Suspicious code snippets 

Activities

aba

2012-10-17 15:05

developer   ~0021817

Conditional blocks in lines 805 and 822 were removed as lines 'nbsamplesOnC1=(5*(nbsamplesOnC1*NbIter))/4;' and 'nbsamplesOnC2=(5*(nbsamplesOnC2*NbIter))/4;' are executed in any case.

aba

2012-10-17 15:07

developer   ~0021818

The git branch CR23289 is ready to be reviewed.

Dear oan, please review.

oan

2012-10-22 11:15

developer   ~0021874

No comments.

Dear JGV,
Please review. Is it right to use the same equation in both cases or another coefficient should be used?

jgv

2012-11-06 13:32

developer   ~0022097

Please find and correct another similar blocks where <nbsamples> is initialized by strange way.

aba

2012-11-06 15:10

developer   ~0022102

Another strange initialization of 'nbsamples' variable was found in IntCurve_IntPolyPolyGen.gxx. The Git branch CR23289 was updated.
Dear jgv, please review.

jgv

2012-11-06 16:59

developer   ~0022109

No remarks

mkv

2012-11-07 17:10

tester   ~0022137

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

Regressions:
Not detected

Improvements:
Not detected

Testing cases:
Not needed

Related Changesets

occt: master 5d351552

2012-11-09 11:47:10

aba

Details Diff		 0023289: IntCurve_IntPolyPolyGen.gxx, suspicious else

if-else blocks in lines 805 and 822 were removed.
Corrected initialization of 'nbsamples' variable		 Affected Issues
0023289
mod - src/IntCurve/IntCurve_IntPolyPolyGen.gxx Diff File

Issue History

Date Modified Username Field Change
2012-07-16 16:24 Pawel New Issue
2012-07-16 16:24 Pawel Assigned To => bugmaster
2012-07-16 16:25 Pawel Relationship added child of 0023132
2012-10-09 18:46 oan Assigned To bugmaster => aba
2012-10-09 18:46 oan Status new => assigned
2012-10-17 15:05 aba Note Added: 0021817
2012-10-17 15:07 aba Note Added: 0021818
2012-10-17 15:07 aba Status assigned => resolved
2012-10-17 15:07 aba Assigned To aba => oan
2012-10-22 11:15 oan Note Added: 0021874
2012-11-01 12:43 oan Assigned To oan => jgv
2012-11-06 13:29 jgv Assigned To jgv => aba
2012-11-06 13:32 jgv Note Added: 0022097
2012-11-06 13:32 jgv Status resolved => assigned
2012-11-06 15:10 aba Note Added: 0022102
2012-11-06 15:10 aba Assigned To aba => jgv
2012-11-06 15:10 aba Status assigned => resolved
2012-11-06 16:59 jgv Note Added: 0022109
2012-11-06 16:59 jgv Status resolved => reviewed
2012-11-06 18:06 mkv Assigned To jgv => mkv
2012-11-07 17:10 mkv Note Added: 0022137
2012-11-07 17:12 mkv Test case number => Not needed
2012-11-07 17:12 mkv Assigned To mkv => bugmaster
2012-11-07 17:12 mkv Status reviewed => tested
2012-11-12 11:33 aba Changeset attached => occt master 5d351552
2012-11-12 11:33 aba Assigned To bugmaster => aba
2012-11-12 11:33 aba Status tested => verified
2012-11-12 11:33 aba Resolution open => fixed
2012-11-16 13:13 bugmaster Fixed in Version => 6.5.4
2012-11-16 13:18 bugmaster Status verified => closed