Fixes from code review

Unity-Technologies · Jul 29, 2021 · 09ce4aa · 09ce4aa
1 parent 53c199d
commit 09ce4aa
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/com.unity.render-pipelines.core/Runtime/Common/DynamicResolutionHandler.cs b/com.unity.render-pipelines.core/Runtime/Common/DynamicResolutionHandler.cs
@@ -502,9 +502,6 @@ public Vector2Int GetScaledSize(Vector2Int size)
 
             Vector2Int scaledSize = ApplyScalesOnSize(size);
 
-            scaledSize.x = Math.Min(scaledSize.x, size.x);
-            scaledSize.y = Math.Min(scaledSize.y, size.y);
-
             m_LastScaledSize = scaledSize;
             return scaledSize;
         }
@@ -530,6 +527,9 @@ internal Vector2Int ApplyScalesOnSize(Vector2Int size, Vector2 scales)
                 scaledSize.y += (1 & scaledSize.y);
             }
 
+            scaledSize.x = Math.Min(scaledSize.x, size.x);
+            scaledSize.y = Math.Min(scaledSize.y, size.y);
+
             return scaledSize;
         }
 

diff --git a/com.unity.render-pipelines.high-definition/Runtime/RenderPipeline/RenderPass/DLSSPass.cs b/com.unity.render-pipelines.high-definition/Runtime/RenderPipeline/RenderPass/DLSSPass.cs
@@ -351,7 +351,6 @@ private bool ShouldUseAutomaticSettings()
 
             public void UpdateViewState(
                 in DlssViewData viewData,
-                /*uint backbufferWidth, uint backbufferHeight,*/
                 CommandBuffer cmdBuffer)
             {
                 bool shouldUseOptimalSettings = ShouldUseAutomaticSettings();