Improve the Bazel build file

Author: vbaderks

.gitignore

@@ -1,11 +1,11 @@
-# Copyright (c) Team CharLS.
+# SPDX-FileCopyrightText: © 2014 Team CharLS
 # SPDX-License-Identifier: BSD-3-Clause
 
 .vscode/
 .vs/
 .idea
 build*/
-bazel-*/
+bazel-*
 MODULE.bazel.lock
 cmake-build-debug/
 TestResults/

BUILD.bazel

@@ -0,0 +1,84 @@
+# SPDX-FileCopyrightText: © 2025 Team CharLS
+# SPDX-License-Identifier: BSD-3-Clause
+
+cc_library(
+    name = "charls",
+    srcs = [
+        "src/charls_jpegls_decoder.cpp",
+        "src/charls_jpegls_encoder.cpp",
+        "src/coding_parameters.hpp",
+        "src/color_transform.hpp",
+        "src/conditional_static_cast.hpp",
+        "src/constants.hpp",
+        "src/copy_from_line_buffer.hpp",
+        "src/copy_to_line_buffer.hpp",
+        "src/default_traits.hpp",
+        "src/golomb_lut.cpp",
+        "src/golomb_lut.hpp",
+        "src/jpeg_marker_code.hpp",
+        "src/jpeg_stream_reader.cpp",
+        "src/jpeg_stream_reader.hpp",
+        "src/jpeg_stream_writer.cpp",
+        "src/jpeg_stream_writer.hpp",
+        "src/jpegls_algorithm.hpp",
+        "src/jpegls_error.cpp",
+        "src/jpegls_preset_coding_parameters.hpp",
+        "src/jpegls_preset_parameters_type.hpp",
+        "src/lossless_traits.hpp",
+        "src/make_scan_codec.cpp",
+        "src/make_scan_codec.hpp",
+        "src/pch.hpp",
+        "src/quantization_lut.cpp",
+        "src/quantization_lut.hpp",
+        "src/regular_mode_context.hpp",
+        "src/run_mode_context.hpp",
+        "src/scan_codec.hpp",
+        "src/scan_decoder.hpp",
+        "src/scan_decoder_impl.hpp",
+        "src/scan_encoder.hpp",
+        "src/scan_encoder_impl.hpp",
+        "src/span.hpp",
+        "src/util.hpp",
+        "src/validate_spiff_header.cpp",
+        "src/version.cpp",
+    ],
+    hdrs = [
+        "include/charls/annotations.h",
+        "include/charls/api_abi.h",
+        "include/charls/charls.h",
+        "include/charls/charls.hpp",
+        "include/charls/charls_jpegls_decoder.h",
+        "include/charls/charls_jpegls_encoder.h",
+        "include/charls/jpegls_decoder.hpp",
+        "include/charls/jpegls_encoder.hpp",
+        "include/charls/jpegls_error.h",
+        "include/charls/jpegls_error.hpp",
+        "include/charls/public_types.h",
+        "include/charls/undef_macros.h",
+        "include/charls/validate_spiff_header.h",
+        "include/charls/version.h",
+    ],
+    strip_include_prefix = "include",
+    visibility = ["//visibility:public"],
+)
+
+cc_binary(
+    name = "charls-test",
+    srcs = [
+        "src/default_traits.hpp",
+        "src/lossless_traits.hpp",
+        "test/bit_stream_damage.cpp",
+        "test/bit_stream_damage.hpp",
+        "test/compliance.cpp",
+        "test/compliance.hpp",
+        "test/main.cpp",
+        "test/performance.cpp",
+        "test/performance.hpp",
+        "test/portable_anymap_file.hpp",
+        "test/util.cpp",
+        "test/util.hpp",
+    ],
+    deps = [
+        "//:charls",
+    ],
+)

CHANGELOG.md

@@ -151,7 +151,7 @@ This has been done to make it possible to have different release cycles.
 ### Removed
 
 - The legacy methods JpegLsEncodeStream, JpegLsDecodeStream and JpegLsReadHeaderStream have been removed as exported methods.
-  These methods were not part of the public API and only used by by the charlstest application
+  These methods were not part of the public API and only used by by the charls-test application
 
 ### Fixed
 
