View Issue Details

IDProjectCategoryView StatusDate SubmittedLast Update
0025015CommunityOCCT:DRAWpublic2014-06-17 18:222014-06-19 13:02
ReporterTimo Assigned ToTimo  
PrioritynormalSeveritytweak 
Status closedResolutionno change required 
Product Version6.7.0 
Target Version6.8.0 
Summary0025015: Remove duplicate Draw commands trollingball and brollingball
DescriptionIn BRepTest_FilletCommands.cxx there are 3 Draw commands that all do the same: rollingball, brollingball, trollingball.

Were they retained for backwards compatibility?
It would be cleaner to keep only one of them, preferrably "rollingball".

They are defined in BRepTest::FilletCommands:
...

  theCommands.Add("rollingball",
          "rollingball r S radius [stopf1 ..] @ [f1 f2 ..] @ [e1 ..]",
          __FILE__,
          rollingball);

  theCommands.Add("brollingball",
          "brollingball r S radius [stopf1 ..] @ [f1 f2 ..] @ [e1 ..]",
          __FILE__,
          rollingball);

  theCommands.Add("trollingball",
          "trollingball r S radius [stopf1 ..] @ [f1 f2 ..] @ [e1 ..]",
          __FILE__,
          rollingball);
TagsNo tags attached.
Test case numberNot needed

Activities

Timo

2014-06-18 12:51

developer   ~0029811

Last edited: 2014-06-18 12:52

 Sorry, I didn't check the code in detail.
There is a difference between the three.
brollingball and trollingball switch on options of rollingball.

So, this issue could be deleted, but I don't have the rights to do it.

mkv

2014-06-18 14:26

tester   ~0029814

Dear Bugmaster, could you close this issue.

Issue History

Date Modified Username Field Change
2014-06-17 18:22 Timo New Issue
2014-06-17 18:22 Timo Assigned To => mkv
2014-06-18 12:51 Timo Note Added: 0029811
2014-06-18 12:51 Timo Status new => resolved
2014-06-18 12:52 Timo Note Edited: 0029811
2014-06-18 14:26 mkv Note Added: 0029814
2014-06-18 14:27 mkv Test case number => Not needed
2014-06-18 14:27 mkv Assigned To mkv => bugmaster
2014-06-18 14:27 mkv Status resolved => reviewed
2014-06-18 14:27 mkv Status reviewed => tested
2014-06-19 13:02 apn Assigned To bugmaster => Timo
2014-06-19 13:02 apn Status tested => closed
2014-06-19 13:02 apn Resolution open => no change required