Code analysis fixes: Refactor code for consistency and readability, i…

[peter-lawrey]

This patch cleans up Java Thread Affinity source and test code after a static-analysis sweep.
It does not change runtime behaviour; all modifications are cosmetic or defensive.

Area	Examples	What changed
Logging	LinuxJNAAffinity, FileLockBasedLockChecker, many others	• Replaced string-concat ("msg "+val) with SLF4J parameterised messages ("msg {}", val).
Assertions & validations	AffinityLock, LockInventory, tests	• Added explicit IllegalArgumentException checks for invalid cpuId. • Converted assertTrue(!x) to assertFalse(x) for clarity. • Removed dead code comments.
Lambda / SAM conversions	AffinityThreadFactory, several tests	• Replaced anonymous classes with lambdas & method refs.
Type & null safety	BootClassPath, VersionHelper, VanillaCpuLayout	• Added @NotNull, StandardCharsets.UTF_8, and .isEmpty() idioms.
Formatting & whitespace	Many files	• Consistent spacing, braces, and indentation.
Suppression & warnings	MicroJitterSampler, JNIClock, tests	• Added targeted @SuppressWarnings and IDE hints (//noinspection …).
Minor refactors	OSGiBundleTest, OSGiTestBase	• Simplified temp variables; removed redundant initialisations.

Motivation

Improving code readability and log consistency helps:

• downstream users reading the source,
• future maintainers running static-analysis tools,
• IDE inspections (fewer warnings).