pipeline c++ lint and remove deprecated APIs

[star-e]

Remove unnecessary inline, const_cast.
Add IWYU comments.
Remove deprecated APIs in the pipeline interface.

Changelog

---

Continuous Integration

This pull request:

• needs automatic test cases check.

  Manual trigger with @cocos-robot run test cases afterward.

• does not change any runtime related code or build configuration

  If any reviewer thinks the CI checks are needed, please uncheck this option, then close and reopen the issue.

---

Compatibility Check

This pull request:

• changes public API, and have ensured backward compatibility with deprecated features.
• affects platform compatibility, e.g. system version, browser version, platform sdk version, platform toolchain, language version, hardware compatibility etc.
• affects file structure of the build package or build configuration which requires user project upgrade.
• introduces breaking changes, please list all changes, affected features and the scope of violation.

[github-actions]

Interface Check Report

! WARNING this pull request has changed these public interfaces:

@@ -36070,16 +36070,8 @@
              * @param texture @en readonly texture @zh 只读的贴图
              */
             setTexture(name: string, texture: gfx.Texture): void;
             /**
-             * @deprecated Method will be removed in the future
-             */
-            setReadWriteBuffer(name: string, buffer: gfx.Buffer): void;
-            /**
-             * @deprecated Method will be removed in the future
-             */
-            setReadWriteTexture(name: string, texture: gfx.Texture): void;
-            /**
              * @en Set sampler descriptor.
              * Type of the sampler should match the one in shader.
              * @zh 设置采样器描述符。类型需要与着色器中的一致。
              * 不匹配会引起未定义行为。
@@ -36094,15 +36086,8 @@
              * @param camera @en The camera instance to be set. @zh 当前相机
              */
             setBuiltinCameraConstants(camera: renderer.scene.Camera): void;
             /**
-             * @deprecated Method will be removed in the future
-             * @en Same as setBuiltinDirectionalLightConstants
-             * @zh 同setBuiltinDirectionalLightConstants
-             * @param light @en The main light. @zh 主光
-             */
-            setBuiltinShadowMapConstants(light: renderer.scene.DirectionalLight): void;
-            /**
              * @en Set builtin directional light and shadow constants.
              * For list of constants, please check CCShadow in cc-shadow.chunk and CCCamera in cc-global.chunk.
              * @zh 设置内置方向光与阴影常量。
              * 具体常量见cc-shadow.chunk中的CCShadow与cc-global.chunk中的CCCamera。

[minggo]

Not all compilers support IWYU pragma.

[star-e]

Shall I remove them? If vscode is used and clang-tidy is enabled with latest llvm, it will show warnings in the IDE.

[minggo]

Can i know what's the warning message?

[star-e]

Included header RenderGraphFwd.h is not used directly (fix available)clangd[unused-includes](https://clangd.llvm.org/guides/include-cleaner) like this.

[minggo]

It is a good feature. So this modification just avoid exporting the following forward declaration?

[star-e]

CustomFwd.h depends on RenderGraphFwd.h.

If user included CustomFwd.h, generally user will also include RenderGraphFwd.h.

So RenderGraphFwd.h is included in CustomFwd.h to export dependent symbols.

However, since no symbol in RenderGraphFwd.h is used in CustomFwd.h, clangd treats it as a warning.

IWYU pragma: begin_exports is added to specify RenderGraphFwd.h as export. The warning is disabled.

[minggo]

Got it. I think keep it is no problem as it just fix warnings.