Add some issues that are triggered if the module is visible

Author: alejandro-alvarez-sonarsource

CMakeLists.txt

@@ -1,22 +1,15 @@
 cmake_minimum_required(VERSION 3.30)
 
-# `import std;` is not supported yet by libstdc++
-set(CMAKE_CXX_COMPILER clang++)
-set(CMAKE_C_COMPILER clang)
-
 project(sonar_scanner_example LANGUAGES CXX)
 
 set(CMAKE_CXX_STANDARD 20)
 set(CMAKE_CXX_STANDARD_REQUIRED ON)
 set(CMAKE_CXX_EXTENSIONS OFF)
-set(CMAKE_CXX_MODULE_STD 1)
-set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wextra -Werror -Wpedantic")
+set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall")
 
 set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
 
 add_executable(sonar_scanner_example src/main.cpp)
-target_compile_features(sonar_scanner_example
-        INTERFACE cxx_std_20)
 target_sources(sonar_scanner_example
         PRIVATE
         FILE_SET CXX_MODULES FILES src/args.cppm

src/args.cppm

@@ -1 +1,40 @@
+module;
+#include <iostream>
+#include <string_view>
+#include <variant>
+
 export module args;
+
+export namespace args {
+
+enum class Error {
+  Ok,
+  TooLong,
+  TooManyArgs,
+  NullPtr,
+};
+
+std::variant<std::string_view, Error> process_args(int argc, char *argv[]) {
+  int num = argc - 1;
+
+  if (num == 0) {
+    std::cout << "No arguments provided\n";
+  } else if (num == 0) { // intentional mistake
+    std::cout << "1 argument provided\n";
+  } else if (num == 2) {
+    std::cout << "2 arguments provided\n";
+  } else {
+    std::cout << num << " arguments provided\n";
+  }
+  if (argv != 0) {
+    std::cout << "argv not null\n";
+    ; // intentional extra-semicolon
+  }
+
+  if (argv == nullptr) {
+    return std::string_view(*argv); // intentional nullptr dereference
+  }
+
+  return std::string_view(argv[0]);
+}
+} // namespace args

src/main.cpp

@@ -1,26 +1,22 @@
 #include <iostream>
+#include <variant>
 
-using namespace std;
+import args;
 
-int main(int argc, char* argv[]) {
-  int num = argc - 1;
+using namespace std;
 
-  if (num == 0) {
-    cout << "No arguments provided\n";
-  } else if (num == 0) { // intentional mistake
-    cout << "1 argument provided\n";
-  } else if (num == 2) {
-    cout << "2 arguments provided\n";
-  } else {
-    cout << num << " arguments provided\n";
-  }
-  if (argv != 0) {
-    cout << "argv not null\n";; // intentional extra-semicolon
-  }
-  if (argv == nullptr) {
-    return **argv; // intentional nullptr dereference
+int main(int argc, char *argv[]) {
+  auto get_proc_name = args::process_args(argc, argv);
+  if (std::holds_alternative<args::Error>(get_proc_name)) {
+    switch (std::get<args::Error>(get_proc_name)) {
+    case args::Error::TooLong:
+      std::cout << "Proc name too long\n";
+      return 1;
+    }
+    return 0;
   }
 
+  auto &&value = std::get<std::string_view>(get_proc_name);
+  std::cout << value << '\n';
   return 0;
 }
-