OpenCascade/occt
1
0
Fork 0
mirror of https://git.dev.opencascade.org/repos/occt.git synced 2025-04-03 17:56:21 +03:00
Code Packages Releases Activity

Coding - Resolving C26439 & type formatting warnings

Browse Source 
Sprintf  with %s always convert values into char*, not safety from int.
Move operators and constructors can be marked as noexcept
This commit is contained in:
dpasukhi 2024-09-10 17:40:12 +00:00
parent 9c6914c3cc
commit 72b244bc98
4 changed files with 5 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/BOPTest/BOPTest_BOPCommands.cxx
View File

@ -229,7 +229,6 @@ Standard_Integer bopsmt(Draw_Interpretor& di,
return 0; return 0;
} }
// //
char buf[64];
Standard_Boolean bRunParallel; Standard_Boolean bRunParallel;
Standard_Integer aNb; Standard_Integer aNb;
BOPAlgo_BOP aBOP; BOPAlgo_BOP aBOP;
@ -237,8 +236,7 @@ Standard_Integer bopsmt(Draw_Interpretor& di,
const TopTools_ListOfShape& aLC=pPF->Arguments(); const TopTools_ListOfShape& aLC=pPF->Arguments();
aNb=aLC.Extent(); aNb=aLC.Extent();
if (aNb!=2) { if (aNb!=2) {
Sprintf (buf, " wrong number of arguments %s\n", aNb); di << " wrong number of arguments " << aNb << '\n';
di << buf;
return 0; return 0;
} }
// //
@ -298,7 +296,6 @@ Standard_Integer bopsection(Draw_Interpretor& di,
return 0; return 0;
} }
// //
char buf[64];
Standard_Boolean bRunParallel; Standard_Boolean bRunParallel;
Standard_Integer aNb; Standard_Integer aNb;
BOPAlgo_Section aBOP; BOPAlgo_Section aBOP;
@ -306,8 +303,7 @@ Standard_Integer bopsection(Draw_Interpretor& di,
const TopTools_ListOfShape& aLC=pPF->Arguments(); const TopTools_ListOfShape& aLC=pPF->Arguments();
aNb=aLC.Extent(); aNb=aLC.Extent();
if (aNb!=2) { if (aNb!=2) {
Sprintf (buf, " wrong number of arguments %s\n", aNb); di << " wrong number of arguments " << aNb << '\n';
di << buf;
return 0; return 0;
} }
// //

2
src/NCollection/NCollection_DynamicArray.hxx
View File

@ -240,7 +240,7 @@ public: //! @name public methods
} }
//! Assignment operator //! Assignment operator
NCollection_DynamicArray& operator= (NCollection_DynamicArray&& theOther) NCollection_DynamicArray& operator= (NCollection_DynamicArray&& theOther) noexcept
{ {
return Assign(std::forward<NCollection_DynamicArray>(theOther)); return Assign(std::forward<NCollection_DynamicArray>(theOther));
} }

2
src/NCollection/NCollection_UtfString.hxx
View File

@ -81,7 +81,7 @@ public:
NCollection_UtfString (const NCollection_UtfString& theCopy); NCollection_UtfString (const NCollection_UtfString& theCopy);
//! Move constructor //! Move constructor
NCollection_UtfString (NCollection_UtfString&& theOther); NCollection_UtfString (NCollection_UtfString&& theOther) noexcept;
//! Copy constructor from UTF-8 string. //! Copy constructor from UTF-8 string.
//! @param theCopyUtf8 UTF-8 string to copy //! @param theCopyUtf8 UTF-8 string to copy

2
src/NCollection/NCollection_UtfString.lxx
View File

@ -95,7 +95,7 @@ NCollection_UtfString<Type>::NCollection_UtfString (const NCollection_UtfString&
// purpose : // purpose :
// ======================================================================= // =======================================================================
template<typename Type> inline template<typename Type> inline
NCollection_UtfString<Type>::NCollection_UtfString (NCollection_UtfString&& theOther) NCollection_UtfString<Type>::NCollection_UtfString (NCollection_UtfString&& theOther) noexcept
: myString(theOther.myString), : myString(theOther.myString),
mySize (theOther.mySize), mySize (theOther.mySize),
myLength(theOther.myLength) myLength(theOther.myLength)