Integration request to master by dpasukhi · Pull Request #670 · Open-Cascade-SAS/OCCT

dpasukhi · 2025-08-14T10:12:30Z

Integration request to collect a sequence of patches as a periodical branch.
Will be available for short time only during development circle.

DEGLTF_Provider.cxx(144,29): warning C4456: declaration of 'aContext' hides previous local declaration

github-actions · 2025-08-14T11:30:05Z

Performance Test Summary

Commit: e170be8b5d4ec2c6ef96e000de0936a3c473159a

Title: Merge 6765a67 into 91000c4

Platform: linux-clang-x64

Total MEMORY difference: 85951816 / 86015806 [-0.07%]
Total CPU difference: 3815.070000000118 / 3913.2600000001335 [-2.51%]
Total IMAGE difference: 12 / 14176 [0.08%]

Platform: macos-x64

Total MEMORY difference: 367812 / 370065 [-0.61%]
Total CPU difference: 4.109999999999999 / 3.769999999999996 [+9.02%]
Total IMAGE difference: 0 / 0

Platform: windows-x64

Total MEMORY difference: 94989371 / 95085133 [-0.10%]
Total CPU difference: 4658.03125 / 4642.5 [+0.33%]
Total IMAGE difference: 19 / 14289 [0.13%]

Fixed regression when retests were failed for not windows.

github-actions · 2025-08-15T09:57:56Z

Performance Test Summary

Commit: 6d00144c96685db2aca34adcda721717d2545789

Title: Merge 02f6fa7 into 91000c4

Platform: linux-clang-x64

Total MEMORY difference: 85435690 / 86017505 [-0.68%]
Total CPU difference: 3887.900000000124 / 3913.3500000001336 [-0.65%]
Total IMAGE difference: 14 / 14176 [0.10%]

Platform: macos-x64

Total MEMORY difference: 368848 / 370065 [-0.33%]
Total CPU difference: 5.949999999999992 / 3.769999999999996 [+57.82%]
Total IMAGE difference: 0 / 0

Platform: windows-x64

Total MEMORY difference: 95001279 / 95086893 [-0.09%]
Total CPU difference: 4571.1875 / 4642.609375 [-1.54%]
Total IMAGE difference: 16 / 14290 [0.11%]

- Standardizes comment formatting by replacing detailed comment blocks with a uniform "//=================================================================================================" separator - Removes redundant "function:" and "purpose:" fields that often contained minimal or no useful information - Maintains code functionality while improving visual consistency across the codebase

github-actions · 2025-08-15T18:29:24Z

Performance Test Summary

Commit: 2c684b8a307e933cc438b625f4b3c60e91705c40

Title: Merge ae4de70 into 91000c4

Platform: linux-clang-x64

Total MEMORY difference: 85954562 / 86016384 [-0.07%]
Total CPU difference: 3879.8800000001092 / 3913.2800000001334 [-0.85%]
Total IMAGE difference: 12 / 14176 [0.08%]

Platform: macos-x64

Total MEMORY difference: 355317 / 370065 [-3.99%]
Total CPU difference: 4.490000000000003 / 3.769999999999996 [+19.10%]
Total IMAGE difference: 0 / 0

Platform: windows-x64

Total MEMORY difference: 95065419 / 95080894 [-0.02%]
Total CPU difference: 4555.625 / 4638.625 [-1.79%]
Total IMAGE difference: 18 / 14289 [0.13%]

github-actions · 2025-08-15T20:19:23Z

Performance Test Summary

Commit: 2c684b8a307e933cc438b625f4b3c60e91705c40

Title: Merge ae4de70 into 91000c4

Platform: linux-clang-x64

Total MEMORY difference: 85957877 / 86018083 [-0.07%]
Total CPU difference: 3877.8800000001193 / 3913.3700000001336 [-0.91%]
Total IMAGE difference: 16 / 14176 [0.11%]

Platform: macos-x64

Total MEMORY difference: 362247 / 370065 [-2.11%]
Total CPU difference: 4.7 / 3.769999999999996 [+24.67%]
Total IMAGE difference: 0 / 0

Platform: windows-x64

Total MEMORY difference: 95065419 / 95080894 [-0.02%]
Total CPU difference: 4555.625 / 4638.625 [-1.79%]
Total IMAGE difference: 18 / 14289 [0.13%]

github-actions · 2025-08-15T21:53:25Z

Performance Test Summary

Commit: 2c684b8a307e933cc438b625f4b3c60e91705c40

Title: Merge ae4de70 into 91000c4

Platform: linux-clang-x64

Total MEMORY difference: 85957877 / 86018083 [-0.07%]
Total CPU difference: 3877.8800000001193 / 3913.3700000001336 [-0.91%]
Total IMAGE difference: 16 / 14176 [0.11%]

