From d6c530d08cb6dfb7e5e2c670a05bdb9c3509c7f9 Mon Sep 17 00:00:00 2001 From: tgsm Date: Tue, 19 Feb 2019 03:09:57 -0500 Subject: [PATCH] video_core: use nested namespaces --- src/video_core/command_processor.cpp | 8 ++------ src/video_core/command_processor.h | 8 ++------ src/video_core/rasterizer_interface.h | 6 ++---- src/video_core/shader/debug_data.h | 6 ++---- src/video_core/shader/shader.cpp | 8 ++------ src/video_core/shader/shader.h | 8 ++------ src/video_core/shader/shader_interpreter.cpp | 8 ++------ src/video_core/shader/shader_interpreter.h | 8 ++------ src/video_core/shader/shader_jit_x64.cpp | 6 ++---- src/video_core/shader/shader_jit_x64.h | 6 ++---- src/video_core/shader/shader_jit_x64_compiler.cpp | 8 ++------ src/video_core/shader/shader_jit_x64_compiler.h | 8 ++------ src/video_core/swrasterizer/clipper.cpp | 6 ++---- src/video_core/swrasterizer/framebuffer.cpp | 6 ++---- src/video_core/swrasterizer/framebuffer.h | 6 ++---- src/video_core/swrasterizer/proctex.cpp | 6 ++---- src/video_core/swrasterizer/proctex.h | 6 ++---- src/video_core/swrasterizer/rasterizer.cpp | 6 ++---- src/video_core/swrasterizer/rasterizer.h | 6 ++---- src/video_core/swrasterizer/swrasterizer.h | 6 ++---- src/video_core/swrasterizer/texturing.cpp | 6 ++---- src/video_core/swrasterizer/texturing.h | 6 ++---- src/video_core/texture/etc1.cpp | 6 ++---- src/video_core/texture/etc1.h | 6 ++---- src/video_core/texture/texture_decode.cpp | 6 ++---- src/video_core/texture/texture_decode.h | 6 ++---- 26 files changed, 52 insertions(+), 120 deletions(-) diff --git a/src/video_core/command_processor.cpp b/src/video_core/command_processor.cpp index 5810537b9..475035af7 100644 --- a/src/video_core/command_processor.cpp +++ b/src/video_core/command_processor.cpp @@ -28,9 +28,7 @@ #include "video_core/vertex_loader.h" #include "video_core/video_core.h" -namespace Pica { - -namespace CommandProcessor { +namespace Pica::CommandProcessor { static int vs_float_regs_counter = 0; static u32 vs_uniform_write_buffer[4]; @@ -675,6 +673,4 @@ void ProcessCommandList(const u32* list, u32 size) { } } -} // namespace CommandProcessor - -} // namespace Pica +} // namespace Pica::CommandProcessor diff --git a/src/video_core/command_processor.h b/src/video_core/command_processor.h index c8f639e53..82b154327 100644 --- a/src/video_core/command_processor.h +++ b/src/video_core/command_processor.h @@ -8,9 +8,7 @@ #include "common/bit_field.h" #include "common/common_types.h" -namespace Pica { - -namespace CommandProcessor { +namespace Pica::CommandProcessor { union CommandHeader { u32 hex; @@ -36,6 +34,4 @@ static_assert(sizeof(CommandHeader) == sizeof(u32), "CommandHeader has incorrect void ProcessCommandList(const u32* list, u32 size); -} // namespace CommandProcessor - -} // namespace Pica +} // namespace Pica::CommandProcessor diff --git a/src/video_core/rasterizer_interface.h b/src/video_core/rasterizer_interface.h index b4d237c61..7222e97ce 100644 --- a/src/video_core/rasterizer_interface.h +++ b/src/video_core/rasterizer_interface.h @@ -11,11 +11,9 @@ namespace OpenGL { struct ScreenInfo; } -namespace Pica { -namespace Shader { +namespace Pica::Shader { struct OutputVertex; -} -} // namespace Pica +} // namespace Pica::Shader namespace VideoCore { diff --git a/src/video_core/shader/debug_data.h b/src/video_core/shader/debug_data.h index 9e82122e1..f75d8d893 100644 --- a/src/video_core/shader/debug_data.h +++ b/src/video_core/shader/debug_data.h @@ -9,8 +9,7 @@ #include "common/vector_math.h" #include "video_core/pica_types.h" -namespace Pica { -namespace Shader { +namespace Pica::Shader { /// Helper structure used to keep track of data useful for inspection of shader emulation template @@ -182,5 +181,4 @@ inline void Record(DebugData& debug_data, u32 offset, ValueType value) { debug_data.records[offset].mask |= type; } -} // namespace Shader -} // namespace Pica +} // namespace Pica::Shader diff --git a/src/video_core/shader/shader.cpp b/src/video_core/shader/shader.cpp index 7bf97e3fc..a94fe388a 100644 --- a/src/video_core/shader/shader.cpp +++ b/src/video_core/shader/shader.cpp @@ -17,9 +17,7 @@ #endif // ARCHITECTURE_x86_64 #include "video_core/video_core.h" -namespace Pica { - -namespace Shader { +namespace Pica::Shader { void OutputVertex::ValidateSemantics(const RasterizerRegs& regs) { unsigned int num_attributes = regs.vs_output_total; @@ -161,6 +159,4 @@ void Shutdown() { #endif // ARCHITECTURE_x86_64 } -} // namespace Shader - -} // namespace Pica +} // namespace Pica::Shader diff --git a/src/video_core/shader/shader.h b/src/video_core/shader/shader.h index 6340abddd..d44ccb8d1 100644 --- a/src/video_core/shader/shader.h +++ b/src/video_core/shader/shader.h @@ -22,9 +22,7 @@ using nihstro::DestRegister; using nihstro::RegisterType; using nihstro::SourceRegister; -namespace Pica { - -namespace Shader { +namespace Pica::Shader { constexpr unsigned MAX_PROGRAM_CODE_LENGTH = 4096; constexpr unsigned MAX_SWIZZLE_DATA_LENGTH = 4096; @@ -262,6 +260,4 @@ public: ShaderEngine* GetEngine(); void Shutdown(); -} // namespace Shader - -} // namespace Pica +} // namespace Pica::Shader diff --git a/src/video_core/shader/shader_interpreter.cpp b/src/video_core/shader/shader_interpreter.cpp index 6d001c709..f94feda88 100644 --- a/src/video_core/shader/shader_interpreter.cpp +++ b/src/video_core/shader/shader_interpreter.cpp @@ -25,9 +25,7 @@ using nihstro::RegisterType; using nihstro::SourceRegister; using nihstro::SwizzlePattern; -namespace Pica { - -namespace Shader { +namespace Pica::Shader { struct CallStackElement { u32 final_address; // Address upon which we jump to return_address @@ -696,6 +694,4 @@ DebugData InterpreterEngine::ProduceDebugInfo(const ShaderSetup& setup, return debug_data; } -} // namespace Shader - -} // namespace Pica +} // namespace Pica::Shader diff --git a/src/video_core/shader/shader_interpreter.h b/src/video_core/shader/shader_interpreter.h index c2a7121ce..978f1e34b 100644 --- a/src/video_core/shader/shader_interpreter.h +++ b/src/video_core/shader/shader_interpreter.h @@ -7,9 +7,7 @@ #include "video_core/shader/debug_data.h" #include "video_core/shader/shader.h" -namespace Pica { - -namespace Shader { +namespace Pica::Shader { class InterpreterEngine final : public ShaderEngine { public: @@ -27,6 +25,4 @@ public: const ShaderRegs& config) const; }; -} // namespace Shader - -} // namespace Pica +} // namespace Pica::Shader diff --git a/src/video_core/shader/shader_jit_x64.cpp b/src/video_core/shader/shader_jit_x64.cpp index 696fe11da..e70d6fdad 100644 --- a/src/video_core/shader/shader_jit_x64.cpp +++ b/src/video_core/shader/shader_jit_x64.cpp @@ -7,8 +7,7 @@ #include "video_core/shader/shader_jit_x64.h" #include "video_core/shader/shader_jit_x64_compiler.h" -namespace Pica { -namespace Shader { +namespace Pica::Shader { JitX64Engine::JitX64Engine() = default; JitX64Engine::~JitX64Engine() = default; @@ -43,5 +42,4 @@ void JitX64Engine::Run(const ShaderSetup& setup, UnitState& state) const { shader->Run(setup, state, setup.engine_data.entry_point); } -} // namespace Shader -} // namespace Pica +} // namespace Pica::Shader diff --git a/src/video_core/shader/shader_jit_x64.h b/src/video_core/shader/shader_jit_x64.h index 078b2cba5..3f6b52cd6 100644 --- a/src/video_core/shader/shader_jit_x64.h +++ b/src/video_core/shader/shader_jit_x64.h @@ -9,8 +9,7 @@ #include "common/common_types.h" #include "video_core/shader/shader.h" -namespace Pica { -namespace Shader { +namespace Pica::Shader { class JitShader; @@ -26,5 +25,4 @@ private: std::unordered_map> cache; }; -} // namespace Shader -} // namespace Pica +} // namespace Pica::Shader diff --git a/src/video_core/shader/shader_jit_x64_compiler.cpp b/src/video_core/shader/shader_jit_x64_compiler.cpp index 42af3ee28..d49181144 100644 --- a/src/video_core/shader/shader_jit_x64_compiler.cpp +++ b/src/video_core/shader/shader_jit_x64_compiler.cpp @@ -26,9 +26,7 @@ using Xbyak::Reg32; using Xbyak::Reg64; using Xbyak::Xmm; -namespace Pica { - -namespace Shader { +namespace Pica::Shader { typedef void (JitShader::*JitFunction)(Instruction instr); @@ -1123,6 +1121,4 @@ Xbyak::Label JitShader::CompilePrelude_Exp2() { return subroutine; } -} // namespace Shader - -} // namespace Pica +} // namespace Pica::Shader diff --git a/src/video_core/shader/shader_jit_x64_compiler.h b/src/video_core/shader/shader_jit_x64_compiler.h index 7a10844c7..bdf569632 100644 --- a/src/video_core/shader/shader_jit_x64_compiler.h +++ b/src/video_core/shader/shader_jit_x64_compiler.h @@ -19,9 +19,7 @@ using nihstro::Instruction; using nihstro::OpCode; using nihstro::SwizzlePattern; -namespace Pica { - -namespace Shader { +namespace Pica::Shader { /// Memory allocated for each compiled shader constexpr std::size_t MAX_SHADER_SIZE = MAX_PROGRAM_CODE_LENGTH * 64; @@ -138,6 +136,4 @@ private: Xbyak::Label exp2_subroutine; }; -} // namespace Shader - -} // namespace Pica +} // namespace Pica::Shader diff --git a/src/video_core/swrasterizer/clipper.cpp b/src/video_core/swrasterizer/clipper.cpp index 79e61edf7..752d8ab1a 100644 --- a/src/video_core/swrasterizer/clipper.cpp +++ b/src/video_core/swrasterizer/clipper.cpp @@ -19,8 +19,7 @@ using Pica::Rasterizer::Vertex; -namespace Pica { -namespace Clipper { +namespace Pica::Clipper { struct ClippingEdge { public: @@ -192,5 +191,4 @@ void ProcessTriangle(const OutputVertex& v0, const OutputVertex& v1, const Outpu } } -} // namespace Clipper -} // namespace Pica +} // namespace Pica::Clipper diff --git a/src/video_core/swrasterizer/framebuffer.cpp b/src/video_core/swrasterizer/framebuffer.cpp index 1365a9835..c97f68ced 100644 --- a/src/video_core/swrasterizer/framebuffer.cpp +++ b/src/video_core/swrasterizer/framebuffer.cpp @@ -16,8 +16,7 @@ #include "video_core/utils.h" #include "video_core/video_core.h" -namespace Pica { -namespace Rasterizer { +namespace Pica::Rasterizer { void DrawPixel(int x, int y, const Math::Vec4& color) { const auto& framebuffer = g_state.regs.framebuffer.framebuffer; @@ -407,5 +406,4 @@ void DrawShadowMapPixel(int x, int y, u32 depth, u8 stencil) { } } -} // namespace Rasterizer -} // namespace Pica +} // namespace Pica::Rasterizer diff --git a/src/video_core/swrasterizer/framebuffer.h b/src/video_core/swrasterizer/framebuffer.h index 0a1c42669..7c73e3f30 100644 --- a/src/video_core/swrasterizer/framebuffer.h +++ b/src/video_core/swrasterizer/framebuffer.h @@ -8,8 +8,7 @@ #include "common/vector_math.h" #include "video_core/regs_framebuffer.h" -namespace Pica { -namespace Rasterizer { +namespace Pica::Rasterizer { void DrawPixel(int x, int y, const Math::Vec4& color); const Math::Vec4 GetPixel(int x, int y); @@ -27,5 +26,4 @@ u8 LogicOp(u8 src, u8 dest, FramebufferRegs::LogicOp op); void DrawShadowMapPixel(int x, int y, u32 depth, u8 stencil); -} // namespace Rasterizer -} // namespace Pica +} // namespace Pica::Rasterizer diff --git a/src/video_core/swrasterizer/proctex.cpp b/src/video_core/swrasterizer/proctex.cpp index d4f47a841..c9517a5e1 100644 --- a/src/video_core/swrasterizer/proctex.cpp +++ b/src/video_core/swrasterizer/proctex.cpp @@ -7,8 +7,7 @@ #include "common/math_util.h" #include "video_core/swrasterizer/proctex.h" -namespace Pica { -namespace Rasterizer { +namespace Pica::Rasterizer { using ProcTexClamp = TexturingRegs::ProcTexClamp; using ProcTexShift = TexturingRegs::ProcTexShift; @@ -219,5 +218,4 @@ Math::Vec4 ProcTex(float u, float v, TexturingRegs regs, State::ProcTex stat } } -} // namespace Rasterizer -} // namespace Pica +} // namespace Pica::Rasterizer diff --git a/src/video_core/swrasterizer/proctex.h b/src/video_core/swrasterizer/proctex.h index 036e4620e..308fc4a2e 100644 --- a/src/video_core/swrasterizer/proctex.h +++ b/src/video_core/swrasterizer/proctex.h @@ -6,11 +6,9 @@ #include "common/vector_math.h" #include "video_core/pica_state.h" -namespace Pica { -namespace Rasterizer { +namespace Pica::Rasterizer { /// Generates procedural texture color for the given coordinates Math::Vec4 ProcTex(float u, float v, TexturingRegs regs, State::ProcTex state); -} // namespace Rasterizer -} // namespace Pica +} // namespace Pica::Rasterizer diff --git a/src/video_core/swrasterizer/rasterizer.cpp b/src/video_core/swrasterizer/rasterizer.cpp index 52650926d..bc62e628d 100644 --- a/src/video_core/swrasterizer/rasterizer.cpp +++ b/src/video_core/swrasterizer/rasterizer.cpp @@ -32,8 +32,7 @@ #include "video_core/utils.h" #include "video_core/video_core.h" -namespace Pica { -namespace Rasterizer { +namespace Pica::Rasterizer { // NOTE: Assuming that rasterizer coordinates are 12.4 fixed-point values struct Fix12P4 { @@ -893,5 +892,4 @@ void ProcessTriangle(const Vertex& v0, const Vertex& v1, const Vertex& v2) { ProcessTriangleInternal(v0, v1, v2); } -} // namespace Rasterizer -} // namespace Pica +} // namespace Pica::Rasterizer diff --git a/src/video_core/swrasterizer/rasterizer.h b/src/video_core/swrasterizer/rasterizer.h index 0670776a9..b54cf7225 100644 --- a/src/video_core/swrasterizer/rasterizer.h +++ b/src/video_core/swrasterizer/rasterizer.h @@ -6,8 +6,7 @@ #include "video_core/shader/shader.h" -namespace Pica { -namespace Rasterizer { +namespace Pica::Rasterizer { struct Vertex : Shader::OutputVertex { Vertex(const OutputVertex& v) : OutputVertex(v) {} @@ -42,5 +41,4 @@ struct Vertex : Shader::OutputVertex { void ProcessTriangle(const Vertex& v0, const Vertex& v1, const Vertex& v2); -} // namespace Rasterizer -} // namespace Pica +} // namespace Pica::Rasterizer diff --git a/src/video_core/swrasterizer/swrasterizer.h b/src/video_core/swrasterizer/swrasterizer.h index f97fe13b4..e2292f4a4 100644 --- a/src/video_core/swrasterizer/swrasterizer.h +++ b/src/video_core/swrasterizer/swrasterizer.h @@ -7,11 +7,9 @@ #include "common/common_types.h" #include "video_core/rasterizer_interface.h" -namespace Pica { -namespace Shader { +namespace Pica::Shader { struct OutputVertex; -} -} // namespace Pica +} // namespace Pica::Shader namespace VideoCore { diff --git a/src/video_core/swrasterizer/texturing.cpp b/src/video_core/swrasterizer/texturing.cpp index 03612a7b0..e4cde3635 100644 --- a/src/video_core/swrasterizer/texturing.cpp +++ b/src/video_core/swrasterizer/texturing.cpp @@ -9,8 +9,7 @@ #include "video_core/regs_texturing.h" #include "video_core/swrasterizer/texturing.h" -namespace Pica { -namespace Rasterizer { +namespace Pica::Rasterizer { using TevStageConfig = TexturingRegs::TevStageConfig; @@ -238,5 +237,4 @@ u8 AlphaCombine(TevStageConfig::Operation op, const std::array& input) { } }; -} // namespace Rasterizer -} // namespace Pica +} // namespace Pica::Rasterizer diff --git a/src/video_core/swrasterizer/texturing.h b/src/video_core/swrasterizer/texturing.h index 24f74a5a3..f0382b2a9 100644 --- a/src/video_core/swrasterizer/texturing.h +++ b/src/video_core/swrasterizer/texturing.h @@ -8,8 +8,7 @@ #include "common/vector_math.h" #include "video_core/regs_texturing.h" -namespace Pica { -namespace Rasterizer { +namespace Pica::Rasterizer { int GetWrappedTexCoord(TexturingRegs::TextureConfig::WrapMode mode, int val, unsigned size); @@ -24,5 +23,4 @@ Math::Vec3 ColorCombine(TexturingRegs::TevStageConfig::Operation op, u8 AlphaCombine(TexturingRegs::TevStageConfig::Operation op, const std::array& input); -} // namespace Rasterizer -} // namespace Pica +} // namespace Pica::Rasterizer diff --git a/src/video_core/texture/etc1.cpp b/src/video_core/texture/etc1.cpp index 10bb06501..ad383372d 100644 --- a/src/video_core/texture/etc1.cpp +++ b/src/video_core/texture/etc1.cpp @@ -10,8 +10,7 @@ #include "common/vector_math.h" #include "video_core/texture/etc1.h" -namespace Pica { -namespace Texture { +namespace Pica::Texture { namespace { @@ -125,5 +124,4 @@ Math::Vec3 SampleETC1Subtile(u64 value, unsigned int x, unsigned int y) { return tile.GetRGB(x, y); } -} // namespace Texture -} // namespace Pica +} // namespace Pica::Texture diff --git a/src/video_core/texture/etc1.h b/src/video_core/texture/etc1.h index e188b19df..39273b4cd 100644 --- a/src/video_core/texture/etc1.h +++ b/src/video_core/texture/etc1.h @@ -7,10 +7,8 @@ #include "common/common_types.h" #include "common/vector_math.h" -namespace Pica { -namespace Texture { +namespace Pica::Texture { Math::Vec3 SampleETC1Subtile(u64 value, unsigned int x, unsigned int y); -} // namespace Texture -} // namespace Pica +} // namespace Pica::Texture diff --git a/src/video_core/texture/texture_decode.cpp b/src/video_core/texture/texture_decode.cpp index 4032cabda..7121a8aac 100644 --- a/src/video_core/texture/texture_decode.cpp +++ b/src/video_core/texture/texture_decode.cpp @@ -15,8 +15,7 @@ using TextureFormat = Pica::TexturingRegs::TextureFormat; -namespace Pica { -namespace Texture { +namespace Pica::Texture { constexpr std::size_t TILE_SIZE = 8 * 8; constexpr std::size_t ETC1_SUBTILES = 2 * 2; @@ -223,5 +222,4 @@ TextureInfo TextureInfo::FromPicaRegister(const TexturingRegs::TextureConfig& co return info; } -} // namespace Texture -} // namespace Pica +} // namespace Pica::Texture diff --git a/src/video_core/texture/texture_decode.h b/src/video_core/texture/texture_decode.h index 8507cfeb8..c5e64f675 100644 --- a/src/video_core/texture/texture_decode.h +++ b/src/video_core/texture/texture_decode.h @@ -8,8 +8,7 @@ #include "common/vector_math.h" #include "video_core/regs_texturing.h" -namespace Pica { -namespace Texture { +namespace Pica::Texture { /// Returns the byte size of a 8*8 tile of the specified texture format. size_t CalculateTileSize(TexturingRegs::TextureFormat format); @@ -56,5 +55,4 @@ Math::Vec4 LookupTexture(const u8* source, unsigned int x, unsigned int y, Math::Vec4 LookupTexelInTile(const u8* source, unsigned int x, unsigned int y, const TextureInfo& info, bool disable_alpha); -} // namespace Texture -} // namespace Pica +} // namespace Pica::Texture