diff options
author | Markus Sauermann <6299227+Sauermann@users.noreply.github.com> | 2023-06-09 17:00:50 +0200 |
---|---|---|
committer | Markus Sauermann <6299227+Sauermann@users.noreply.github.com> | 2023-06-21 22:37:26 +0200 |
commit | 890fdd56df313fbf2035492f8e33fa2516ee23c5 (patch) | |
tree | fb3d904883d2cbca7e491e621712282bc12c219d /servers/rendering/renderer_rd/effects/ss_effects.cpp | |
parent | f2ce0b6741049b69efb455394b8a161f68a468a6 (diff) | |
download | redot-engine-890fdd56df313fbf2035492f8e33fa2516ee23c5.tar.gz |
Code simplifications
CPPcheck found most of them.
no need to assign the variable twice:
- AnimationTrackEditTypeAudio
- SSEffects
variable is assigned in all if-else clauses:
- EditorHelp
- AndroidInputHandler
- MenuBar
- ShaderCompiler
same if clause:
- ItemList
clearing an empty bitfield has no effect:
- Viewport
Diffstat (limited to 'servers/rendering/renderer_rd/effects/ss_effects.cpp')
-rw-r--r-- | servers/rendering/renderer_rd/effects/ss_effects.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/servers/rendering/renderer_rd/effects/ss_effects.cpp b/servers/rendering/renderer_rd/effects/ss_effects.cpp index 96e3683560..b9346c0201 100644 --- a/servers/rendering/renderer_rd/effects/ss_effects.cpp +++ b/servers/rendering/renderer_rd/effects/ss_effects.cpp @@ -77,10 +77,9 @@ SSEffects::SSEffects() { for (int pass = 0; pass < 4; pass++) { for (int subPass = 0; subPass < sub_pass_count; subPass++) { int a = pass; - int b = subPass; int spmap[5]{ 0, 1, 4, 3, 2 }; - b = spmap[subPass]; + int b = spmap[subPass]; float ca, sa; float angle0 = (float(a) + float(b) / float(sub_pass_count)) * Math_PI * 0.5f; |