Fix unused variable errors (warnings as errors) (#18914)

* Add missing [[maybe_unused]] keywords for unused parameters to satisfy sensitive compiler settings

Signed-off-by: Steve Pham <[email protected]>

* Minor fix

Signed-off-by: Steve Pham <[email protected]>

---------

Signed-off-by: Steve Pham <[email protected]>

Code/Editor/AssetImporter/AssetImporterManager/AssetImporterManager.cpp

@@ -540,7 +540,7 @@ void AssetImporterManager::ProcessMoveFiles()
     }
 }
 
-bool AssetImporterManager::Copy(QString relativePath, QString oldAbsolutePath, QString destinationAbsolutePath)
+bool AssetImporterManager::Copy([[maybe_unused]] QString relativePath, QString oldAbsolutePath, QString destinationAbsolutePath)
 {
     QString fileName = GetFileName(destinationAbsolutePath);
     QString subPath = QFileInfo(destinationAbsolutePath).absoluteDir().absolutePath();

Code/Editor/CryEdit.cpp

@@ -1765,7 +1765,7 @@ bool CCryEditApp::InitInstance()
 }
 
 //////////////////////////////////////////////////////////////////////////
-void CCryEditApp::LoadFile(QString fileName)
+void CCryEditApp::LoadFile([[maybe_unused]] QString fileName)
 {
     if (GetIEditor()->GetViewManager()->GetViewCount() == 0)
     {

Code/Framework/AzCore/AzCore/Instance/InstancePool.h

@@ -148,7 +148,7 @@ namespace AZ
         template<class T>
         AZ::Outcome<AZStd::shared_ptr<InstancePool<T>>, AZStd::string> CreatePool(
             typename InstancePool<T>::ResetInstance resetFunc = [](T&){}, 
-            typename InstancePool<T>::CreateInstance createFunc = []() { return new T{}; })
+            [[maybe_unused]] typename InstancePool<T>::CreateInstance createFunc = []() { return new T{}; })
         {
             return CreatePool<T>(GetDefaultName<T>(), resetFunc);
         }

Code/Framework/AzCore/Tests/Asset/AssetManagerLoadingTests.cpp

@@ -3350,7 +3350,7 @@ namespace UnitTest
                 BusDisconnect();
             }
 
-            void OnAssetReady(Asset<AssetData> asset) override
+            void OnAssetReady([[maybe_unused]] Asset<AssetData> asset) override
             {
                 m_loadSignal->release();
             }
@@ -3542,7 +3542,7 @@ namespace UnitTest
                 BusDisconnect();
             }
 
-            void OnAssetReady(Asset<AssetData> asset) override
+            void OnAssetReady([[maybe_unused]] Asset<AssetData> asset) override
             {
                 ColoredPrintf(COLOR_YELLOW, "ThreadA: OnAssetReady called \n");
                 m_onAssetReadySignal.release();

Code/Framework/AzCore/Tests/Script.cpp

@@ -326,7 +326,7 @@ namespace UnitTest
 
         virtual void OnEventConst() const {};
 
-        virtual BehaviorTestClass OnEventResultWithBehaviorClassParameter(BehaviorTestClass data) { return BehaviorTestClass(); };
+        virtual BehaviorTestClass OnEventResultWithBehaviorClassParameter([[maybe_unused]] BehaviorTestClass data) { return BehaviorTestClass(); };
     };
 
     typedef AZ::EBus<BehaviorTestBusEvents> BehaviorTestBus;

Code/Framework/AzFramework/AzFramework/Archive/Archive.cpp

@@ -1536,7 +1536,7 @@ namespace AZ::IO
     //////////////////////////////////////////////////////////////////////////
     // open the physical archive file - creates if it doesn't exist
     // returns nullptr if it's invalid or can't open the file
-    AZStd::intrusive_ptr<INestedArchive> Archive::OpenArchive(AZStd::string_view szPath, AZStd::string_view bindRoot, uint32_t nFlags, AZStd::intrusive_ptr<AZ::IO::MemoryBlock> pData)
+    AZStd::intrusive_ptr<INestedArchive> Archive::OpenArchive(AZStd::string_view szPath, AZStd::string_view bindRoot, uint32_t nFlags, [[maybe_unused]] AZStd::intrusive_ptr<AZ::IO::MemoryBlock> pData)
     {
         auto szFullPath = AZ::IO::FileIOBase::GetDirectInstance()->ResolvePath(szPath);
         if (!szFullPath)

Code/Framework/AzFramework/AzFramework/Archive/ArchiveBus.h

@@ -26,7 +26,7 @@ namespace AZ::IO
         static const AZ::EBusHandlerPolicy HandlerPolicy = AZ::EBusHandlerPolicy::Multiple;
         static const AZ::EBusAddressPolicy AddressPolicy = AZ::EBusAddressPolicy::Single;
 
-        virtual void BundleOpened([[maybe_unused]] const char* bundleName, AZStd::shared_ptr<AzFramework::AssetBundleManifest> bundleManifest, [[maybe_unused]] const char* nextBundle, AZStd::shared_ptr<AzFramework::AssetRegistry> bundleCatalog) {}
+        virtual void BundleOpened([[maybe_unused]] const char* bundleName, [[maybe_unused]] AZStd::shared_ptr<AzFramework::AssetBundleManifest> bundleManifest, [[maybe_unused]] const char* nextBundle, [[maybe_unused]] AZStd::shared_ptr<AzFramework::AssetRegistry> bundleCatalog) {}
         virtual void BundleClosed([[maybe_unused]] const char* bundleName) {}
 
         // Sent when a file is accessed through Archive

Code/Framework/AzQtComponents/AzQtComponents/Components/Widgets/ColorPicker/PaletteCardCollection.cpp

@@ -206,7 +206,7 @@ namespace AzQtComponents
         }
     }
 
