!5477 [MD] Remove two -wno options

Merge pull request !5477 from xiefangqi/xfq_fix_wno_option

mindspore/ccsrc/minddata/dataset/CMakeLists.txt

@@ -1,8 +1,6 @@
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-sign-compare")
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-reorder")
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-switch")
-set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-sequence-point")
-set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-unused-variable")
 
 if (${CMAKE_SYSTEM_NAME} MATCHES "Darwin")
     set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-uninitialized")

mindspore/ccsrc/minddata/dataset/engine/datasetops/batch_op.cc

@@ -101,8 +101,9 @@ Status BatchOp::operator()() {
       table->emplace_back(new_row);
       // if # of rows is enough to make 1 batch (1 batch is buffer), send it to worker_queue
       if (table->size() == static_cast<size_t>(cur_batch_size)) {
-        RETURN_IF_NOT_OK(worker_queues_[cnt++ % num_workers_]->EmplaceBack(
-          std::make_pair(std::move(table), CBatchInfo(epoch_num, batch_num++, cnt - epoch_num))));
+        RETURN_IF_NOT_OK(worker_queues_[cnt % num_workers_]->EmplaceBack(
+          std::make_pair(std::move(table), CBatchInfo(epoch_num, batch_num++, cnt + 1 - epoch_num))));
+        cnt++;
         table = std::make_unique<TensorQTable>();
         RETURN_IF_NOT_OK(GetBatchSize(&cur_batch_size, CBatchInfo(epoch_num, batch_num, cnt - epoch_num)));
       }
@@ -110,8 +111,9 @@ Status BatchOp::operator()() {
     }
     // Reminder logic, execute only when there is a remainder (table is non empty) and don't drop
     if (drop_ == false && table->empty() == false) {
-      RETURN_IF_NOT_OK(worker_queues_[cnt++ % num_workers_]->EmplaceBack(
-        std::make_pair(std::move(table), CBatchInfo(epoch_num, batch_num++, cnt - epoch_num))));
+      RETURN_IF_NOT_OK(worker_queues_[cnt % num_workers_]->EmplaceBack(
+        std::make_pair(std::move(table), CBatchInfo(epoch_num, batch_num++, cnt + 1 - epoch_num))));
+      cnt++;
     }
     table = std::make_unique<TensorQTable>();  // this drops when drop == true
     // end of the current epoch, batch_num should start from 0 again

mindspore/ccsrc/minddata/dataset/kernels/image/bounding_box.cc

@@ -120,7 +120,6 @@ Status BoundingBox::UpdateBBoxesForCrop(TensorPtr *bbox_list, size_t *bbox_count
   // Also PAss X/Y Min/Max of image cropped region - normally obtained from 'GetCropBox' functions
   std::vector<dsize_t> correct_ind;
   std::vector<bbox_float> copyVals;
-  bool retFlag = false;  // true unless overlap found
   dsize_t bboxDim = (*bbox_list)->shape()[1];
   for (dsize_t i = 0; i < *bbox_count; i++) {
     std::shared_ptr<BoundingBox> bbox;