Platform: macos-x64

Total MEMORY difference: 362247 / 370065 [-2.11%]
Total CPU difference: 4.7 / 3.769999999999996 [+24.67%]
Total IMAGE difference: 0 / 0

Platform: windows-x64

Total MEMORY difference: 95003184 / 95081587 [-0.08%]
Total CPU difference: 4494.828125 / 4638.625 [-3.10%]
Total IMAGE difference: 17 / 14289 [0.12%]

Added condition to always allocate an array in case of empty list

github-actions · 2025-08-16T09:37:39Z

Performance Test Summary

Commit: 5f1eb1f7ed16cc110fc9a2b502310df5aee0fda7

Title: Merge afccf69 into 91000c4

Platform: linux-clang-x64

Total MEMORY difference: 85947906 / 86015890 [-0.08%]
Total CPU difference: 3811.3600000000947 / 3913.2500000001337 [-2.60%]
Total IMAGE difference: 13 / 14176 [0.09%]

Platform: macos-x64

Total MEMORY difference: 367991 / 370065 [-0.56%]
Total CPU difference: 7.059999999999996 / 3.769999999999996 [+87.27%]
Total IMAGE difference: 0 / 0

Platform: windows-x64

Total MEMORY difference: 95028453 / 95086221 [-0.06%]
Total CPU difference: 4500.09375 / 4642.578125 [-3.07%]
Total IMAGE difference: 11 / 14290 [0.08%]

- Comprehensive translation of French comments to English throughout the StepData module - Enhanced inline documentation explaining complex algorithms and data structures - Improved variable name comments and method descriptions for better code comprehension

github-actions · 2025-08-16T11:22:13Z

Performance Test Summary

Commit: 1519a13316daa89fb67484b8ed62389fb7f27144

Title: Merge a50e3c9 into 91000c4

Platform: linux-clang-x64

Total MEMORY difference: 85957239 / 86017541 [-0.07%]
Total CPU difference: 3832.770000000114 / 3913.3100000001336 [-2.06%]
Total IMAGE difference: 14 / 14176 [0.10%]

Platform: macos-x64

Total MEMORY difference: 376917 / 370065 [+1.85%]
Total CPU difference: 6.439999999999988 / 3.769999999999996 [+70.82%]
Total IMAGE difference: 0 / 0

Platform: windows-x64

Total MEMORY difference: 95012820 / 95081566 [-0.07%]
Total CPU difference: 4395.46875 / 4638.65625 [-5.24%]
Total IMAGE difference: 17 / 14289 [0.12%]

- Replaced try-catch blocks with explicit validation checks in PCDM_ReadWriter_1.cxx - Added division-by-zero protection in XSDRAWSTL.cxx using precision-based comparison - Improved error handling for integer parsing operations

github-actions · 2025-08-16T14:10:34Z

Performance Test Summary

Commit: 519d6cb1afa474dc4a5c3db9865cc2afe291d736

Title: Merge d1b00bf into 91000c4

Platform: linux-clang-x64

Total MEMORY difference: 85975748 / 86017101 [-0.05%]
Total CPU difference: 3812.030000000118 / 3913.3000000001334 [-2.59%]
Total IMAGE difference: 11 / 14175 [0.08%]

Platform: macos-x64

Total MEMORY difference: 372574 / 370065 [+0.68%]
Total CPU difference: 5.5199999999999925 / 3.769999999999996 [+46.42%]
Total IMAGE difference: 0 / 0

Platform: windows-x64

Total MEMORY difference: 95001602 / 95084440 [-0.09%]
Total CPU difference: 4505.109375 / 4642.5 [-2.96%]
Total IMAGE difference: 14 / 14289 [0.10%]

- Updates `.clang-format` to include `OCC_CATCH_SIGNALS`, `DEFINE_STANDARD_RTTIEXT`, and `Draw_Drawable3D_FACTORY` as statement macros - Adds a GitHub action step to automatically remove empty lines after `Standard_DEPRECATED` macro declarations - Reformats numerous files by removing blank lines after deprecated method declarations and improving macro formatting

github-actions · 2025-08-16T21:02:33Z

Performance Test Summary

Commit: 3dc2261bd2cdbc7ac44b21b6a42db72b33aac2c2

Title: Merge 314527f into 91000c4

Platform: linux-clang-x64

Total MEMORY difference: 85979142 / 86016323 [-0.04%]
Total CPU difference: 3824.300000000108 / 3913.2800000001334 [-2.27%]
Total IMAGE difference: 14 / 14176 [0.10%]

Platform: macos-x64

