Fixed the code stripper stripping too much

Chman · Chman · commit f208bd85c1d8 · 2019-03-26T12:58:32.000+01:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,11 @@ All notable changes to this package will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## [2.2.0] - 2019-03-26
+
+### Fixed
+- Post-processing would crash if "Managed Stripping Level" was set to Medium or High.
+
 ## [2.1.5] - 2019-03-25
 
 ### Fixed
diff --git a/PostProcessing/Runtime/Effects/AmbientOcclusion.cs b/PostProcessing/Runtime/Effects/AmbientOcclusion.cs
@@ -207,6 +207,9 @@ internal interface IAmbientOcclusionMethod
         void Release();
     }
 
+#if UNITY_2017_1_OR_NEWER
+    [UnityEngine.Scripting.Preserve]
+#endif
     internal sealed class AmbientOcclusionRenderer : PostProcessEffectRenderer<AmbientOcclusion>
     {
         IAmbientOcclusionMethod[] m_Methods;
diff --git a/PostProcessing/Runtime/Effects/AutoExposure.cs b/PostProcessing/Runtime/Effects/AutoExposure.cs
@@ -87,6 +87,9 @@ public override bool IsEnabledAndSupported(PostProcessRenderContext context)
         }
     }
 
+#if UNITY_2017_1_OR_NEWER
+    [UnityEngine.Scripting.Preserve]
+#endif
     internal sealed class AutoExposureRenderer : PostProcessEffectRenderer<AutoExposure>
     {
         const int k_NumEyes = 2;
diff --git a/PostProcessing/Runtime/Effects/Bloom.cs b/PostProcessing/Runtime/Effects/Bloom.cs
@@ -93,6 +93,9 @@ public override bool IsEnabledAndSupported(PostProcessRenderContext context)
         }
     }
 
+#if UNITY_2017_1_OR_NEWER
+    [UnityEngine.Scripting.Preserve]
+#endif
     internal sealed class BloomRenderer : PostProcessEffectRenderer<Bloom>
     {
         enum Pass
diff --git a/PostProcessing/Runtime/Effects/ChromaticAberration.cs b/PostProcessing/Runtime/Effects/ChromaticAberration.cs
@@ -37,6 +37,9 @@ public override bool IsEnabledAndSupported(PostProcessRenderContext context)
         }
     }
 
+#if UNITY_2017_1_OR_NEWER
+    [UnityEngine.Scripting.Preserve]
+#endif
     internal sealed class ChromaticAberrationRenderer : PostProcessEffectRenderer<ChromaticAberration>
     {
         Texture2D m_InternalSpectralLut;
diff --git a/PostProcessing/Runtime/Effects/ColorGrading.cs b/PostProcessing/Runtime/Effects/ColorGrading.cs
@@ -387,6 +387,9 @@ public override bool IsEnabledAndSupported(PostProcessRenderContext context)
         }
     }
 
+#if UNITY_2017_1_OR_NEWER
+    [UnityEngine.Scripting.Preserve]
+#endif
     internal sealed class ColorGradingRenderer : PostProcessEffectRenderer<ColorGrading>
     {
         enum Pass
diff --git a/PostProcessing/Runtime/Effects/DepthOfField.cs b/PostProcessing/Runtime/Effects/DepthOfField.cs
@@ -77,6 +77,9 @@ public override bool IsEnabledAndSupported(PostProcessRenderContext context)
         }
     }
 
