Fix some static analysis warnings in the shiboken tests

- else after return
- Do not repeat return types
- Use range-based for for std::map
- Use constructor member initialization
- Initialize variables
- Use override instead of repeating virtual
- Use noexcept for move special functions
- Upper case integer literals
- Various other small fixes

Pick-to: 6.6 6.5
Change-Id: I06924c60fcd0d8bfcad9cc2cd6e79e72621cb766
Reviewed-by: Shyamnath Premnadh <[email protected]>

1 files changed, 2 insertions, 2 deletions

diff --git a/sources/shiboken6/tests/libsample/pen.cpp b/sources/shiboken6/tests/libsample/pen.cpp
index 2d58fa54c..76473a264 100644
--- a/sources/shiboken6/tests/libsample/pen.cpp
+++ b/sources/shiboken6/tests/libsample/pen.cpp
@@ -59,9 +59,9 @@ Pen::Pen(const Pen &) : m_ctor(CopyCtor)
 {
 }
 
-Pen::Pen(Pen &&) = default;
+Pen::Pen(Pen &&) noexcept = default;
 Pen &Pen::operator=(const Pen &pen) = default;
-Pen &Pen::operator=(Pen &&) = default;
+Pen &Pen::operator=(Pen &&) noexcept = default;
 
 int Pen::ctorType()
 {