Changesets: DarkRadiant
|
declmanager aa91f536 17.07.2022 11:50 Details Diff |
0005977: Rename MaterialManager interface type to IMaterialManager, implementation is now renamed from Doom3ShaderSystem to MaterialManager. |
Affected Issues 0005977 |
|
| mod - include/ishaders.h | Diff File | ||
| mod - radiantcore/CMakeLists.txt | Diff File | ||
| mod - radiantcore/shaders/CShader.cpp | Diff File | ||
| mod - radiantcore/shaders/Doom3ShaderLayer.cpp | Diff File | ||
| mod - radiantcore/shaders/ShaderExpression.cpp | Diff File | ||
| mod - radiantcore/shaders/TextureMatrix.cpp | Diff File | ||
| mod - radiantcore/shaders/textures/TextureManipulator.cpp | Diff File | ||
| mod - tools/msvc/DarkRadiantCore.vcxproj | Diff File | ||
| mod - tools/msvc/DarkRadiantCore.vcxproj.filters | Diff File | ||
|
declmanager 038da88c 17.07.2022 11:28 Details Diff |
0005977: Remove a bunch of unneeded code, remove ShaderDefinition type |
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/ShaderDefinition.h | Diff | ||
| 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 | ||
| mod - tools/msvc/DarkRadiantCore.vcxproj | Diff File | ||
| mod - tools/msvc/DarkRadiantCore.vcxproj.filters | Diff File | ||
|
declmanager cd10548e 17.07.2022 11:08 Details Diff |
0005977: Remove a couple of signal and realise/unrealise methods from the MaterialManager interface. So far it turns out their absence doesn't make any difference, since the realisation of the OpenGLRenderSystem is depending on other modules which take much longer during startup (i.e. openGL extensionsInitialised). |
Affected Issues 0005977 |
|
| mod - include/ishaders.h | Diff File | ||
| mod - radiant/ui/mediabrowser/MediaBrowser.cpp | Diff File | ||
| mod - radiant/ui/texturebrowser/TextureBrowser.cpp | Diff File | ||
| mod - radiantcore/rendersystem/OpenGLRenderSystem.cpp | Diff File | ||
| mod - radiantcore/rendersystem/OpenGLRenderSystem.h | Diff File | ||
| mod - radiantcore/shaders/Doom3ShaderSystem.cpp | Diff File | ||
| mod - radiantcore/shaders/Doom3ShaderSystem.h | Diff File | ||
| mod - radiantcore/shaders/MaterialSourceGenerator.cpp | Diff File | ||
| mod - radiantcore/shaders/MaterialSourceGenerator.h | Diff File | ||
|
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 bd54305c 17.07.2022 09:01 Details Diff |
0005799: Fix the failing unit test, and add one more test with a single-line scenario |
Affected Issues 0005799 |
|
| mod - radiantcore/decl/DeclarationManager.cpp | Diff File | ||
| 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 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 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 | ||