Bug#40549 Use of non-standard C++ syntax should be removed from the code

 - Fix for 6.2

Author: Magnus Svensson

BUILD/SETUP.sh

@@ -61,7 +61,7 @@ prefix="/usr/local/mysql"
 just_print=
 just_configure=
 full_debug=
-warning_mode=
+warning_mode="pedantic"
 
 parse_options "$@"
 

storage/ndb/src/kernel/blocks/suma/Suma.hpp

@@ -200,7 +200,7 @@ class Suma : public SimulatedBlock {
       R_SUB_STOP_REQ,
       R_START_ME_REQ,
       R_API_FAIL_REQ,
-      R_SUB_ABORT_START_REQ,
+      R_SUB_ABORT_START_REQ
     };
 
     Uint32 m_opType;

storage/ndb/src/ndbapi/NdbInterpretedCode.cpp

@@ -836,7 +836,7 @@ NdbInterpretedCode::ret_sub()
   m_flags&= ~(InSubroutineDef);
 
   return add1(Interpreter::RETURN);
-};
+}
 
 /* Get a CodeMetaInfo object given a number
  * Label numbers start from 0.  Subroutine numbers start from

storage/ndb/src/ndbapi/NdbScanOperation.cpp

@@ -227,7 +227,7 @@ NdbScanOperation::addInterpretedCode(Uint32 aTC_ConnectPtr,
   theAI_LenInCurrAI= theCurrentATTRINFO->getLength();
 
   return res;
-};
+}
 
 /* Method for handling scanoptions passed into 
  * NdbTransaction::scanTable or scanIndex
@@ -969,7 +969,7 @@ NdbScanOperation::readTuples(NdbScanOperation::LockMode lm,
   m_savedBatchOldApi= batch;
 
   return 0;
-};
+}
 
 /* Most of the scan definition work for old + NdbRecord API scans is done here */
 int 
@@ -1350,7 +1350,7 @@ NdbScanOperation::nextResult(bool fetchAllowed, bool forceSend)
   return nextResult(&dummyOutRowPtr,
                     fetchAllowed,
                     forceSend);
-};
+}
 
 /* nextResult() for NdbRecord operation. */
 int