Changesets: DarkRadiant
declmanager ed847374 17.07.2022 09:01 Details Diff |
0005977: Fix the failing unit test, and add one more test with a single-line scenario |
Affected Issues 0005977 |
|
mod - radiantcore/decl/DeclarationManager.cpp | Diff File | ||
mod - test/DeclManager.cpp | Diff File | ||
mod - test/resources/tdm/testdecls/numbers.decl | Diff File | ||
declmanager 7cab1def 17.07.2022 08:57 Details Diff |
0005799: Add unit test covering IDeclarationManager::saveDeclaration not being able to find the declaration with a commment after the opening brace. |
Affected Issues 0005799 |
|
mod - test/DeclManager.cpp | Diff File | ||
mod - test/resources/tdm/testdecls/numbers.decl | Diff File | ||
declmanager 6f224d19 17.07.2022 08:57 Details Diff |
0005977: Add unit test covering IDeclarationManager::saveDeclaration not being able to find the declaration with a commment after the opening brace. |
Affected Issues 0005977 |
|
mod - test/DeclManager.cpp | Diff File | ||
mod - test/resources/tdm/testdecls/numbers.decl | Diff File | ||
declmanager 43c84201 16.07.2022 14:47 Details Diff |
0005977: Cleanup. Migrate MaterialManager::saveMaterial to use the declaration manager interface instead of the home-grown algorithm. The material export test is failing now due to the trailing single-line comment in the .mtr file. |
Affected Issues 0005977 |
|
mod - radiantcore/shaders/CShader.cpp | Diff File | ||
mod - radiantcore/shaders/CShader.h | Diff File | ||
mod - radiantcore/shaders/Doom3ShaderSystem.cpp | Diff File | ||
mod - radiantcore/shaders/Doom3ShaderSystem.h | Diff File | ||
rm - radiantcore/shaders/ShaderFileLoader.h | Diff | ||
mod - radiantcore/shaders/ShaderLibrary.cpp | Diff File | ||
mod - radiantcore/shaders/ShaderLibrary.h | Diff File | ||
mod - radiantcore/shaders/ShaderTemplate.cpp | Diff File | ||
mod - tools/msvc/DarkRadiantCore.vcxproj | Diff File | ||
mod - tools/msvc/DarkRadiantCore.vcxproj.filters | Diff File | ||
declmanager c1161ddc 16.07.2022 13:48 Details Diff |
0005977: Remove old vfs::FileInfo member from CShader |
Affected Issues 0005977 |
|
mod - radiantcore/shaders/CShader.cpp | Diff File | ||
mod - radiantcore/shaders/CShader.h | Diff File | ||
declmanager 4ca47f89 16.07.2022 13:46 Details Diff |
0005977: EditableDeclaration constructor is now protected, declare the default copy constructor and make actual use of it when cloning ShaderTemplates. |
Affected Issues 0005977 |
|
mod - libs/decl/DeclarationBase.h | Diff File | ||
mod - libs/decl/EditableDeclaration.h | Diff File | ||
mod - radiantcore/shaders/CShader.cpp | Diff File | ||
mod - radiantcore/shaders/ShaderTemplate.cpp | Diff File | ||
declmanager 061ce5d4 16.07.2022 10:36 Details Diff |
0005977: Add IDeclaration::setFileInfo method, since we need the ability to set the target path quite a few times |
Affected Issues 0005977 |
|
mod - include/ideclmanager.h | Diff File | ||
mod - libs/decl/DeclarationBase.h | Diff File | ||
mod - radiantcore/particles/ParticleDef.h | Diff File | ||
mod - radiantcore/shaders/Doom3ShaderSystem.cpp | Diff File | ||
mod - radiantcore/shaders/ShaderLibrary.cpp | Diff File | ||
mod - test/DeclManager.cpp | Diff File | ||
mod - test/Particles.cpp | Diff File | ||
declmanager 01ac3ae2 16.07.2022 09:33 Details Diff |
0005977: Migrate more MaterialManager methods |
Affected Issues 0005977 |
|
mod - radiantcore/shaders/Doom3ShaderSystem.cpp | Diff File | ||
mod - radiantcore/shaders/ShaderLibrary.cpp | Diff File | ||
mod - radiantcore/shaders/ShaderLibrary.h | Diff File | ||
declmanager 44cc8106 16.07.2022 09:24 Details Diff |
0005977: Implement IDeclarationManager::renameDeclaration |
Affected Issues 0005977 |
|
mod - radiantcore/decl/DeclarationManager.cpp | Diff File | ||
declmanager fc2340e6 15.07.2022 16:58 Details Diff |
0005977: Add IDeclarationManager::renameDeclaration method and unit tests (no implementation yet). |
Affected Issues 0005977 |
|
mod - include/ideclmanager.h | Diff File | ||
mod - radiantcore/decl/DeclarationManager.cpp | Diff File | ||
mod - radiantcore/decl/DeclarationManager.h | Diff File | ||
mod - test/DeclManager.cpp | Diff File | ||
declmanager 218228e4 15.07.2022 16:34 Details Diff |
0005977: Disable the old ShaderFileLoader code. Re-implement the ShaderLibrary::copyDefinition method. |
Affected Issues 0005977 |
|
mod - radiantcore/shaders/Doom3ShaderSystem.cpp | Diff File | ||
mod - radiantcore/shaders/Doom3ShaderSystem.h | Diff File | ||
mod - radiantcore/shaders/ShaderFileLoader.h | Diff File | ||
mod - radiantcore/shaders/ShaderLibrary.cpp | Diff File | ||
declmanager bc39715c 14.07.2022 16:21 Details Diff |
0005977: Let Declarations define their own set of delimiters when parsing the def blocks. As long as I can't confirm that the set is valid for all types of declarations, I'll stick to what worked up to now and keep the virtual. |
Affected Issues 0005977 |
|
mod - libs/decl/DeclarationBase.h | Diff File | ||
mod - radiantcore/shaders/ShaderTemplate.h | Diff File | ||
declmanager fdfed79b 14.07.2022 15:26 Details Diff |
0005977: Start adjusting the ShaderTemplate/ShaderDefinition classes to fit into the Declaration framework. |
Affected Issues 0005977 |
|
mod - include/ishaders.h | Diff File | ||
mod - libs/parser/DefTokeniser.h | Diff File | ||
mod - radiantcore/shaders/CShader.cpp | Diff File | ||
mod - radiantcore/shaders/CShader.h | Diff File | ||
mod - radiantcore/shaders/Doom3ShaderSystem.cpp | Diff File | ||
mod - radiantcore/shaders/Doom3ShaderSystem.h | Diff File | ||
mod - radiantcore/shaders/ShaderLibrary.cpp | Diff File | ||
mod - radiantcore/shaders/ShaderLibrary.h | Diff File | ||
mod - radiantcore/shaders/ShaderTemplate.cpp | Diff File | ||
mod - radiantcore/shaders/ShaderTemplate.h | Diff File | ||
master d2cc12a6 13.07.2022 19:45 Details Diff |
0000401: 0-argument executeCommand() overload removed This overload can be provided implicitly via a default empty argument list parameter to the list-based method. |
Affected Issues 0000410 |
|
mod - include/icommandsystem.h | Diff File | ||
master dfaa3ec6 13.07.2022 19:26 Details Diff |
0000401: 3-argument executeCommand() overload removed Now that we have C++11 initializer lists, it is much easier to construct an argument list inline as "{first, second, third}", making this rarely-used overload less necessary. |
Affected Issues 0000410 |
|
mod - include/icommandsystem.h | Diff File | ||
mod - radiant/ui/modelselector/ModelSelector.h | Diff File | ||
mod - radiant/ui/ortho/OrthoContextMenu.cpp | Diff File | ||
mod - radiant/xyview/tools/BrushCreatorTool.cpp | Diff File | ||
mod - test/Brush.cpp | Diff File | ||
mod - test/Prefabs.cpp | Diff File | ||
master 403841f7 13.07.2022 19:09 Details Diff |
0000401: executeCommand() overloads are no longer virtual The executeCommand() overloads which accept 0-3 parameters do not need separate back-end implementations, since they are just constructing an ArgumentList and passing it on to the list-based overload. These are now non-virtual utility functions implemented in ICommandSystem, while the CommandSystem implementing class only needs to implement the list-based method. |
Affected Issues 0000410 |
|
mod - include/icommandsystem.h | Diff File | ||
mod - radiantcore/commandsystem/CommandSystem.cpp | Diff File | ||
mod - radiantcore/commandsystem/CommandSystem.h | Diff File | ||
master 57244bd3 12.07.2022 19:58 Details Diff |
0000401: test passing Vector2 and Vector3 args to a command |
Affected Issues 0000410 |
|
mod - test/CommandSystem.cpp | Diff File | ||
master 5f4715f9 12.07.2022 19:33 Details Diff |
0000401: refactor common code in CommandSystem test Re-introduce a local struct (TestCommandReceiver) to encapsulate the command name, run count and last received args, instead of needing to explicitly declare these in each test function. |
Affected Issues 0000410 |
|
mod - test/CommandSystem.cpp | Diff File | ||
declmanager 301aa150 10.07.2022 17:44 Details Diff |
0005977: Migrated the table parsing code |
Affected Issues 0005977 |
|
mod - include/idecltypes.h | Diff File | ||
mod - include/ishaders.h | Diff File | ||
mod - radiantcore/shaders/Doom3ShaderSystem.cpp | Diff File | ||
mod - radiantcore/shaders/ShaderExpression.h | Diff File | ||
mod - radiantcore/shaders/ShaderFileLoader.h | Diff File | ||
mod - radiantcore/shaders/ShaderLibrary.cpp | Diff File | ||
mod - radiantcore/shaders/ShaderLibrary.h | Diff File | ||
mod - radiantcore/shaders/TableDefinition.cpp | Diff File | ||
mod - radiantcore/shaders/TableDefinition.h | Diff File | ||
mod - test/Materials.cpp | Diff File | ||
declmanager 8bc1f0db 10.07.2022 15:54 Details Diff |
0005977: Remove named EntityClass constructor and set the eclass default colour to DefaultEntityColour in the constructor, this effectively produces the same result as the static constructor. |
Affected Issues 0005977 |
|
rm - radiantcore/eclass/DefCreators.h | Diff | ||
mod - radiantcore/eclass/EClassManager.cpp | Diff File | ||
mod - radiantcore/eclass/EntityClass.cpp | Diff File | ||
mod - tools/msvc/DarkRadiantCore.vcxproj | Diff File | ||
mod - tools/msvc/DarkRadiantCore.vcxproj.filters | Diff File | ||
declmanager f05d2383 10.07.2022 15:48 Details Diff |
0005977: Merge declaration creators into one decl::DeclarationCreator<> template |
Affected Issues 0005977 |
|
add - libs/decl/DeclarationCreator.h | Diff File | ||
mod - plugins/sound/SoundManager.cpp | Diff File | ||
rm - plugins/sound/SoundShaderParser.h | Diff | ||
mod - radiantcore/eclass/DefCreators.h | Diff File | ||
mod - radiantcore/eclass/EClassManager.cpp | Diff File | ||
rm - radiantcore/particles/ParticleDefCreator.h | Diff | ||
mod - radiantcore/particles/ParticlesManager.cpp | Diff File | ||
mod - radiantcore/skins/Doom3SkinCache.cpp | Diff File | ||
rm - radiantcore/skins/SkinCreator.h | Diff | ||
mod - tools/msvc/DarkRadiantCore.vcxproj | Diff File | ||
mod - tools/msvc/DarkRadiantCore.vcxproj.filters | Diff File | ||
mod - tools/msvc/libs.vcxproj | Diff File | ||
mod - tools/msvc/libs.vcxproj.filters | Diff File | ||
mod - tools/msvc/sound.vcxproj | Diff File | ||
mod - tools/msvc/sound.vcxproj.filters | Diff File | ||
declmanager fc33ff61 10.07.2022 15:33 Details Diff |
0006002: Replace the old ParticlesManager::saveParticleDef algorithm with a call to IDeclarationManager::saveDeclaration. This also resolves the request to remove the comments above the particle declarations. |
Affected Issues 0006002 |
|
mod - include/iparticles.h | Diff File | ||
mod - radiantcore/particles/ParticlesManager.cpp | Diff File | ||
declmanager 84f8ae10 10.07.2022 15:27 Details Diff |
0005977: Move DeclarationBase and EditableDeclaration to libs/decl/ folder |
Affected Issues 0005977 |
|
mod - plugins/sound/SoundShader.h | Diff File | ||
mod - radiantcore/eclass/Doom3ModelDef.h | Diff File | ||
mod - radiantcore/eclass/EntityClass.h | Diff File | ||
mod - radiantcore/particles/ParticleDef.h | Diff File | ||
mod - radiantcore/skins/Doom3ModelSkin.h | Diff File | ||
mod - test/DeclManager.cpp | Diff File | ||
mod - tools/msvc/libs.vcxproj | Diff File | ||
mod - tools/msvc/libs.vcxproj.filters | Diff File | ||
declmanager cb040b0f 10.07.2022 15:23 Details Diff |
0005977: Add unit tests covering the various typename variants when replacing declarations |
Affected Issues 0005977 |
|
mod - radiantcore/decl/DeclarationManager.cpp | Diff File | ||
mod - test/DeclManager.cpp | Diff File | ||
declmanager 2aef403b 10.07.2022 15:05 Details Diff |
0005977: Stabilise the code, adjust unit tests assertions, simplify SpliceHelper (this breaks the particle and material editor output for the moment being) |
Affected Issues 0005977 |
|
mod - libs/decl/SpliceHelper.h | Diff File | ||
mod - radiantcore/decl/DeclarationManager.cpp | Diff File | ||
mod - test/DeclManager.cpp | Diff File |