-    QString PaletteCardCollection::uniquePaletteName(QSharedPointer<PaletteCard> card, const QString& name) const
+    QString PaletteCardCollection::uniquePaletteName([[maybe_unused]] QSharedPointer<PaletteCard> card, const QString& name) const
     {
         const auto paletteNameExists = [this](const QString& name)
         {

Code/Framework/AzToolsFramework/AzToolsFramework/ActionManager/Action/ActionManagerNotificationBus.h

@@ -23,7 +23,7 @@ namespace AzToolsFramework
         //////////////////////////////////////////////////////////////////////////
 
         //! Triggered when an action's enabled or checked state changes.
-        virtual void OnActionStateChanged(AZStd::string actionIdentifier) {}
+        virtual void OnActionStateChanged([[maybe_unused]] AZStd::string actionIdentifier) {}
 
     protected:
         ~ActionManagerNotifications() = default;

Code/Framework/AzToolsFramework/AzToolsFramework/Slice/SliceUtilities.cpp

@@ -1029,7 +1029,7 @@ namespace AzToolsFramework
         }
 
         //=========================================================================
-        SliceTransaction::Result SlicePreSaveCallbackForWorldEntities(SliceTransaction::TransactionPtr transaction, const char* fullPath, SliceTransaction::SliceAssetPtr& asset)
+        SliceTransaction::Result SlicePreSaveCallbackForWorldEntities([[maybe_unused]] SliceTransaction::TransactionPtr transaction, const char* fullPath, SliceTransaction::SliceAssetPtr& asset)
         {
             AZ_PROFILE_SCOPE(AzToolsFramework, "SliceUtilities::SlicePreSaveCallbackForWorldEntities");
 

Code/Framework/AzToolsFramework/Tests/SliceUpgradeTestsData.h

@@ -131,7 +131,7 @@ namespace UnitTest
                 serializeContext->Class<TestComponentA_V1, AzToolsFramework::Components::EditorComponentBase>()
                     ->Version(1)
                     ->Field("NewData", &TestComponentA_V1::m_data)
-                    ->TypeChange<TestDataA, NewTestDataA>("Data", 0, 1, [](TestDataA in) -> NewTestDataA {
+                    ->TypeChange<TestDataA, NewTestDataA>("Data", 0, 1, []([[maybe_unused]] TestDataA in) -> NewTestDataA {
                         return NewTestDataA();
                     })
                     ->NameChange(0, 1, "Data", "NewData")

Code/Legacy/CrySystem/XML/XMLBinaryNode.h

@@ -91,7 +91,7 @@ public:
     virtual bool getAttributeByIndex(int index, XmlString& key, XmlString& value);
 
 
-    void copyAttributes(XmlNodeRef fromNode) override { assert(0); };
+    void copyAttributes([[maybe_unused]] XmlNodeRef fromNode) override { assert(0); };
 
     //! Get XML Node attribute for specified key.
     const char* getAttr(const char* key) const override;

Code/Tools/AssetProcessor/native/AssetManager/AssetRequestHandler.cpp

@@ -290,7 +290,7 @@ namespace
         return response;
     }
 
-    GetScanFoldersResponse HandleGetScanFoldersRequest(MessageData<GetScanFoldersRequest> messageData)
+    GetScanFoldersResponse HandleGetScanFoldersRequest([[maybe_unused]] MessageData<GetScanFoldersRequest> messageData)
     {
         bool success = true;
         AZStd::vector<AZStd::string> scanFolders;
@@ -305,7 +305,7 @@ namespace
         return GetScanFoldersResponse(move(scanFolders));
     }
 
-    GetAssetSafeFoldersResponse HandleGetAssetSafeFoldersRequest(MessageData<GetAssetSafeFoldersRequest> messageData)
+    GetAssetSafeFoldersResponse HandleGetAssetSafeFoldersRequest([[maybe_unused]] MessageData<GetAssetSafeFoldersRequest> messageData)
     {
         bool success = true;
         AZStd::vector<AZStd::string> assetSafeFolders;

Code/Tools/AssetProcessor/native/AssetManager/assetProcessorManager.cpp

@@ -350,7 +350,7 @@ namespace AssetProcessor
     }
 
     //! A network request came in, Given a Job Run Key (from the above Job Request), asking for the actual log for that job.
-    GetAbsoluteAssetDatabaseLocationResponse AssetProcessorManager::ProcessGetAbsoluteAssetDatabaseLocationRequest(MessageData<GetAbsoluteAssetDatabaseLocationRequest> messageData)
+    GetAbsoluteAssetDatabaseLocationResponse AssetProcessorManager::ProcessGetAbsoluteAssetDatabaseLocationRequest([[maybe_unused]] MessageData<GetAbsoluteAssetDatabaseLocationRequest> messageData)
     {
         GetAbsoluteAssetDatabaseLocationResponse response;
 

Code/Tools/AssetProcessor/native/tests/UnitTestUtilities.cpp

@@ -166,7 +166,7 @@ namespace UnitTests
     }
 
     void MockMultiBuilderInfoHandler::ProcessJob(
-        AssetBuilderExtraInfo extraInfo,
+        [[maybe_unused]] AssetBuilderExtraInfo extraInfo,
         [[maybe_unused]] const AssetBuilderSDK::ProcessJobRequest& request,
         AssetBuilderSDK::ProcessJobResponse& response)
     {

Code/Tools/AssetProcessor/native/tests/assetmanager/AssetManagerTestingBase.cpp

@@ -161,7 +161,7 @@ namespace UnitTests
         QObject::connect(
             m_rc.get(),
             &AssetProcessor::RCController::FileFailed,
-            [this](auto entryIn)
+            [this]([[maybe_unused]] auto entryIn)
             {
                 m_fileFailed = true;
             });
@@ -228,7 +228,7 @@ namespace UnitTests
         QObject::connect(
             m_assetProcessorManager.get(),
             &AssetProcessor::AssetProcessorManager::ProcessingDelayed,
-            [&delayed](QString filePath)
+            [&delayed]([[maybe_unused]] QString filePath)
             {
                 delayed = true;
             });
@@ -236,7 +236,7 @@ namespace UnitTests
         QObject::connect(
             m_assetProcessorManager.get(),
             &AssetProcessor::AssetProcessorManager::ProcessingResumed,
-            [&delayed](QString filePath)
+            [&delayed]([[maybe_unused]] QString filePath)
             {
                 delayed = false;
             });
@@ -420,7 +420,7 @@ namespace UnitTests
         QObject::connect(
             m_rc.get(),
             &AssetProcessor::RCController::FileCompiled,
-            [this](AssetProcessor::JobEntry entry, AssetBuilderSDK::ProcessJobResponse response)
+            [this](AssetProcessor::JobEntry entry, [[maybe_unused]] AssetBuilderSDK::ProcessJobResponse response)
             {
                 QMetaObject::invokeMethod(m_rc.get(), "OnAddedToCatalog", Qt::QueuedConnection, Q_ARG(AssetProcessor::JobEntry, entry));
             });

Code/Tools/AssetProcessor/native/tests/assetmanager/AssetProcessorManagerTest.cpp

@@ -5621,7 +5621,7 @@ TEST_F(ChainJobDependencyTest, TestChainDependency_Multi)
 
     // Wait for the file compiled event and trigger OnAddedToCatalog with a delay, this is what causes rccontroller to process out of order
     AZStd::vector<JobEntry> finishedJobs;
-    QObject::connect(m_data->m_rcController.get(), &RCController::FileCompiled, [this, &finishedJobs](JobEntry entry, AssetBuilderSDK::ProcessJobResponse response)
+    QObject::connect(m_data->m_rcController.get(), &RCController::FileCompiled, [this, &finishedJobs](JobEntry entry, [[maybe_unused]] AssetBuilderSDK::ProcessJobResponse response)
         {
             finishedJobs.push_back(entry);
 

Code/Tools/AssetProcessor/native/tests/assetmanager/MockAssetProcessorManager.cpp

@@ -10,17 +10,17 @@
 
 namespace UnitTests
 {
-    void MockAssetProcessorManager::AssessAddedFile(QString filePath)
+    void MockAssetProcessorManager::AssessAddedFile([[maybe_unused]] QString filePath)
     {
         m_events[TestEvents::Added].Signal();
     }
 
-    void MockAssetProcessorManager::AssessModifiedFile(QString filePath)
+    void MockAssetProcessorManager::AssessModifiedFile([[maybe_unused]] QString filePath)
     {
         m_events[TestEvents::Modified].Signal();
     }
 
-    void MockAssetProcessorManager::AssessDeletedFile(QString filePath)
+    void MockAssetProcessorManager::AssessDeletedFile([[maybe_unused]] QString filePath)
     {
         m_events[TestEvents::Deleted].Signal();
     }

Code/Tools/AssetProcessor/native/tests/assetmanager/MockFileProcessor.cpp

@@ -10,12 +10,12 @@
 
 namespace UnitTests
 {
-     void MockFileProcessor::AssessAddedFile(QString fileName)
+     void MockFileProcessor::AssessAddedFile([[maybe_unused]] QString fileName)
     {
         m_events[TestEvents::Added].Signal();
     }
 
-     void MockFileProcessor::AssessDeletedFile(QString fileName)
+     void MockFileProcessor::AssessDeletedFile([[maybe_unused]] QString fileName)
     {
         m_events[TestEvents::Deleted].Signal();
     }

Code/Tools/AssetProcessor/native/tests/resourcecompiler/RCControllerTest.cpp

@@ -222,7 +222,7 @@ TEST_F(RCcontrollerTest_Simple, DISABLED_SameJobIsCompletedMultipleTimes_Complet
     using namespace AssetProcessor;
 
     AZStd::vector<JobEntry> jobEntries;
-    QObject::connect(m_rcController.get(), &RCController::FileCompiled, [&jobEntries](JobEntry entry, AssetBuilderSDK::ProcessJobResponse response)
+    QObject::connect(m_rcController.get(), &RCController::FileCompiled, [&jobEntries](JobEntry entry, [[maybe_unused]] AssetBuilderSDK::ProcessJobResponse response)
     {
         jobEntries.push_back(entry);
     });

Code/Tools/AssetProcessor/native/ui/ProductAssetDetailsPanel.cpp

@@ -261,7 +261,7 @@ namespace AssetProcessor
     }
 
     void ProductAssetDetailsPanel::BuildIncomingProductDependencies(
-        const AZStd::shared_ptr<const ProductAssetTreeItemData> productItemData,
+        [[maybe_unused]] const AZStd::shared_ptr<const ProductAssetTreeItemData> productItemData,
         const AZ::Data::AssetId& assetId,
         const AZStd::string& platform)
     {

Code/Tools/AssetProcessor/native/unittests/AssetRequestHandlerUnitTests.cpp

@@ -66,7 +66,7 @@ namespace AssetProcessor
             m_fenceId = fenceId;
             return m_fenceFileName;
         }
-        bool DeleteFenceFile(QString fenceFileName) override
+        bool DeleteFenceFile([[maybe_unused]] QString fenceFileName) override
         {
             m_numTimesDeleteFenceFileCalled++;
             return m_deleteFenceFileResult;

Code/Tools/AssetProcessor/native/unittests/RCcontrollerUnitTests.cpp

@@ -202,7 +202,7 @@ void RCcontrollerUnitTests::ConnectCompileGroupSignalsAndSlots(bool& gotCreated,
 
 void RCcontrollerUnitTests::ConnectJobSignalsAndSlots(bool& allJobsCompleted, JobEntry& completedJob)
 {
-    QObject::connect(m_rcController.get(), &RCController::FileCompiled, this, [&](JobEntry entry, AssetBuilderSDK::ProcessJobResponse response)
+    QObject::connect(m_rcController.get(), &RCController::FileCompiled, this, [&](JobEntry entry, [[maybe_unused]] AssetBuilderSDK::ProcessJobResponse response)
         {
             completedJob = entry;
         });

Code/Tools/AssetProcessor/native/utilities/AssetUtilEBusHelper.h

@@ -39,9 +39,9 @@ public:
 
     virtual ~AssetProcessorPlaformBusTraits() {}
     //Informs that the AP got a connection for this platform.
-    virtual void AssetProcessorPlatformConnected(const AZStd::string platform) {}
+    virtual void AssetProcessorPlatformConnected([[maybe_unused]] const AZStd::string platform) {}
     //Informs that a connection got disconnected for this platform.
-    virtual void AssetProcessorPlatformDisconnected(const AZStd::string platform) {}
+    virtual void AssetProcessorPlatformDisconnected([[maybe_unused]] const AZStd::string platform) {}
 };
 
 using AssetProcessorPlatformBus = AZ::EBus<AssetProcessorPlaformBusTraits>;

Code/Tools/AssetProcessor/native/utilities/BatchApplicationManager.cpp

@@ -96,7 +96,7 @@ const char* BatchApplicationManager::GetLogBaseName()
     return "AP_Batch";
 }
 
-ApplicationManager::RegistryCheckInstructions BatchApplicationManager::PopupRegistryProblemsMessage(QString warningText)
+ApplicationManager::RegistryCheckInstructions BatchApplicationManager::PopupRegistryProblemsMessage([[maybe_unused]] QString warningText)
 {
     return RegistryCheckInstructions::Exit;
 }

Code/Tools/AssetProcessor/native/utilities/BuilderManager.cpp

@@ -66,7 +66,7 @@ namespace AssetProcessor
         }
     }
 
-    void BuilderManager::IncomingBuilderPing(AZ::u32 connId, AZ::u32 /*type*/, AZ::u32 serial, QByteArray payload, QString platform)
+    void BuilderManager::IncomingBuilderPing(AZ::u32 connId, AZ::u32 /*type*/, AZ::u32 serial, QByteArray payload, [[maybe_unused]] QString platform)
     {
         AssetBuilder::BuilderHelloRequest requestPing;
         AssetBuilder::BuilderHelloResponse responsePing;

Gems/Atom/Asset/ImageProcessingAtom/Code/Source/Compressors/ISPCTextureCompressor.cpp

@@ -248,7 +248,7 @@ namespace ImageProcessingAtom
         return destinationImage;
     }
 
-    IImageObjectPtr ISPCCompressor::DecompressImage(IImageObjectPtr sourceImage, [[maybe_unused]] EPixelFormat destinationFormat) const
+    IImageObjectPtr ISPCCompressor::DecompressImage([[maybe_unused]] IImageObjectPtr sourceImage, [[maybe_unused]] EPixelFormat destinationFormat) const
     {
         return nullptr;
     }

Gems/Atom/Feature/Common/Code/Source/Mesh/MeshFeatureProcessor.cpp

@@ -1841,7 +1841,7 @@ namespace AZ
             OnAssetReady(asset);
         }
 
-        void ModelDataInstance::MeshLoader::OnAssetError(Data::Asset<Data::AssetData> asset)
+        void ModelDataInstance::MeshLoader::OnAssetError([[maybe_unused]] Data::Asset<Data::AssetData> asset)
         {
             // Note: m_modelAsset and asset represents same asset, but only m_modelAsset contains the file path in its hint from serialization
             AZ_Error(

Gems/Atom/RHI/Code/Tests/Device.h

@@ -74,7 +74,7 @@ namespace UnitTest
 
         AZ::RHI::ResourceMemoryRequirements GetResourceMemoryRequirements([[maybe_unused]] const AZ::RHI::BufferDescriptor& descriptor) { return AZ::RHI::ResourceMemoryRequirements{}; };
 
-        void ObjectCollectionNotify(AZ::RHI::ObjectCollectorNotifyFunction notifyFunction) override {}
+        void ObjectCollectionNotify([[maybe_unused]] AZ::RHI::ObjectCollectorNotifyFunction notifyFunction) override {}
 
         AZ::RHI::ShadingRateImageValue ConvertShadingRate([[maybe_unused]] AZ::RHI::ShadingRate rate) const override
         {

Gems/Atom/RHI/DX12/Code/Source/RHI/RayTracingShaderTable.cpp

@@ -51,7 +51,7 @@ namespace AZ
                                                            const RHI::DeviceRayTracingBufferPools& bufferPools,
                                                            const RHI::DeviceRayTracingShaderTableRecordList& recordList,
                                                            uint32_t shaderRecordSize,
-                                                           AZStd::wstring shaderTableName,
+                                                           [[maybe_unused]] AZStd::wstring shaderTableName,
                                                            Microsoft::WRL::ComPtr<ID3D12StateObjectProperties>& stateObjectProperties)
         {
 

Gems/Atom/RPI/Code/Include/Atom/RPI.Public/SceneBus.h

@@ -52,7 +52,7 @@ namespace AZ
             //! @deprecated use OnRenderPipelineChanged(RenderPipeline*, RenderPipelineChangeType::Added)
             //! Notifies when a render pipeline is added to this scene. 
             //! @param pipeline The render pipeline which was added
-            virtual void OnRenderPipelineAdded(RenderPipelinePtr pipeline) {};
+            virtual void OnRenderPipelineAdded([[maybe_unused]] RenderPipelinePtr pipeline) {};
                         
             //! O3DE_DEPRECATION_NOTICE(GHI-12687)
             //! @deprecated use OnRenderPipelineChanged(RenderPipeline*, RenderPipelineChangeType::PassChanged)
@@ -77,7 +77,7 @@ namespace AZ
             //! @param viewTag The viewTag in this render pipeline which the new view was set to
             //! @param newView The view which was set to the render pipeline's view tag
             //! @param previousView The previous view associates to render pipeline's view tag before the new view was set
-            virtual void OnRenderPipelinePersistentViewChanged([[maybe_unused]] RenderPipeline* renderPipeline, PipelineViewTag viewTag, ViewPtr newView, ViewPtr previousView) {}
+            virtual void OnRenderPipelinePersistentViewChanged([[maybe_unused]] RenderPipeline* renderPipeline, [[maybe_unused]] PipelineViewTag viewTag, [[maybe_unused]] ViewPtr newView, [[maybe_unused]] ViewPtr previousView) {}
 
             //! Notifies that the pipeline state lookup table has been rebuilt, so the pipeline state data (multisample state,
             //! render attachment configuration, etc) for a DrawListTag may have changed. 

Gems/Atom/RPI/Code/Tests/Common/RHI/Stubs.h

@@ -75,7 +75,7 @@ namespace UnitTest
             void PreShutdown() override {}
             AZ::RHI::ResourceMemoryRequirements GetResourceMemoryRequirements([[maybe_unused]] const AZ::RHI::ImageDescriptor& descriptor) override { return AZ::RHI::ResourceMemoryRequirements{}; };
             AZ::RHI::ResourceMemoryRequirements GetResourceMemoryRequirements([[maybe_unused]] const AZ::RHI::BufferDescriptor& descriptor) override { return AZ::RHI::ResourceMemoryRequirements{}; };
-            void ObjectCollectionNotify(AZ::RHI::ObjectCollectorNotifyFunction notifyFunction) override {}
+            void ObjectCollectionNotify([[maybe_unused]] AZ::RHI::ObjectCollectorNotifyFunction notifyFunction) override {}
             AZ::RHI::ShadingRateImageValue ConvertShadingRate([[maybe_unused]] AZ::RHI::ShadingRate rate) const override
             {
                 return AZ::RHI::ShadingRateImageValue{};

Gems/Atom/RPI/Code/Tests/Common/TestFeatureProcessors.h

@@ -48,7 +48,7 @@ namespace UnitTest
         void Render(const RenderPacket&)  override {};
 
         // Overrides for scene notification bus handler
-        void OnRenderPipelinePersistentViewChanged(AZ::RPI::RenderPipeline* renderPipeline, AZ::RPI::PipelineViewTag viewTag, AZ::RPI::ViewPtr newView, AZ::RPI::ViewPtr previousView) override
+        void OnRenderPipelinePersistentViewChanged(AZ::RPI::RenderPipeline* renderPipeline, [[maybe_unused]] AZ::RPI::PipelineViewTag viewTag, [[maybe_unused]] AZ::RPI::ViewPtr newView, [[maybe_unused]] AZ::RPI::ViewPtr previousView) override
         {
             m_viewSetCount++;
             m_lastPipeline = renderPipeline;

Gems/AtomLyIntegration/AtomBridge/Code/Source/AtomDebugDisplayViewportInterface.cpp

@@ -151,7 +151,7 @@ namespace AZ::AtomBridge
     }
 
 
-    void AtomDebugDisplayViewportInterface::OnViewportDefaultViewChanged(AZ::RPI::ViewPtr view)
+    void AtomDebugDisplayViewportInterface::OnViewportDefaultViewChanged([[maybe_unused]] AZ::RPI::ViewPtr view)
     {
         ResetRenderState();
         if (!m_defaultInstance)

Gems/AtomLyIntegration/CommonFeatures/Code/Include/AtomLyIntegration/CommonFeatures/Decals/DecalBus.h

@@ -105,7 +105,7 @@ namespace AZ
 
             //! Signals that the material has changed
             //! @param materialAsset The material asset of the decal
-            virtual void OnMaterialChanged(Data::Asset<RPI::MaterialAsset> materialAsset){ }
+            virtual void OnMaterialChanged([[maybe_unused]] Data::Asset<RPI::MaterialAsset> materialAsset){ }
         };
 
         /// The EBus for decal notification events.

Gems/AtomLyIntegration/CommonFeatures/Code/Source/Material/EditorMaterialComponentSlot.cpp

@@ -464,7 +464,7 @@ namespace AZ
             }
         }
 
-        void EditorMaterialComponentSlot::OnAssetReloaded(Data::Asset<Data::AssetData> asset)
+        void EditorMaterialComponentSlot::OnAssetReloaded([[maybe_unused]] Data::Asset<Data::AssetData> asset)
         {
             UpdatePreview();
         }

Gems/DiffuseProbeGrid/Code/Source/Components/DiffuseProbeGridComponentController.cpp

@@ -204,7 +204,7 @@ namespace AZ
             m_boxChangedByGridEvent.Signal(true);
         }
 
-        void DiffuseProbeGridComponentController::OnAssetReady(Data::Asset<Data::AssetData> asset)
+        void DiffuseProbeGridComponentController::OnAssetReady([[maybe_unused]] Data::Asset<Data::AssetData> asset)
         {
             // if all assets are ready we can set the baked texture images
             if (m_configuration.m_bakedIrradianceTextureAsset.IsReady() &&

Gems/DiffuseProbeGrid/Code/Source/Render/DiffuseProbeGridFeatureProcessor.cpp

@@ -859,7 +859,7 @@ namespace AZ
         }
 
         void DiffuseProbeGridFeatureProcessor::OnRenderPipelinePersistentViewChanged(
-            RPI::RenderPipeline* renderPipeline, RPI::PipelineViewTag viewTag, RPI::ViewPtr newView, RPI::ViewPtr previousView)
+            RPI::RenderPipeline* renderPipeline, RPI::PipelineViewTag viewTag, RPI::ViewPtr newView, [[maybe_unused]] RPI::ViewPtr previousView)
         {
             if (m_sceneAndViewShader)
             {

Gems/EMotionFX/Code/EMotionFX/Tools/EMotionStudio/EMStudioSDK/Source/FileManager.cpp

@@ -237,7 +237,7 @@ namespace EMStudio
         return false;
     }
 
-    void FileManager::SourceFileChanged(AZStd::string relativePath, AZStd::string scanFolder, [[maybe_unused]] AZ::TypeId sourceTypeId)
+    void FileManager::SourceFileChanged(AZStd::string relativePath, [[maybe_unused]] AZStd::string scanFolder, [[maybe_unused]] AZ::TypeId sourceTypeId)
     {
         AZStd::string filename;
         AZStd::string assetSourcePath = AZ::IO::FileIOBase::GetInstance()->GetAlias("@projectroot@");

Gems/EMotionFX/Code/Source/Integration/Editor/Components/EditorActorComponent.cpp

@@ -596,7 +596,7 @@ namespace EMotionFX
                 if (dependencyAsset && dependencyAsset.GetType() == azrtti_typeid<AZ::RPI::ModelAsset>())
                 {
                      m_modelReloadedEventHandler = AZ::Render::ModelReloadedEvent::Handler(
-                        [this](AZ::Data::Asset<AZ::RPI::ModelAsset> modelAsset)
+                        [this]([[maybe_unused]] AZ::Data::Asset<AZ::RPI::ModelAsset> modelAsset)
                         {
                             m_actorAsset.QueueLoad();
                         });

Gems/EditorPythonBindings/Code/Source/PythonMarshalComponent.cpp

@@ -263,7 +263,7 @@ namespace EditorPythonBindings
             return aznew AZStd::any(address, valueInfo);
         }
 
-        AZStd::optional<PythonMarshalTypeRequests::BehaviorValueResult> PythonToParameterWithProxy(EditorPythonBindings::PythonProxyObject* proxyObj, pybind11::object pyObj, AZ::BehaviorArgument& outValue)
+        AZStd::optional<PythonMarshalTypeRequests::BehaviorValueResult> PythonToParameterWithProxy(EditorPythonBindings::PythonProxyObject* proxyObj, [[maybe_unused]] pybind11::object pyObj, AZ::BehaviorArgument& outValue)
         {
             auto behaviorObject = proxyObj->GetBehaviorObject();
             if (!behaviorObject)

Gems/GameState/Code/Include/GameState/GameStateNotificationBus.h

@@ -31,8 +31,8 @@ namespace GameState
         //! Called when a game state transition occurs
         //! \param[in] oldGameState The old game state we are transitioning from (can be null)
         //! \param[in] newGameState The new game state we are transitioning into (can be null)
-        virtual void OnActiveGameStateChanged(AZStd::shared_ptr<IGameState> oldGameState,
-                                              AZStd::shared_ptr<IGameState> newGameState) {}
+        virtual void OnActiveGameStateChanged([[maybe_unused]] AZStd::shared_ptr<IGameState> oldGameState,
+                                              [[maybe_unused]] AZStd::shared_ptr<IGameState> newGameState) {}
     };
     using GameStateNotificationBus = AZ::EBus<GameStateNotifications>;
 } // namespace GameState

Gems/GameStateSamples/Code/Include/GameStateSamples/GameStatePrimaryUserMonitor.inl

@@ -42,7 +42,7 @@ namespace GameStateSamples
 
     ////////////////////////////////////////////////////////////////////////////////////////////////
     inline void GameStatePrimaryUserMonitor::OnActiveGameStateChanged(AZStd::shared_ptr<IGameState> oldGameState,
-                                                                      AZStd::shared_ptr<IGameState> newGameState)
+                                                                      [[maybe_unused]] AZStd::shared_ptr<IGameState> newGameState)
     {
         if (m_primaryUserSignedOutWhileLevelLoading &&
             azrtti_istypeof<GameStateLevelLoading>(oldGameState.get()))

Gems/GraphModel/Code/Include/GraphModel/GraphModelBus.h

@@ -233,11 +233,11 @@ namespace GraphModelIntegration
 
         //! Sent whenever a graph model slot value changes
         //! \param slot The slot that was modified in the graph.
-        virtual void OnGraphModelSlotModified(GraphModel::SlotPtr slot){};
+        virtual void OnGraphModelSlotModified(GraphModel::SlotPtr /*slot*/){};
 
         //! Something in the graph has been modified
         //! \param node The node that was modified in the graph.  If this is nullptr, some metadata on the graph itself was modified
-        virtual void OnGraphModelGraphModified(GraphModel::NodePtr node){};
+        virtual void OnGraphModelGraphModified(GraphModel::NodePtr /*node*/){};
 
         //! A request has been made to record data for an undoable operation 
         virtual void OnGraphModelRequestUndoPoint(){};

Gems/GraphModel/Code/Include/GraphModel/Integration/EditorMainWindow.h

@@ -50,7 +50,7 @@ namespace GraphModelIntegration
 
         /// Client can override this to handle click events on a wrapper node's action widget
         /// using a GraphModel::NodePtr instead of the lower-level GraphCanvas::NodeId
-        virtual void HandleWrapperNodeActionWidgetClicked(GraphModel::NodePtr wrapperNode, [[maybe_unused]] const QRect& actionWidgetBoundingRect, [[maybe_unused]] const QPointF& scenePoint, [[maybe_unused]] const QPoint& screenPoint) {}
+        virtual void HandleWrapperNodeActionWidgetClicked([[maybe_unused]] GraphModel::NodePtr wrapperNode, [[maybe_unused]] const QRect& actionWidgetBoundingRect, [[maybe_unused]] const QPointF& scenePoint, [[maybe_unused]] const QPoint& screenPoint) {}
 
         /// Keep track of the graphs we create on behalf of the client when
         /// new editor dock widgets are created.

Gems/GraphModel/Code/Source/Model/Module/ModuleGraphManager.cpp

@@ -44,7 +44,7 @@ namespace GraphModel
         AzToolsFramework::AssetSystemBus::Handler::BusDisconnect();
     }
 
-    void ModuleGraphManager::SourceFileChanged(AZStd::string relativePath, AZStd::string scanFolder, AZ::Uuid sourceUUID)
+    void ModuleGraphManager::SourceFileChanged(AZStd::string relativePath, [[maybe_unused]] AZStd::string scanFolder, AZ::Uuid sourceUUID)
     {
         AZStd::string extension;
         if (AzFramework::StringFunc::Path::GetExtension(relativePath.data(), extension) && extension == m_moduleFileExtension)

Gems/LandscapeCanvas/Code/Source/Editor/MainWindow.cpp

@@ -674,7 +674,7 @@ namespace LandscapeCanvasEditor
         UpdateConnectionData(connection, false /* added */);
     }
 
-    void MainWindow::PreOnGraphModelNodeWrapped(GraphModel::NodePtr wrapperNode, GraphModel::NodePtr node)
+    void MainWindow::PreOnGraphModelNodeWrapped([[maybe_unused]] GraphModel::NodePtr wrapperNode, GraphModel::NodePtr node)
     {
         if (m_ignoreGraphUpdates)
         {

Gems/LyShine/Code/Include/LyShine/Animation/IUiAnimation.h

@@ -1060,7 +1060,7 @@ struct IUiAnimationListener
     virtual ~IUiAnimationListener(){}
     //! callback on UI animation events
     virtual void OnUiAnimationEvent(EUiAnimationEvent uiAnimationEvent, IUiAnimSequence* pAnimSequence) = 0;
-    virtual void OnUiTrackEvent(AZStd::string eventName, AZStd::string valueName, [[maybe_unused]] IUiAnimSequence* pAnimSequence) {}
+    virtual void OnUiTrackEvent([[maybe_unused]] AZStd::string eventName, [[maybe_unused]] AZStd::string valueName, [[maybe_unused]] IUiAnimSequence* pAnimSequence) {}
     // </interfuscator:shuffle>
 };
 

Gems/LyShine/Code/Include/LyShine/Bus/UiAnimationBus.h

@@ -116,7 +116,7 @@ public: // member functions
     virtual void OnUiAnimationEvent(IUiAnimationListener::EUiAnimationEvent uiAnimationEvent, AZStd::string animSequenceName) = 0;
 
     //! Called on animation track event triggered
-    virtual void OnUiTrackEvent(AZStd::string eventName, AZStd::string valueName, AZStd::string animSequenceName) {}
+    virtual void OnUiTrackEvent([[maybe_unused]] AZStd::string eventName, [[maybe_unused]] AZStd::string valueName, [[maybe_unused]] AZStd::string animSequenceName) {}
 };
 
 typedef AZ::EBus<UiAnimationNotifications> UiAnimationNotificationBus;

Gems/LyShine/Code/Include/LyShine/Bus/UiVisualBus.h

@@ -39,7 +39,7 @@ public: // member functions
 
     //! Set the override font, if this visual component uses a font this will override it
     //! \param font   If null the font on the visual component will not be overridden
-    virtual void SetOverrideFont(FontFamilyPtr fontFamily) {};
+    virtual void SetOverrideFont([[maybe_unused]] FontFamilyPtr fontFamily) {};
 
     //! Set the override font effect, if this visual component uses a font this will override it
     virtual void SetOverrideFontEffect([[maybe_unused]] unsigned int fontEffectIndex) {};

Gems/LyShine/Code/Tests/AnimationTest.cpp

@@ -55,7 +55,7 @@ namespace UnitTest
             UiAnimationNotificationBus::Handler::BusDisconnect(m_busId);
         }
 
-        void OnUiAnimationEvent([[maybe_unused]] IUiAnimationListener::EUiAnimationEvent uiAnimationEvent, AZStd::string animSequenceName) {};
+        void OnUiAnimationEvent([[maybe_unused]] IUiAnimationListener::EUiAnimationEvent uiAnimationEvent, [[maybe_unused]] AZStd::string animSequenceName) {};
         void OnUiTrackEvent(AZStd::string eventName, AZStd::string valueName, AZStd::string animSequenceName)
         {
             m_recievedEvents.push_back(EventInfo{ eventName, valueName, animSequenceName });

Gems/MessagePopup/Code/Source/LyShineMessagePopup.cpp

@@ -117,7 +117,7 @@ namespace MessagePopup
     }
 
     //-----------------------------------------------------------------------------
-    void LyShineMessagePopup::OnShowPopup(AZ::u32 _popupID, const AZStd::string& _message, EPopupButtons _buttons, EPopupKind _kind, AZStd::function<void(int _button)> _callback, void** _popupClientID)
+    void LyShineMessagePopup::OnShowPopup(AZ::u32 _popupID, const AZStd::string& _message, EPopupButtons _buttons, EPopupKind _kind, [[maybe_unused]] AZStd::function<void(int _button)> _callback, void** _popupClientID)
     {
         AZ::EntityId canvasEntityId;
         bool isNavigationSupported = false;

Gems/Multiplayer/Code/Source/NetworkEntity/NetworkEntityManager.cpp

@@ -654,7 +654,7 @@ namespace Multiplayer
         return ticket;
     }
 
-    void NetworkEntityManager::OnRootSpawnableAssigned(AZ::Data::Asset<AzFramework::Spawnable> rootSpawnable,
+    void NetworkEntityManager::OnRootSpawnableAssigned([[maybe_unused]] AZ::Data::Asset<AzFramework::Spawnable> rootSpawnable,
         [[maybe_unused]] uint32_t generation)
     {
         auto* multiplayer = GetMultiplayer();

Gems/PhysX/Core/Code/Source/HeightfieldColliderComponent.cpp

@@ -110,7 +110,7 @@ namespace PhysX
         }
     }
 
-    void HeightfieldColliderComponent::OnAssetError(AZ::Data::Asset<AZ::Data::AssetData> asset)
+    void HeightfieldColliderComponent::OnAssetError([[maybe_unused]] AZ::Data::Asset<AZ::Data::AssetData> asset)
     {
         InitHeightfieldCollider(HeightfieldCollider::DataSource::GenerateNewHeightfield);
     }

Gems/ScriptCanvas/Code/Editor/GraphCanvas/Components/NodeDescriptors/FunctionNodeDescriptorComponent.cpp

@@ -68,17 +68,17 @@ namespace ScriptCanvasEditor
         AZ::Data::AssetBus::Handler::BusDisconnect();
     }
 
-    void FunctionNodeDescriptorComponent::OnAssetReloaded(AZ::Data::Asset<AZ::Data::AssetData> asset)
+    void FunctionNodeDescriptorComponent::OnAssetReloaded([[maybe_unused]] AZ::Data::Asset<AZ::Data::AssetData> asset)
     {
         TriggerUpdate();
     }
 
-    void FunctionNodeDescriptorComponent::OnAssetReloadError(AZ::Data::Asset<AZ::Data::AssetData> asset)
+    void FunctionNodeDescriptorComponent::OnAssetReloadError([[maybe_unused]] AZ::Data::Asset<AZ::Data::AssetData> asset)
     {
         TriggerUpdate();
     }
 
-    void FunctionNodeDescriptorComponent::OnAssetError(AZ::Data::Asset<AZ::Data::AssetData> asset)
+    void FunctionNodeDescriptorComponent::OnAssetError([[maybe_unused]] AZ::Data::Asset<AZ::Data::AssetData> asset)
     {
         TriggerUpdate();
     }

Gems/ScriptCanvas/Code/Editor/GraphCanvas/Components/NodeDescriptors/ScriptEventReceiverNodeDescriptorComponent.cpp

@@ -466,7 +466,7 @@ namespace ScriptCanvasEditor
         return this;
     }
 
-    void ScriptEventReceiverNodeDescriptorComponent::OnAssetReloaded(AZ::Data::Asset<AZ::Data::AssetData> asset)
+    void ScriptEventReceiverNodeDescriptorComponent::OnAssetReloaded([[maybe_unused]] AZ::Data::Asset<AZ::Data::AssetData> asset)
     {
         AZ::EntityId graphCanvasGraphId;
         GraphCanvas::SceneMemberRequestBus::EventResult(graphCanvasGraphId, GetEntityId(), &GraphCanvas::SceneMemberRequests::GetScene);

Gems/ScriptCanvas/Code/Editor/GraphCanvas/Components/NodeDescriptors/ScriptEventSenderNodeDescriptorComponent.cpp

@@ -74,7 +74,7 @@ namespace ScriptCanvasEditor
         SignalNeedsVersionConversion();
     }
 
-    void ScriptEventSenderNodeDescriptorComponent::OnAssetReloaded(AZ::Data::Asset<AZ::Data::AssetData> asset)
+    void ScriptEventSenderNodeDescriptorComponent::OnAssetReloaded([[maybe_unused]] AZ::Data::Asset<AZ::Data::AssetData> asset)
     {
         SignalNeedsVersionConversion();
     }

Gems/ScriptCanvas/Code/Include/ScriptCanvas/Grammar/ParsingUtilities.cpp

@@ -145,7 +145,7 @@ namespace ParsingUtilitiesCpp
             m_result += "\n";
         }
 
-        void EvaluateRoot(ExecutionTreeConstPtr node, const Slot*)
+        void EvaluateRoot(ExecutionTreeConstPtr, const Slot*)
         {
             m_result += "\nRoot:\n";
         }

Gems/ScriptCanvas/Code/Include/ScriptCanvas/Libraries/Core/ReceiveScriptEvent.cpp

@@ -140,7 +140,7 @@ namespace ScriptCanvas
                 }
             }
 
-            void ReceiveScriptEvent::InitializeEvent(AZ::Data::Asset<ScriptEvents::ScriptEventsAsset> asset, int eventIndex, SlotIdMapping& populationMapping)
+            void ReceiveScriptEvent::InitializeEvent([[maybe_unused]] AZ::Data::Asset<ScriptEvents::ScriptEventsAsset> asset, int eventIndex, SlotIdMapping& populationMapping)
             {
                 if (!m_handler)
                 {

Gems/ScriptCanvas/Code/Include/ScriptCanvas/Libraries/Core/SendScriptEvent.cpp

@@ -369,7 +369,7 @@ namespace ScriptCanvas
                 }
             }
 
-            bool SendScriptEvent::RegisterScriptEvent(AZ::Data::Asset<ScriptEvents::ScriptEventsAsset> asset)
+            bool SendScriptEvent::RegisterScriptEvent([[maybe_unused]] AZ::Data::Asset<ScriptEvents::ScriptEventsAsset> asset)
             {
                 if (!m_scriptEvent)
                 {

Gems/ScriptCanvas/Code/Include/ScriptCanvas/Translation/GraphToLua.cpp

@@ -1501,7 +1501,7 @@ namespace ScriptCanvas
             }
         }
 
-        void GraphToLua::WriteDebugInfoVariableChange(Grammar::ExecutionTreeConstPtr execution, Grammar::OutputAssignmentConstPtr output)
+        void GraphToLua::WriteDebugInfoVariableChange([[maybe_unused]] Grammar::ExecutionTreeConstPtr execution, Grammar::OutputAssignmentConstPtr output)
         {
             if (IsDebugInfoWritten())
             {
@@ -1663,7 +1663,7 @@ namespace ScriptCanvas
             }
         }
 
-        void GraphToLua::WriteFunctionCallNullCheckPost(Grammar::ExecutionTreeConstPtr execution)
+        void GraphToLua::WriteFunctionCallNullCheckPost([[maybe_unused]] Grammar::ExecutionTreeConstPtr execution)
         {
             m_dotLua.Outdent();
             m_dotLua.WriteLineIndented("end");

Gems/ScriptCanvas/Code/Include/ScriptCanvas/Translation/GraphToLuaUtility.cpp

@@ -29,7 +29,7 @@ namespace ScriptCanvas
     namespace Translation
     {
 
-        void CheckConversionStringPost(Writer& writer, Grammar::VariableConstPtr source, const Grammar::ConversionByIndex& conversions, size_t index)
+        void CheckConversionStringPost(Writer& writer, [[maybe_unused]] Grammar::VariableConstPtr source, const Grammar::ConversionByIndex& conversions, size_t index)
         {
             auto iter = conversions.find(index);
             if (iter == conversions.end())

Gems/ScriptEvents/Code/Include/ScriptEvents/ScriptEventsAssetRef.h

@@ -82,7 +82,7 @@ namespace ScriptEvents
         void OnAssetReloaded(AZ::Data::Asset<AZ::Data::AssetData> asset) override;
         void OnAssetUnloaded(const AZ::Data::AssetId assetId, const AZ::Data::AssetType assetType) override;
 
-        void OnAssetSaved(AZ::Data::Asset<AZ::Data::AssetData> asset, [[maybe_unused]] bool isSuccessful) override
+        void OnAssetSaved([[maybe_unused]] AZ::Data::Asset<AZ::Data::AssetData> asset, [[maybe_unused]] bool isSuccessful) override
         {
             SetAsset(m_asset);
         }

Gems/ScriptEvents/Code/Source/ScriptEventsAssetRef.cpp

@@ -99,7 +99,7 @@ namespace ScriptEvents
         return AZ::Edit::PropertyRefreshLevels::None;
     }
 
-    void ScriptEventsAssetRef::OnAssetReady(AZ::Data::Asset<AZ::Data::AssetData> asset)
+    void ScriptEventsAssetRef::OnAssetReady([[maybe_unused]] AZ::Data::Asset<AZ::Data::AssetData> asset)
     {
         if (ScriptEventsAsset* scriptEventAsset = m_asset.GetAs<ScriptEventsAsset>())
         {

Gems/ScriptEvents/Code/Tests/Tests/ScriptEventsTest_Core.cpp

@@ -231,7 +231,7 @@ namespace ScriptEventsTests
             AZ::Data::AssetBus::Handler::BusDisconnect();
         }
         
-        void OnAssetError(AZ::Data::Asset<AZ::Data::AssetData> assetData) override
+        void OnAssetError([[maybe_unused]] AZ::Data::Asset<AZ::Data::AssetData> assetData) override
         {
             EXPECT_TRUE(false);
             AZ::Data::AssetBus::Handler::BusDisconnect();
@@ -247,7 +247,7 @@ namespace ScriptEventsTests
             m_onReadyCallback();
         }
 
-        void OnAssetSaved(AZ::Data::Asset<AZ::Data::AssetData> asset, [[maybe_unused]] bool isSuccessful) override
+        void OnAssetSaved([[maybe_unused]] AZ::Data::Asset<AZ::Data::AssetData> asset, [[maybe_unused]] bool isSuccessful) override
         {
             AZ::Data::AssetBus::Handler::BusDisconnect();
             AZ::Data::AssetManager::Instance().DispatchEvents();

Gems/Vegetation/Code/Tests/PrefabInstanceSpawnerTests.cpp

@@ -128,7 +128,7 @@ namespace UnitTest
             assetTypes.push_back(AZ::AzTypeInfo<AzFramework::Spawnable>::Uuid());
         }
         AZ::Data::AssetHandler::LoadResult LoadAssetData(
-            const AZ::Data::Asset<AZ::Data::AssetData>& asset,
+            [[maybe_unused]] const AZ::Data::Asset<AZ::Data::AssetData>& asset,
             AZStd::shared_ptr<AZ::Data::AssetDataStream> stream,
             [[maybe_unused]] const AZ::Data::AssetFilterCB& assetLoadFilterCB) override
         {