View Issue Details

IDProjectCategoryView StatusLast Update
0024912CommunityOCCT:Codingpublic2014-11-11 12:52
ReporterEpy Assigned Toapn  
PrioritylowSeverityintegration request 
Status closedResolutionfixed 
PlatformLinuxOSDebian 6.0 
Product Version6.7.1 
Target Version6.8.0Fixed in Version6.8.0 
Summary0024912: Remove unused Graphic3d_Strips
DescriptionClang gives the following new warnings with 6.7.1:

src/Graphic3d/Graphic3d_Strips.cxx:248:7: warning:
template argument uses local type 'edg' [-Wlocal-type-template-args]
Standard::Free(cedge);

src/Graphic3d/Graphic3d_Strips.cxx:251:3: warning:
template argument uses local type 'edg' [-Wlocal-type-template-args]
Standard::Free(edges);

src/Graphic3d/Graphic3d_Strips.cxx:889:7: warning:
template argument uses local type 'edg' [-Wlocal-type-template-args]
Standard::Free(cedge);

src/Graphic3d/Graphic3d_Strips.cxx:892:3: warning:
template argument uses local type 'edg' [-Wlocal-type-template-args]
Standard::Free(edges);

This is allowed per C++11, however, this is prohibited in C++03. Better to make the template a global type.
TagsNo tags attached.
Test case numberNot needed

Relationships

related to 0024489 closedbugmaster Open CASCADE Avoid type casts in call to Standard::Free() 

Activities

kgv

2014-05-12 08:03

developer   ~0029269

Last edited: 2014-05-12 08:04

Dear Sergey,

the file looks abandoned and placed in wrong package - probably it is more reasonable to remove it entirely.
Could you please comment if there are any known users of this functionality?

kgv

2014-05-13 22:27

developer   ~0029310

Please check compilation of the patch in branch CR24912 - regression testing is not needed.

apn

2014-05-15 13:55

administrator   ~0029345

Dear BugMaster,

Branch CR24912 (and products from GIT master) was compiled on Linux, Windows and MacOS platforms.
SHA-1: 380e2e2de4e20a3c2dbdeb66ccf4f6bfd24b352c

Number of compiler warnings:

occt component :
Linux: 17 (17 on master)
Windows: 0 (0 on master)
MacOS: 201 (212 on master)

products component :
Linux: 11 (11 on master)
Windows: 2 (2 on master)

Related Changesets

occt: master 9efc0d14

2014-05-13 18:25:11

kgv


Committer: apn Details Diff
0024912: Remove unused Graphic3d_Strips Affected Issues
0024912
mod - src/Graphic3d/Graphic3d.cdl Diff File
rm - src/Graphic3d/Graphic3d_Strips.cdl Diff File
rm - src/Graphic3d/Graphic3d_Strips.cxx Diff File

Issue History

Date Modified Username Field Change
2014-05-09 02:02 Epy New Issue
2014-05-09 02:02 Epy Assigned To => kgv
2014-05-09 09:10 abv Target Version => 6.8.0
2014-05-12 07:54 kgv Relationship added related to 0024489
2014-05-12 07:57 kgv Assigned To kgv => san
2014-05-12 07:57 kgv Severity minor => integration request
2014-05-12 07:57 kgv Priority normal => low
2014-05-12 08:03 kgv Note Added: 0029269
2014-05-12 08:04 kgv Note Edited: 0029269
2014-05-13 22:27 kgv Note Added: 0029310
2014-05-13 22:27 kgv Assigned To san => bugmaster
2014-05-13 22:27 kgv Status new => resolved
2014-05-13 22:27 kgv Summary Template argument uses local type in src/Graphic3d/Graphic3d_Strips.cxx => Remove unused Graphic3d_Strips
2014-05-13 22:27 kgv Status resolved => reviewed
2014-05-14 16:38 apn Assigned To bugmaster => apn
2014-05-15 13:55 apn Note Added: 0029345
2014-05-15 13:55 apn Test case number => Not needed
2014-05-15 13:55 apn Assigned To apn => bugmaster
2014-05-15 13:55 apn Status reviewed => tested
2014-05-16 14:44 apn Changeset attached => occt master 9efc0d14
2014-05-16 14:44 apn Assigned To bugmaster => apn
2014-05-16 14:44 apn Status tested => verified
2014-05-16 14:44 apn Resolution open => fixed
2014-11-11 12:46 aiv Fixed in Version => 6.8.0
2014-11-11 12:52 aiv Status verified => closed