Skip to content

fix error C2589#2

Open
shimenga wants to merge 2 commits into
Pondot:mainfrom
shimenga:main
Open

fix error C2589#2
shimenga wants to merge 2 commits into
Pondot:mainfrom
shimenga:main

Conversation

@shimenga

Copy link
Copy Markdown
  • backdrop.cpp / glass.cpp: expand compact single-line D3D11 state-creation blocks (buffer/sampler/blend/rasterizer/depth-stencil) into multi-line, properly indented braces for readability.
  • surfaces.cpp / glass.cpp: add #undef max / #undef min to prevent the Windows macros from interfering with std::numeric_limits and template uses.
  • surfaces.cpp: expand compact loops and inline branches in RoundedGrad, ImgOrGrad and SurfaceLaunchpad into clearer multi-line form.

No runtime behavior changes; whitespace/brace reformatting plus the min/max undef guard.

Summary

Applied Visual Studio code formatting to reorganize the affected files, expanding compact blocks into multi-line structures for improved readability. Also added #undef max / #undef min preprocessor directives to fix MSVC error C2589 caused by Windows macros conflicting with std::numeric_limits.

Testing

Full project build completed successfully with no compilation errors.

- backdrop.cpp / glass.cpp: expand compact single-line D3D11 state-creation
  blocks (buffer/sampler/blend/rasterizer/depth-stencil) into multi-line,
  properly indented braces for readability.
- surfaces.cpp / glass.cpp: add `#undef max` / `#undef min` to prevent the
  Windows macros from interfering with std::numeric_limits and template uses.
- surfaces.cpp: expand compact loops and inline branches in RoundedGrad,
  ImgOrGrad and SurfaceLaunchpad into clearer multi-line form.

No runtime behavior changes; whitespace/brace reformatting plus the min/max
undef guard.
@shimenga

Copy link
Copy Markdown
Author

After downloading, the code wouldn't compile, so I made modifications to fix error C2589.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant