Cleans up more legacy code from ACE

Rewrote some code to replace gEnv->pCryPak and ISystem usage.  One place
was grabbing a Camera view from ISystem, replaced with simple identity
matrix.  Cleaned up include headers.

Signed-off-by: amzn-phist <52085794+amzn-phist@users.noreply.github.com>
monroegm-disable-blank-issue-2
amzn-phist 4 years ago
parent 5b148b1f40
commit 976272d9cf

@ -13,7 +13,6 @@
#include <AzCore/std/string/conversions.h> #include <AzCore/std/string/conversions.h>
#include <AzCore/StringFunc/StringFunc.h> #include <AzCore/StringFunc/StringFunc.h>
#include <AudioControlsEditorUndo.h> #include <AudioControlsEditorUndo.h>
#include <IEditor.h>
namespace AudioControls namespace AudioControls
{ {

@ -16,13 +16,8 @@
#include <ATLControlsModel.h> #include <ATLControlsModel.h>
#include <AudioControl.h> #include <AudioControl.h>
#include <AudioControlsEditorPlugin.h> #include <AudioControlsEditorPlugin.h>
#include <CryFile.h>
#include <CryPath.h>
#include <Cry_Camera.h>
#include <IAudioSystem.h>
#include <IAudioSystemControl.h> #include <IAudioSystemControl.h>
#include <IAudioSystemEditor.h> #include <IAudioSystemEditor.h>
#include <IEditor.h>
#include <QAudioControlEditorIcons.h> #include <QAudioControlEditorIcons.h>
#include <QWidgetAction> #include <QWidgetAction>

@ -14,7 +14,6 @@
#include <AudioControlsEditorPlugin.h> #include <AudioControlsEditorPlugin.h>
#include <AudioControlsEditorUndo.h> #include <AudioControlsEditorUndo.h>
#include <IAudioSystemControl.h> #include <IAudioSystemControl.h>
#include <IEditor.h>
#include <ImplementationManager.h> #include <ImplementationManager.h>
namespace AudioControls namespace AudioControls

@ -14,9 +14,6 @@
#include <AudioControlsLoader.h> #include <AudioControlsLoader.h>
#include <AudioControlsWriter.h> #include <AudioControlsWriter.h>
#include <CryFile.h>
#include <CryPath.h>
#include <Cry_Camera.h>
#include <Include/IResourceSelectorHost.h> #include <Include/IResourceSelectorHost.h>
#include <IAudioSystem.h> #include <IAudioSystem.h>
@ -28,7 +25,6 @@
using namespace AudioControls; using namespace AudioControls;
using namespace PathUtil;
CATLControlsModel CAudioControlsEditorPlugin::ms_ATLModel; CATLControlsModel CAudioControlsEditorPlugin::ms_ATLModel;
QATLTreeModel CAudioControlsEditorPlugin::ms_layoutModel; QATLTreeModel CAudioControlsEditorPlugin::ms_layoutModel;
@ -152,20 +148,18 @@ void CAudioControlsEditorPlugin::ExecuteTrigger(const AZStd::string_view sTrigge
Audio::AudioSystemRequestBus::BroadcastResult(ms_nAudioTriggerID, &Audio::AudioSystemRequestBus::Events::GetAudioTriggerID, sTriggerName.data()); Audio::AudioSystemRequestBus::BroadcastResult(ms_nAudioTriggerID, &Audio::AudioSystemRequestBus::Events::GetAudioTriggerID, sTriggerName.data());
if (ms_nAudioTriggerID != INVALID_AUDIO_CONTROL_ID) if (ms_nAudioTriggerID != INVALID_AUDIO_CONTROL_ID)
{ {
const CCamera& camera = GetIEditor()->GetSystem()->GetViewCamera();
Audio::SAudioRequest request; Audio::SAudioRequest request;
request.nFlags = Audio::eARF_PRIORITY_NORMAL; request.nFlags = Audio::eARF_PRIORITY_NORMAL;
const AZ::Matrix3x4 cameraMatrix = LYTransformToAZMatrix3x4(camera.GetMatrix()); const AZ::Matrix3x4 listenerTxfm = AZ::Matrix3x4::CreateIdentity();
Audio::SAudioListenerRequestData<Audio::eALRT_SET_POSITION> requestData(cameraMatrix); Audio::SAudioListenerRequestData<Audio::eALRT_SET_POSITION> requestData(listenerTxfm);
requestData.oNewPosition.NormalizeForwardVec(); requestData.oNewPosition.NormalizeForwardVec();
requestData.oNewPosition.NormalizeUpVec(); requestData.oNewPosition.NormalizeUpVec();
request.pData = &requestData; request.pData = &requestData;
Audio::AudioSystemRequestBus::Broadcast(&Audio::AudioSystemRequestBus::Events::PushRequest, request); Audio::AudioSystemRequestBus::Broadcast(&Audio::AudioSystemRequestBus::Events::PushRequest, request);
ms_pIAudioProxy->SetPosition(cameraMatrix); ms_pIAudioProxy->SetPosition(listenerTxfm);
ms_pIAudioProxy->ExecuteTrigger(ms_nAudioTriggerID); ms_pIAudioProxy->ExecuteTrigger(ms_nAudioTriggerID);
} }
} }

@ -9,7 +9,6 @@
#include <AudioControlsEditorWindow.h> #include <AudioControlsEditorWindow.h>
#include <AzCore/StringFunc/StringFunc.h>
#include <AzCore/Utils/Utils.h> #include <AzCore/Utils/Utils.h>
#include <AzToolsFramework/API/ToolsApplicationAPI.h> #include <AzToolsFramework/API/ToolsApplicationAPI.h>
@ -18,22 +17,14 @@
#include <ATLControlsPanel.h> #include <ATLControlsPanel.h>
#include <AudioControlsEditorPlugin.h> #include <AudioControlsEditorPlugin.h>
#include <AudioControlsEditorUndo.h> #include <AudioControlsEditorUndo.h>
#include <AudioFileUtils.h>
#include <AudioSystemPanel.h> #include <AudioSystemPanel.h>
#include <DockTitleBarWidget.h>
#include <IAudioSystem.h> #include <IAudioSystem.h>
#include <ImplementationManager.h> #include <ImplementationManager.h>
#include <InspectorPanel.h> #include <InspectorPanel.h>
#include <CryFile.h>
#include <CryPath.h>
#include <ISystem.h>
#include <Util/PathUtil.h>
#include <QAudioControlEditorIcons.h> #include <QAudioControlEditorIcons.h>
#include <DockTitleBarWidget.h>
#include <QPaintEvent> #include <QPaintEvent>
#include <QPushButton> #include <QPushButton>
@ -41,6 +32,7 @@
#include <QPainter> #include <QPainter>
#include <QMessageBox> #include <QMessageBox>
void InitACEResources() void InitACEResources()
{ {
Q_INIT_RESOURCE(AudioControlsEditorUI); Q_INIT_RESOURCE(AudioControlsEditorUI);
@ -116,25 +108,16 @@ namespace AudioControls
{ {
m_fileSystemWatcher.addPath(folder.data()); m_fileSystemWatcher.addPath(folder.data());
AZStd::string search; auto fileIO = AZ::IO::FileIOBase::GetInstance();
AZ::StringFunc::Path::Join(folder.data(), "*", search, true, false); auto foundFiles = Audio::FindFilesInPath(folder, "*");
auto pCryPak = gEnv->pCryPak; for (auto& file : foundFiles)
AZ::IO::ArchiveFileIterator handle = pCryPak->FindFirst(search.c_str());
if (handle)
{
do
{ {
AZStd::string sName = static_cast<AZStd::string_view>(handle.m_filename); if (fileIO->IsDirectory(file.c_str()))
if (!sName.empty() && sName[0] != '.')
{ {
if ((handle.m_fileDesc.nAttrib & AZ::IO::FileDesc::Attribute::Subdirectory) == AZ::IO::FileDesc::Attribute::Subdirectory) AZ::IO::FixedMaxPath resolvedPath;
{ fileIO->ReplaceAlias(resolvedPath, file);
AZ::StringFunc::Path::Join(folder.data(), sName.c_str(), sName); StartWatchingFolder(file.Native());
StartWatchingFolder(sName);
}
} }
} while (handle = pCryPak->FindNext(handle));
pCryPak->FindClose(handle);
} }
} }

@ -12,7 +12,6 @@
#if !defined(Q_MOC_RUN) #if !defined(Q_MOC_RUN)
#include <ATLControlsModel.h> #include <ATLControlsModel.h>
#include <AzCore/std/string/string_view.h> #include <AzCore/std/string/string_view.h>
#include <IEditor.h>
#include <QMainWindow> #include <QMainWindow>
#include <QFileSystemWatcher> #include <QFileSystemWatcher>

@ -10,7 +10,6 @@
#include <AudioControlsLoader.h> #include <AudioControlsLoader.h>
#include <AzCore/std/string/conversions.h> #include <AzCore/std/string/conversions.h>
#include <AzCore/StringFunc/StringFunc.h>
#include <AzCore/Utils/Utils.h> #include <AzCore/Utils/Utils.h>
#include <ACEEnums.h> #include <ACEEnums.h>

@ -9,7 +9,6 @@
#include <ATLControlsResourceDialog.h> #include <ATLControlsResourceDialog.h>
#include <AudioControlsEditorPlugin.h> #include <AudioControlsEditorPlugin.h>
#include <IEditor.h>
#include <Include/IResourceSelectorHost.h> #include <Include/IResourceSelectorHost.h>
#include <QAudioControlEditorIcons.h> #include <QAudioControlEditorIcons.h>
#include <AzToolsFramework/API/ToolsApplicationAPI.h> #include <AzToolsFramework/API/ToolsApplicationAPI.h>

@ -12,10 +12,7 @@
#include <ATLControlsModel.h> #include <ATLControlsModel.h>
#include <AudioControl.h> #include <AudioControl.h>
#include <AudioControlsEditorPlugin.h> #include <AudioControlsEditorPlugin.h>
#include <CryFile.h>
#include <CryPath.h>
#include <IAudioSystemEditor.h> #include <IAudioSystemEditor.h>
#include <IEditor.h>
#include <QAudioControlEditorIcons.h> #include <QAudioControlEditorIcons.h>
#include <QWidgetAction> #include <QWidgetAction>

@ -14,8 +14,6 @@
#include <ATLControlsModel.h> #include <ATLControlsModel.h>
#include <AudioControlsEditorPlugin.h> #include <AudioControlsEditorPlugin.h>
#include <IAudioSystemEditor.h> #include <IAudioSystemEditor.h>
#include <IConsole.h>
#include <IEditor.h>
//-----------------------------------------------------------------------------------------------// //-----------------------------------------------------------------------------------------------//

@ -10,9 +10,6 @@
#pragma once #pragma once
#if !defined(Q_MOC_RUN) #if !defined(Q_MOC_RUN)
#include <AzCore/std/smart_ptr/unique_ptr.h>
#include <AzCore/Module/DynamicModuleHandle.h>
#include <QObject> #include <QObject>
#endif #endif

@ -12,7 +12,6 @@
#include <ACETypes.h> #include <ACETypes.h>
#include <AudioControlsEditorPlugin.h> #include <AudioControlsEditorPlugin.h>
#include <IAudioSystemControl.h> #include <IAudioSystemControl.h>
#include <IEditor.h>
#include <QAudioControlEditorIcons.h> #include <QAudioControlEditorIcons.h>
#include <QMessageBox> #include <QMessageBox>

@ -14,7 +14,6 @@
#include <AudioControlsEditorPlugin.h> #include <AudioControlsEditorPlugin.h>
#include <IAudioSystemControl.h> #include <IAudioSystemControl.h>
#include <IAudioSystemEditor.h> #include <IAudioSystemEditor.h>
#include <IEditor.h>
#include <ImplementationManager.h> #include <ImplementationManager.h>
#include <QDropEvent> #include <QDropEvent>

Loading…
Cancel
Save