lili/OCCT
1
0
Fork 0
mirror of https://github.com/Open-Cascade-SAS/OCCT.git synced 2026-06-15 12:07:41 +08:00
Code Issues Packages Projects Releases Wiki Activity

Coding - Fixing analyzer reports (#885)

Browse Source 
- Replaced manual absolute value logic (conditional negation) with `std::abs` for clarity
- Added `const` qualifiers to variables that are not modified after initialization
- Moved variable declarations closer to their first use and removed unused variable assignments
This commit is contained in:
Pasukhin Dmitry
2025-12-05 21:20:08 +00:00
committed by GitHub
parent d349cd4a67
commit c770a1a7f7
28 changed files with 142 additions and 249 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/ModelingData/TKGeomBase/GeomConvert/GeomConvert_1.cxx
View File

@@ -566,12 +566,10 @@ Handle(Geom_BSplineSurface) GeomConvert::SurfaceToBSplineSurface(const Handle(Ge
if (C->IsRational())
C->Weights(Weights);
Standard_Integer nbUSpans;
Standard_Real AlfaU;
Standard_Real AlfaU;
if (Strim->IsUPeriodic())
{
NbUKnots = 4;
nbUSpans = 3;
AlfaU = M_PI / 3.;
NbUPoles = 6;
periodic = Standard_True;
@@ -579,7 +577,8 @@ Handle(Geom_BSplineSurface) GeomConvert::SurfaceToBSplineSurface(const Handle(Ge
else
{
// Nombre de spans : ouverture maximale = 150 degres ( = PI / 1.2 rds)
nbUSpans = (Standard_Integer)std::trunc(1.2 * (ULast - UFirst) / M_PI) + 1;
const Standard_Integer nbUSpans =
(Standard_Integer)std::trunc(1.2 * (ULast - UFirst) / M_PI) + 1;
AlfaU = (ULast - UFirst) / (nbUSpans * 2);
NbUPoles = 2 * nbUSpans + 1;
NbUKnots = nbUSpans + 1;