@@ -218,7 +218,7 @@ This has been done to make it possible to have different release cycles.
 
 - Fixes [#10](https://github.com/team-charls/charls/issues/10), Fixed the problem that "output buffer to small" was not
   detected when writing encoded bytes to a fixed output buffer. This could cause memory corruption problems
-- Fixes [11](https://github.com/team-charls/charls/issues/11), Update charlstest to return EXIT_SUCCESS/FAILURE
+- Fixes [11](https://github.com/team-charls/charls/issues/11), Update charls-test to return EXIT_SUCCESS/FAILURE
 - Fixed the issue that DecodeToPnm would set params.colorTransform = ColorTransformation::BigEndian but the library didn’t support this option during decoding
 
 ## [1.1.0] - 2016-5-15

CMakeLists.txt

@@ -1,4 +1,4 @@
-# Copyright (c) Team CharLS.
+# SPDX-FileCopyrightText: © 2009 Team CharLS
 # SPDX-License-Identifier: BSD-3-Clause
 
 cmake_minimum_required(VERSION 3.16...3.31)
@@ -228,10 +228,10 @@ if(CHARLS_BUILD_TESTS)
   enable_testing()
   add_subdirectory(test)
 
-  # The unit tests project depends on the VS C++ unit test framework. Use the -unittest option of the charlstest tool for other platforms.
+  # The unit tests project depends on the VS C++ unit test framework. Use the -unittest option of the charls-test tool for other platforms.
   add_test(
     NAME basic-test
-    COMMAND charlstest -unittest
+    COMMAND charls-test -unittest
     WORKING_DIRECTORY ${CMAKE_CURRENT_LIST_DIR}
   )
 endif()

CharLS.sln.DotSettings

@@ -56,7 +56,6 @@
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=bugprone/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=byteswap/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=CHARLS/@EntryIndexedValue">True</s:Boolean>
-	<s:Boolean x:Key="/Default/UserDictionary/Words/=charlstest/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=cmove/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=cmyk/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=comparepnm/@EntryIndexedValue">True</s:Boolean>

MODULE.bazel

@@ -1,4 +1,3 @@
-
 # SPDX-FileCopyrightText: © 2025 Team CharLS
 # SPDX-License-Identifier: BSD-3-Clause
 
@@ -9,8 +8,8 @@ This MODULE.bazel file defines the module and its dependencies for the CharLS pr
 module(
     name = "charls",
     version = "3.0.0",
+    bazel_compatibility = [">=7.5.0"],
     compatibility_level = 3,
-    bazel_compatibility = [">=7.5.0"]
 )
 
-bazel_dep(name = "rules_cc", version = "0.1.1")
+bazel_dep(name = "rules_cc", version = "0.1.1")

src/BUILD.bazel

@@ -1,26 +1,27 @@
-# Copyright (c) Team CharLS.
+# SPDX-FileCopyrightText: © 2009 Team CharLS
 # SPDX-License-Identifier: BSD-3-Clause
 
-add_executable(charlstest "")
+add_executable(charls-test "")
 
-target_sources(charlstest
+target_sources(charls-test
   PRIVATE
-    bitstreamdamage.cpp
-    bitstreamdamage.hpp
+    bit_stream_damage.cpp
+    bit_stream_damage.hpp
     compliance.cpp
     compliance.hpp
     main.cpp
     performance.cpp
     performance.hpp
     util.cpp
     util.hpp
+    portable_anymap_file.hpp
 )
 
 if(WIN32 AND MSVC_VERSION GREATER_EQUAL 1920)
   # Only add the manifest file when building a Windows app
-  target_sources(charlstest PRIVATE app.manifest)
+  target_sources(charls-test PRIVATE app.manifest)
 endif()
 
-set_target_properties(charlstest PROPERTIES CXX_VISIBILITY_PRESET hidden)
+set_target_properties(charls-test PROPERTIES CXX_VISIBILITY_PRESET hidden)
 
-target_link_libraries(charlstest PRIVATE charls)
+target_link_libraries(charls-test PRIVATE charls)

test/CMakeLists.txt

@@ -174,14 +174,14 @@
     </Link>
   </ItemDefinitionGroup>
   <ItemGroup>
-    <ClCompile Include="bitstreamdamage.cpp" />
+    <ClCompile Include="bit_stream_damage.cpp" />
     <ClCompile Include="compliance.cpp" />
     <ClCompile Include="main.cpp" />
     <ClCompile Include="performance.cpp" />
     <ClCompile Include="util.cpp" />
   </ItemGroup>
   <ItemGroup>
-    <ClInclude Include="bitstreamdamage.hpp" />
+    <ClInclude Include="bit_stream_damage.hpp" />
     <ClInclude Include="compliance.hpp" />
     <ClInclude Include="portable_anymap_file.hpp" />
     <ClInclude Include="performance.hpp" />

test/CharLSTest.vcxproj

@@ -14,7 +14,7 @@
     </Filter>
   </ItemGroup>
   <ItemGroup>
-    <ClCompile Include="bitstreamdamage.cpp">
+    <ClCompile Include="bit_stream_damage.cpp">
       <Filter>Source Files</Filter>
     </ClCompile>
     <ClCompile Include="compliance.cpp">
@@ -34,7 +34,7 @@
     <ClInclude Include="util.hpp">
       <Filter>Header Files</Filter>
     </ClInclude>
-    <ClInclude Include="bitstreamdamage.hpp">
+    <ClInclude Include="bit_stream_damage.hpp">
       <Filter>Header Files</Filter>
     </ClInclude>
     <ClInclude Include="compliance.hpp">

test/CharLSTest.vcxproj.filters

@@ -1,7 +1,7 @@
 // Copyright (c) Team CharLS.
 // SPDX-License-Identifier: BSD-3-Clause
 
-#include "bitstreamdamage.hpp"
+#include "bit_stream_damage.hpp"
 #include "util.hpp"
 
 #include <iostream>

test/bitstreamdamage.cpp test/bit_stream_damage.cpp

@@ -1,10 +1,10 @@
-// Copyright (c) Team CharLS.
+// SPDX-FileCopyrightText: © 2016 Team CharLS
 // SPDX-License-Identifier: BSD-3-Clause
 
 #include "../src/default_traits.hpp"
 #include "../src/lossless_traits.hpp"
 
-#include "bitstreamdamage.hpp"
+#include "bit_stream_damage.hpp"
 #include "compliance.hpp"
 #include "performance.hpp"
 #include "util.hpp"