Total MEMORY difference: 374068 / 370065 [+1.08%]
Total CPU difference: 5.399999999999994 / 3.769999999999996 [+43.24%]
Total IMAGE difference: 0 / 0

Platform: windows-x64

Total MEMORY difference: 95018941 / 95086221 [-0.07%]
Total CPU difference: 4387.125 / 4642.578125 [-5.50%]
Total IMAGE difference: 17 / 14290 [0.12%]

- Updated comments in IGESGeom_ToolCircularArc, IGESGeom_ToolCurveOnSurface, IGESGeom_ToolOffsetCurve, IGESGeom_ToolPlane, IGESGeom_ToolSplineSurface, IGESGeom_ToolTransformationMatrix, IGESGraph_ToolColor, IGESGraph_ToolDrawingUnits, IGESSelect, IGESSelect_Activator, IGESSelect_RebuildGroups, IGESSelect_SelectBypassGroup, IGESSelect_SelectBypassSubfigure, IGESSelect_SignStatus, IGESSelect_ViewSorter, IGESSelect_WorkLibrary, IGESToBRep, IGESToBRep_BRepEntity, IGESToBRep_BasicCurve, IGESToBRep_BasicSurface, IGESToBRep_TopoCurve, IGESToBRep_TopoSurface to improve readability and understanding. - Translated French comments to English where applicable. - Ensured comments accurately reflect the functionality and intent of the code.

github-actions · 2025-08-17T10:10:28Z

Performance Test Summary

Commit: 58339555e8c2967d4966dc62567bc9670629a604

Title: Merge 45eefc4 into 91000c4

Platform: linux-clang-x64

Total MEMORY difference: 85956987 / 86016985 [-0.07%]
Total CPU difference: 3822.760000000119 / 3913.2900000001337 [-2.31%]
Total IMAGE difference: 18 / 14176 [0.13%]

Platform: macos-x64

Total MEMORY difference: 378068 / 370065 [+2.16%]
Total CPU difference: 4.570000000000003 / 3.769999999999996 [+21.22%]
Total IMAGE difference: 0 / 0

Platform: windows-x64

Total MEMORY difference: 95014330 / 95085133 [-0.07%]
Total CPU difference: 4483.703125 / 4642.5 [-3.42%]
Total IMAGE difference: 15 / 14289 [0.10%]

- Updates function calls from `sprintf` to `Sprintf` across the entire codebase - Updates comments and documentation references to reflect the new function name - Adds format attribute annotations to the `Sprintf` function declaration for better compiler validation

github-actions · 2025-08-17T13:05:07Z

Performance Test Summary

Commit: a27fc71709d20b92ad83c8c2374c865c62b6eb2f

Title: Merge 5ae60b5 into 91000c4

Platform: linux-clang-x64

Total MEMORY difference: 85959436 / 86017123 [-0.07%]
Total CPU difference: 3856.4700000001076 / 3913.3000000001334 [-1.45%]
Total IMAGE difference: 13 / 14175 [0.09%]

Platform: macos-x64

Total MEMORY difference: 377705 / 370065 [+2.06%]
Total CPU difference: 5.529999999999991 / 3.769999999999996 [+46.68%]
Total IMAGE difference: 0 / 0

Platform: windows-x64

Total MEMORY difference: 94977194 / 95081982 [-0.11%]
Total CPU difference: 4490.53125 / 4638.703125 [-3.19%]
Total IMAGE difference: 19 / 14290 [0.13%]

- Updated French comments to English for better accessibility and understanding. - Enhanced clarity of comments to better describe the functionality and purpose of the code. - Ensured consistency in terminology and phrasing throughout the codebase. - Made minor adjustments to comment formatting for improved readability.

github-actions · 2025-08-17T19:56:41Z

Performance Test Summary

Commit: 9a2c1361540feb863f6b7b7fbb48ea921c9ba473

Title: Merge 3eb7ce1 into 91000c4

Platform: linux-clang-x64

Total MEMORY difference: 85974393 / 86018083 [-0.05%]
Total CPU difference: 3884.6600000001054 / 3913.3700000001336 [-0.73%]
Total IMAGE difference: 11 / 14176 [0.08%]

Platform: macos-x64

Total MEMORY difference: 370802 / 370065 [+0.20%]
Total CPU difference: 5.9299999999999935 / 3.769999999999996 [+57.29%]
Total IMAGE difference: 0 / 0

Platform: windows-x64

Total MEMORY difference: 94989313 / 95081566 [-0.10%]
Total CPU difference: 4567.25 / 4638.65625 [-1.54%]
Total IMAGE difference: 12 / 14289 [0.08%]

Fix sscanf condition checks in iges_lire function for better error handling