+#if UNITY_2017_1_OR_NEWER
+    [UnityEngine.Scripting.Preserve]
+#endif
     // TODO: Doesn't play nice with alpha propagation, see if it can be fixed without killing performances
     internal sealed class DepthOfFieldRenderer : PostProcessEffectRenderer<DepthOfField>
     {
diff --git a/PostProcessing/Runtime/Effects/Dithering.cs b/PostProcessing/Runtime/Effects/Dithering.cs
@@ -3,6 +3,9 @@
 
 namespace UnityEngine.Rendering.PostProcessing
 {
+#if UNITY_2017_1_OR_NEWER
+    [UnityEngine.Scripting.Preserve]
+#endif
     [Serializable]
     internal sealed class Dithering
     {
diff --git a/PostProcessing/Runtime/Effects/FastApproximateAntialiasing.cs b/PostProcessing/Runtime/Effects/FastApproximateAntialiasing.cs
@@ -6,6 +6,9 @@ namespace UnityEngine.Rendering.PostProcessing
     /// <summary>
     /// This class holds settings for the Fast Approximate Anti-aliasing (FXAA) effect.
     /// </summary>
+#if UNITY_2017_1_OR_NEWER
+    [UnityEngine.Scripting.Preserve]
+#endif
     [Serializable]
     public sealed class FastApproximateAntialiasing
     {
diff --git a/PostProcessing/Runtime/Effects/Fog.cs b/PostProcessing/Runtime/Effects/Fog.cs
@@ -5,6 +5,9 @@ namespace UnityEngine.Rendering.PostProcessing
     /// <summary>
     /// This class holds settings for the Fog effect with the deferred rendering path.
     /// </summary>
+#if UNITY_2017_1_OR_NEWER
+    [UnityEngine.Scripting.Preserve]
+#endif
     [Serializable]
     public sealed class Fog
     {
diff --git a/PostProcessing/Runtime/Effects/Grain.cs b/PostProcessing/Runtime/Effects/Grain.cs
@@ -44,7 +44,9 @@ public override bool IsEnabledAndSupported(PostProcessRenderContext context)
 #if POSTFX_DEBUG_STATIC_GRAIN
     #pragma warning disable 414
 #endif
-
+#if UNITY_2017_1_OR_NEWER
+    [UnityEngine.Scripting.Preserve]
+#endif
     internal sealed class GrainRenderer : PostProcessEffectRenderer<Grain>
     {
         RenderTexture m_GrainLookupRT;
diff --git a/PostProcessing/Runtime/Effects/LensDistortion.cs b/PostProcessing/Runtime/Effects/LensDistortion.cs
@@ -57,6 +57,9 @@ public override bool IsEnabledAndSupported(PostProcessRenderContext context)
         }
     }
 
+#if UNITY_2017_1_OR_NEWER
+    [UnityEngine.Scripting.Preserve]
+#endif
     internal sealed class LensDistortionRenderer : PostProcessEffectRenderer<LensDistortion>
     {
         public override void Render(PostProcessRenderContext context)
diff --git a/PostProcessing/Runtime/Effects/MotionBlur.cs b/PostProcessing/Runtime/Effects/MotionBlur.cs
@@ -38,6 +38,9 @@ public override bool IsEnabledAndSupported(PostProcessRenderContext context)
         }
     }
 
+#if UNITY_2017_1_OR_NEWER
+    [UnityEngine.Scripting.Preserve]
+#endif
     internal sealed class MotionBlurRenderer : PostProcessEffectRenderer<MotionBlur>
     {
         enum Pass
diff --git a/PostProcessing/Runtime/Effects/MultiScaleVO.cs b/PostProcessing/Runtime/Effects/MultiScaleVO.cs
@@ -6,6 +6,7 @@ namespace UnityEngine.Rendering.PostProcessing
     // TODO: Fix VR support
 
 #if UNITY_2017_1_OR_NEWER
+    [UnityEngine.Scripting.Preserve]
     [Serializable]
     internal sealed class MultiScaleVO : IAmbientOcclusionMethod
     {
diff --git a/PostProcessing/Runtime/Effects/ScalableAO.cs b/PostProcessing/Runtime/Effects/ScalableAO.cs
@@ -3,6 +3,9 @@
 namespace UnityEngine.Rendering.PostProcessing
 {
     // Scalable ambient obscurance
+#if UNITY_2017_1_OR_NEWER
+    [UnityEngine.Scripting.Preserve]
+#endif
     [Serializable]
     internal sealed class ScalableAO : IAmbientOcclusionMethod
     {
diff --git a/PostProcessing/Runtime/Effects/ScreenSpaceReflections.cs b/PostProcessing/Runtime/Effects/ScreenSpaceReflections.cs
@@ -111,6 +111,9 @@ public override bool IsEnabledAndSupported(PostProcessRenderContext context)
         }
     }
 
+#if UNITY_2017_1_OR_NEWER
+    [UnityEngine.Scripting.Preserve]
+#endif
     internal sealed class ScreenSpaceReflectionsRenderer : PostProcessEffectRenderer<ScreenSpaceReflections>
     {
         RenderTexture m_Resolve;
diff --git a/PostProcessing/Runtime/Effects/SubpixelMorphologicalAntialiasing.cs b/PostProcessing/Runtime/Effects/SubpixelMorphologicalAntialiasing.cs
@@ -5,6 +5,9 @@ namespace UnityEngine.Rendering.PostProcessing
     /// <summary>
     /// This class holds settings for the Subpixel Morphological Anti-aliasing (SMAA) effect.
     /// </summary>
+#if UNITY_2017_1_OR_NEWER
+    [UnityEngine.Scripting.Preserve]
+#endif
     [Serializable]
     public sealed class SubpixelMorphologicalAntialiasing
     {
diff --git a/PostProcessing/Runtime/Effects/TemporalAntialiasing.cs b/PostProcessing/Runtime/Effects/TemporalAntialiasing.cs
@@ -5,6 +5,9 @@ namespace UnityEngine.Rendering.PostProcessing
     /// <summary>
     /// This class holds settings for the Temporal Anti-aliasing (TAA) effect.
     /// </summary>
+#if UNITY_2017_1_OR_NEWER
+    [UnityEngine.Scripting.Preserve]
+#endif
     [Serializable]
     public sealed class TemporalAntialiasing
     {
diff --git a/PostProcessing/Runtime/Effects/Vignette.cs b/PostProcessing/Runtime/Effects/Vignette.cs
@@ -94,6 +94,9 @@ public override bool IsEnabledAndSupported(PostProcessRenderContext context)
         }
     }
 
+#if UNITY_2017_1_OR_NEWER
+    [UnityEngine.Scripting.Preserve]
+#endif
     internal sealed class VignetteRenderer : PostProcessEffectRenderer<Vignette>
     {
         public override void Render(PostProcessRenderContext context)
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
     "name": "com.unity.postprocessing",
-    "version": "2.1.5",
+    "version": "2.2.0",
     "displayName": "Post Processing",
     "unity": "2018.1",
     "description": "The post-processing stack (v2) comes with a collection of effects and image filters you can apply to your cameras to improve the visuals of your games.",

Original file line number	Diff line number	Diff line change
`@@ -207,6 +207,9 @@ internal interface IAmbientOcclusionMethod`
`207`	`207`	`void Release();`
`208`	`208`	`}`
`209`	`209`
	`210`	`+#if UNITY_2017_1_OR_NEWER`
	`211`	`+ [UnityEngine.Scripting.Preserve]`
	`212`	`+#endif`
`210`	`213`	`internal sealed class AmbientOcclusionRenderer : PostProcessEffectRenderer<AmbientOcclusion>`
`211`	`214`	`{`
`212`	`215`	`IAmbientOcclusionMethod[] m_Methods;`
Original file line number	Diff line number	Diff line change
`@@ -87,6 +87,9 @@ public override bool IsEnabledAndSupported(PostProcessRenderContext context)`
`87`	`87`	`}`
`88`	`88`	`}`
`89`	`89`
	`90`	`+#if UNITY_2017_1_OR_NEWER`
	`91`	`+ [UnityEngine.Scripting.Preserve]`
	`92`	`+#endif`
`90`	`93`	`internal sealed class AutoExposureRenderer : PostProcessEffectRenderer<AutoExposure>`
`91`	`94`	`{`
`92`	`95`	`const int k_NumEyes = 2;`
Original file line number	Diff line number	Diff line change
`@@ -93,6 +93,9 @@ public override bool IsEnabledAndSupported(PostProcessRenderContext context)`
`93`	`93`	`}`
`94`	`94`	`}`
`95`	`95`
	`96`	`+#if UNITY_2017_1_OR_NEWER`
	`97`	`+ [UnityEngine.Scripting.Preserve]`
	`98`	`+#endif`
`96`	`99`	`internal sealed class BloomRenderer : PostProcessEffectRenderer<Bloom>`
`97`	`100`	`{`
`98`	`101`	`enum Pass`
Original file line number	Diff line number	Diff line change
`@@ -37,6 +37,9 @@ public override bool IsEnabledAndSupported(PostProcessRenderContext context)`
`37`	`37`	`}`
`38`	`38`	`}`
`39`	`39`
	`40`	`+#if UNITY_2017_1_OR_NEWER`
	`41`	`+ [UnityEngine.Scripting.Preserve]`
	`42`	`+#endif`
`40`	`43`	`internal sealed class ChromaticAberrationRenderer : PostProcessEffectRenderer<ChromaticAberration>`
`41`	`44`	`{`
`42`	`45`	`Texture2D m_InternalSpectralLut;`
Original file line number	Diff line number	Diff line change
`@@ -387,6 +387,9 @@ public override bool IsEnabledAndSupported(PostProcessRenderContext context)`
`387`	`387`	`}`
`388`	`388`	`}`
`389`	`389`
	`390`	`+#if UNITY_2017_1_OR_NEWER`
	`391`	`+ [UnityEngine.Scripting.Preserve]`
	`392`	`+#endif`
`390`	`393`	`internal sealed class ColorGradingRenderer : PostProcessEffectRenderer<ColorGrading>`
`391`	`394`	`{`
`392`	`395`	`enum Pass`
Original file line number	Diff line number	Diff line change
`@@ -77,6 +77,9 @@ public override bool IsEnabledAndSupported(PostProcessRenderContext context)`
`77`	`77`	`}`
`78`	`78`	`}`
`79`	`79`
	`80`	`+#if UNITY_2017_1_OR_NEWER`
	`81`	`+ [UnityEngine.Scripting.Preserve]`
	`82`	`+#endif`
`80`	`83`	`// TODO: Doesn't play nice with alpha propagation, see if it can be fixed without killing performances`
`81`	`84`	`internal sealed class DepthOfFieldRenderer : PostProcessEffectRenderer<DepthOfField>`
`82`	`85`	`{`
Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,9 @@`
`3`	`3`
`4`	`4`	`namespace UnityEngine.Rendering.PostProcessing`
`5`	`5`	`{`
	`6`	`+#if UNITY_2017_1_OR_NEWER`
	`7`	`+ [UnityEngine.Scripting.Preserve]`
	`8`	`+#endif`
`6`	`9`	`[Serializable]`
`7`	`10`	`internal sealed class Dithering`
`8`	`11`	`{`