fix gcc -Wmaybe-uninitialized warning

microsoft · Dec 5, 2023 · b62e46d · b62e46d
1 parent 7f0de8f
commit b62e46d
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/.ci/lint-cpp.sh b/.ci/lint-cpp.sh
@@ -30,7 +30,7 @@ get_omp_pragmas_without_num_threads() {
         --include='*.h' \
         --include='*.hpp' \
         'pragma omp parallel' \
-    | grep -v ' num_threads' \
+    | grep -v ' num_threads'
 }
 PROBLEMATIC_LINES=$(
     get_omp_pragmas_without_num_threads

diff --git a/R-package/tests/testthat/test_multithreading.R b/R-package/tests/testthat/test_multithreading.R
@@ -1,6 +1,6 @@
 test_that("getLGBMthreads() and setLGBMthreads() work as expected", {
     # works with integer input
-    setLGBMthreads(2L)
+    ret <- setLGBMthreads(2L)
     expect_null(ret)
     expect_equal(getLGBMthreads(), 2L)
 

diff --git a/include/LightGBM/utils/openmp_wrapper.h b/include/LightGBM/utils/openmp_wrapper.h
@@ -37,7 +37,7 @@ LIGHTGBM_EXTERN_C int LGBM_DEFAULT_NUM_THREADS;
       - https://gcc.gnu.org/onlinedocs/libgomp/omp_005fget_005fmax_005fthreads.html
 */
 inline int OMP_NUM_THREADS() {
-  int default_num_threads;
+  int default_num_threads = 1;
 
   if (LGBM_DEFAULT_NUM_THREADS > 0) {
     // if LightGBM-specific default has been set, ignore OpenMP-global config