- Replaced hardcoded buffer sizes with `constexpr size_t aBufferSize` constants - Added `width()` calls to input streams before reading to prevent buffer overflows - Applied consistent buffer size management across multiple input operations

github-actions · 2025-08-17T21:12:22Z

Performance Test Summary

Commit: 20e720c7688f66e769a144d9eef75b5df7fca951

Title: Merge 5c87568 into 91000c4

Platform: linux-clang-x64

Total MEMORY difference: 85940956 / 86018083 [-0.09%]
Total CPU difference: 3825.090000000104 / 3913.3700000001336 [-2.26%]
Total IMAGE difference: 12 / 14176 [0.08%]

Platform: macos-x64

Total MEMORY difference: 367026 / 370065 [-0.82%]
Total CPU difference: 5.909999999999994 / 3.769999999999996 [+56.76%]
Total IMAGE difference: 0 / 0

Platform: windows-x64

Total MEMORY difference: 95033132 / 95086200 [-0.06%]
Total CPU difference: 4470.4375 / 4642.609375 [-3.71%]
Total IMAGE difference: 12 / 14290 [0.08%]

github-actions · 2025-08-17T22:25:06Z

Performance Test Summary

Commit: 8d73bfe6bc203302b8d4252447493021de897052

Title: Merge 5c814b0 into 91000c4

Platform: linux-clang-x64

Total MEMORY difference: 85938275 / 86016545 [-0.09%]
Total CPU difference: 3832.2100000001215 / 3913.2800000001334 [-2.07%]
Total IMAGE difference: 13 / 14175 [0.09%]

Platform: macos-x64

Total MEMORY difference: 364142 / 370065 [-1.60%]
Total CPU difference: 6.429999999999988 / 3.769999999999996 [+70.56%]
Total IMAGE difference: 0 / 0

Platform: windows-x64

Total MEMORY difference: 95015082 / 95081587 [-0.07%]
Total CPU difference: 4493.03125 / 4638.625 [-3.14%]
Total IMAGE difference: 17 / 14289 [0.12%]

Coding, DEGLTF_Provider - Hiding declaration (#669)

6765a67

DEGLTF_Provider.cxx(144,29): warning C4456: declaration of 'aContext' hides previous local declaration

github-project-automation Bot added this to Maintenance Aug 14, 2025

dpasukhi added the 2. IR Integration request (Used for marking PR as a collection of multiple integration for master) label Aug 14, 2025

github-project-automation Bot moved this to Todo in Maintenance Aug 14, 2025

Testing - Removed old test reference (#673)

02f6fa7

Fixed regression when retests were failed for not windows.

Data Exchange, STP - Crash on empty list (#671)

afccf69

Added condition to always allocate an array in case of empty list

Coding - Reducing relying on exceptions (#676)

d1b00bf

- Replaced try-catch blocks with explicit validation checks in PCDM_ReadWriter_1.cxx - Added division-by-zero protection in XSDRAWSTL.cxx using precision-based comparison - Improved error handling for integer parsing operations

dpasukhi added 2 commits August 17, 2025 21:01

Coding - Incorrect return-value check for a 'scanf'-like function (#680)

5c87568

Fix sscanf condition checks in iges_lire function for better error handling

Coding - Dangerous use of 'cin' (#681)

5c814b0

- Replaced hardcoded buffer sizes with `constexpr size_t aBufferSize` constants - Added `width()` calls to input streams before reading to prevent buffer overflows - Applied consistent buffer size management across multiple input operations

AtheneNoctuaPt approved these changes Aug 18, 2025

View reviewed changes

github-project-automation Bot moved this from Todo to Integration in Maintenance Aug 18, 2025

dpasukhi merged commit 5c814b0 into master Aug 18, 2025
23 checks passed

github-project-automation Bot moved this from Integration to Done in Maintenance Aug 18, 2025

Conversation

dpasukhi commented Aug 14, 2025

Uh oh!

github-actions Bot commented Aug 14, 2025

Uh oh!

github-actions Bot commented Aug 15, 2025

Uh oh!

github-actions Bot commented Aug 15, 2025

Uh oh!

github-actions Bot commented Aug 15, 2025

Uh oh!

github-actions Bot commented Aug 15, 2025

Uh oh!

github-actions Bot commented Aug 16, 2025

Uh oh!

github-actions Bot commented Aug 16, 2025

Uh oh!

github-actions Bot commented Aug 16, 2025

Uh oh!

github-actions Bot commented Aug 16, 2025

Uh oh!

github-actions Bot commented Aug 17, 2025

Uh oh!

github-actions Bot commented Aug 17, 2025

Uh oh!

github-actions Bot commented Aug 17, 2025

Uh oh!

github-actions Bot commented Aug 17, 2025

Uh oh!

github-actions Bot commented Aug 17, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants