[Coastrnaut]

Shaders that name samplers through macros failed to parse:

• Preprocessor (Fed + Geiss - Color Pox Remix). hlslparser built macro values from the trimmed tokens, so #define smp sampler sampler_manyfish; got the value samplersampler_manyfish;, and it put every value in parentheses when substituting it, which is only valid for expressions. The expansion became (samplersampler_manyfish;). Values now keep the whitespace between their tokens (in all three places where they're built), and values containing a semicolon, i.e. statements and declarations, are inserted without parentheses. Expression macros are still wrapped as before.
• Samplers named via macros ([dylan] cube in a room …, 2 presets). #define base01 main, sampler base01; and tex2D(base01, uv) turn into sampler_fw_main in the preprocessor, but projectM only searched the original code for sampler_ names, so it didn't declare it. GetReferencedSamplers() now searches the macro-expanded code as well.

Results

• The 3 presets compile now. With all open shader PRs combined, the only presets in the four packs below whose shaders still fall back are the two shifter + flexi - liquid circuitry … ones: their composite shader has an extra } left over from a commented-out loop, so it doesn't compile in Milkdrop either.
• For all other presets, including every preset that uses #define, shader compile/link results and rendered frames are identical to master.
• New HLSLPreprocessorTest. All unit tests pass.

How I tested it

I loaded all 14,536 presets from cream-of-the-crop, en-d, milkdrop-original and projectm-classic one after another (Windows, RTX A6000, GL 3.3 core and GLES contexts), with and without the preset folders as texture search path, rendered frames with fixed audio and frame times, and recorded shader compile/link results, failed loads, logged shader fallbacks and frame hashes. For the pixel comparison both builds had their random seeds fixed (a local patch only, identical on both sides). The presets this PR fixes are excluded from the pixel comparison, since loading them changes shared state (e.g. the expression evaluator's random generator) for every preset after them.

Retested on current master (2026-11-12)

• Unit tests: 168/168 libprojectM and 98/98 playlist tests pass.
• Presets: I loaded all 14,536 presets from cream-of-the-crop, en-d, milkdrop-original and projectm-classic one after another with this branch and with master (GL 3.3 core, RTX A6000, preset folders as texture search path, 30 frames each with fixed audio, frame times and random seeds; the seeds are a local patch only, identical on both sides). The only compile/link changes are the 3 presets whose shaders now compile; no preset gets worse. All frames before the first of them are identical to master; later ones differ because each preset continues from the state the previous one left behind.
• Together with my other PRs: All my open PRs (Fix calling convention of glGetStringi pointer in GLProbe #1059 to Build for Windows Store (UWP) without app-side workarounds #1065, Return int from the NaN-safe multiplication of two ints #1067 to Fix shaders that declare or name samplers through macros #1072, Build Apple frameworks for iOS, tvOS, watchOS and visionOS, and support EAGL contexts #1075, Don't shift the blur textures by Direct3D's half-pixel offset #1078 and Render with 10 bits per color channel if the target framebuffer has more than 8 #1079) merge with each other and with master without conflicts. Merged together (all except Render with 10 bits per color channel if the target framebuffer has more than 8 #1079, which I opened after this run), they build on desktop GL and GLES, all unit tests pass, and the corpus gives 0 failed loads, 0 shader compile/link failures and 0 GL errors in an OpenGL ES context, with and without textures.