Changesets: DarkRadiant
map_merge 07ca6d94 23.05.2021 05:17 Details Diff |
0005622: Patch and Entity fingerprint unit tests. The order of spawnargs on entity nodes do not affect the fingerprint. |
Affected Issues 0005622 |
|
mod - radiantcore/entity/EntityNode.cpp | Diff File | ||
mod - test/MapMerging.cpp | Diff File | ||
mod - test/resources/tdm/maps/fingerprinting.mapx | Diff File | ||
map_merge be56c0e4 23.05.2021 04:38 Details Diff |
0005622: Brush fingerprint unit tests |
Affected Issues 0005622 |
|
mod - test/CMakeLists.txt | Diff File | ||
add - test/MapMerging.cpp | Diff File | ||
add - test/resources/tdm/maps/fingerprinting.mapx | Diff File | ||
mod - tools/msvc/Tests/Tests.vcxproj | Diff File | ||
mod - tools/msvc/Tests/Tests.vcxproj.filters | Diff File | ||
map_merge ffe86cfb 22.05.2021 19:18 Details Diff |
0005622: Implement fingerprinting of entity nodes (layer or grouping information are disregarded) |
Affected Issues 0005622 |
|
mod - radiantcore/entity/EntityNode.cpp | Diff File | ||
mod - radiantcore/entity/EntityNode.h | Diff File | ||
map_merge c33d8839 22.05.2021 19:09 Details Diff |
0005622: Implement fingerprinting of patch nodes (layer or grouping information are disregarded) |
Affected Issues 0005622 |
|
mod - include/icomparablenode.h | Diff File | ||
mod - radiantcore/brush/BrushNode.cpp | Diff File | ||
mod - radiantcore/patch/PatchNode.cpp | Diff File | ||
mod - radiantcore/patch/PatchNode.h | Diff File | ||
map_merge f27c073f 22.05.2021 18:59 Details Diff |
0005622: Implement fingerprinting of brush nodes (layer or grouping information are disregarded) |
Affected Issues 0005622 |
|
mod - radiantcore/brush/BrushNode.cpp | Diff File | ||
mod - radiantcore/brush/BrushNode.h | Diff File | ||
mod - radiantcore/map/algorithm/Import.cpp | Diff File | ||
map_merge d70645d5 22.05.2021 18:58 Details Diff |
0005622: Move hashing code to math namespace |
Affected Issues 0005622 |
|
add - libs/math/Hash.h | Diff File | ||
mod - libs/render/VertexHashing.h | Diff File | ||
mod - tools/msvc/mathlib.vcxproj | Diff File | ||
map_merge 059d6aba 22.05.2021 16:55 Details Diff |
0005622: Basic algorithm to assemble a map fingerprint for all contained nodes |
Affected Issues 0005622 |
|
add - include/icomparablenode.h | Diff File | ||
mod - radiantcore/map/Map.cpp | Diff File | ||
mod - radiantcore/map/algorithm/Import.cpp | Diff File | ||
mod - radiantcore/map/algorithm/Import.h | Diff File | ||
mod - tools/msvc/include.vcxproj | Diff File | ||
map_merge 1a5db5d4 22.05.2021 15:59 Details Diff |
0005622: Rename map::algorithm::mergeMap to map::algorithm::importMap to not collide with the upcoming changes |
Affected Issues 0005622 |
|
mod - radiantcore/map/Map.cpp | Diff File | ||
mod - radiantcore/map/algorithm/Import.cpp | Diff File | ||
mod - radiantcore/map/algorithm/Import.h | Diff File | ||
map_merge 2303d335 22.05.2021 15:54 Details Diff |
0005622: Add empty MergeMap command registration, no algorithm yet. |
Affected Issues 0005622 |
|
mod - install/menu.xml | Diff File | ||
mod - radiantcore/map/Map.cpp | Diff File | ||
mod - radiantcore/map/Map.h | Diff File | ||
master 281397cc 22.05.2021 12:26 Details Diff |
0005620: Add option to Import Prefab dialog to be able to choose whether to recalculate the origin before insertion |
Affected Issues 0005620 |
|
mod - install/user.xml | Diff File | ||
mod - radiant/ui/MapCommands.cpp | Diff File | ||
mod - radiant/ui/prefabselector/PrefabSelector.cpp | Diff File | ||
mod - radiant/ui/prefabselector/PrefabSelector.h | Diff File | ||
master 7f5e5af0 22.05.2021 12:12 Details Diff |
0005620: Add optional parameter to suppress the prefab origin correction |
Affected Issues 0005620 |
|
mod - radiantcore/map/Map.cpp | Diff File | ||
mod - test/Prefabs.cpp | Diff File | ||
master ab1a0a8a 22.05.2021 11:43 Details Diff |
0005608: The GenericEntityNode::revertTransform/freezeTransform methods that were merged GenericEntityNode are not really overrides of the Transformable method inherited with the same name. Remove the override again, and use a private name, like the EClassModelNode is doing. This whole transformation related code is very obscure and full of callbacks and indirections, and (probably) full of duplicated code paths - too much to tackle with this issue. |
Affected Issues 0005608 |
|
mod - radiantcore/entity/generic/GenericEntityNode.cpp | Diff File | ||
mod - radiantcore/entity/generic/GenericEntityNode.h | Diff File | ||
master 24539c95 22.05.2021 11:32 Details Diff |
0005608: Add unit test covering the newly introduce MoveSelected command, as well as the faulty behaviour in the issue at hand |
Affected Issues 0005608 |
|
mod - radiantcore/entity/generic/GenericEntityNode.h | Diff File | ||
mod - test/CMakeLists.txt | Diff File | ||
add - test/Transformation.cpp | Diff File | ||
mod - test/resources/tdm/def/base.def | Diff File | ||
mod - tools/msvc/Tests/Tests.vcxproj | Diff File | ||
mod - tools/msvc/Tests/Tests.vcxproj.filters | Diff File | ||
master 13d07da7 22.05.2021 11:31 Details Diff |
0005608: Add MoveSelected console command, accepting any 3-component vector as translation argument |
Affected Issues 0005608 |
|
mod - radiantcore/selection/algorithm/General.cpp | Diff File | ||
mod - radiantcore/selection/algorithm/Transformation.cpp | Diff File | ||
mod - radiantcore/selection/algorithm/Transformation.h | Diff File | ||
master ed5dc937 22.05.2021 06:48 Details Diff |
0005555: Disable map save progress dialog when updating entities |
Affected Issues 0005555 |
|
mod - plugins/dm.gameconnection/GameConnection.cpp | Diff File | ||
master 66c3df0f 22.05.2021 06:36 Details Diff |
0005555: Avoid double-entering the think() method from external triggers (wxTimer). The lock cannot be placed within the GameConnection::think() body, since think is recursively calling itself by design. |
Affected Issues 0005555 |
|
mod - plugins/dm.gameconnection/GameConnection.cpp | Diff File | ||
mod - plugins/dm.gameconnection/GameConnection.h | Diff File | ||
master dfc0ae4f 19.05.2021 20:09 Details Diff |
Confirm PointTrace can return correct point positions PointTrace has a new method points() which returns the point vector, which is now examined in the unit test. |
Affected Issues 0005603 |
|
mod - libs/scene/PointTrace.h | Diff File | ||
mod - test/PointTrace.cpp | Diff File | ||
master 66b3c078 19.05.2021 19:35 Details Diff |
Remove hard-coded text colouring in Entity Inspector The hard-coded colours assumed a light background, but the default black text is very difficult to see with a dark theme applied. Instead of forcing colours to distinguish entity properties from inherited properties, we now render the inherited properties in italic text while leaving colours to be chosen by the theme. |
||
mod - radiant/ui/einspector/EntityInspector.cpp | Diff File | ||
master 884043d3 18.05.2021 20:13 Details Diff |
forEachPointfile() sorts pointfile names Sorting the names lexicographically should ensure that <mapname>.lin is always first in the list, so the actual leak pointfile will take precedence over any visportal pointfiles. |
Affected Issues 0005603 |
|
mod - radiantcore/map/Map.cpp | Diff File | ||
master 2690984f 18.05.2021 20:05 Details Diff |
PointFile::parse() makes use of forEachPointfile() Since there is not yet any selection GUI, the first pointfile found is used. Currently the pointfile order is undefined if there is more than one present. |
Affected Issues 0005603 |
|
mod - radiantcore/map/PointFile.cpp | Diff File | ||
master dc165ce7 18.05.2021 19:46 Details Diff |
Test enumerated pointfiles have correct names PointfileFunctor now accepts fs::path references rather than bare std::strings, and this information is now recorded and examined in the unit test to ensure the returned pointfiles have the correct filenames (ignoring parent paths). |
Affected Issues 0005603 |
|
mod - include/imap.h | Diff File | ||
mod - test/PointTrace.cpp | Diff File | ||
master e1072c83 18.05.2021 19:33 Details Diff |
Test that pointfiles are not returned for an unrelated map This test actually provoked a bug in the forEachPointfile() method, which was not checking if the map directory actually existed before constructing a directory_iterator, possibly resulting in an exception. |
Affected Issues 0005603 |
|
mod - radiantcore/map/Map.cpp | Diff File | ||
mod - test/PointTrace.cpp | Diff File | ||
master b75c4f20 13.05.2021 17:05 Details Diff |
0005607: Make sure the render system is realised by the time we initialise the preview. |
Affected Issues 0005607 |
|
mod - libs/wxutil/preview/RenderPreview.cpp | Diff File | ||
master a88849ef 13.05.2021 17:02 Details Diff |
0005609: When realising the render system in the constructor, give the code a chance to invoke extensionsInitialised() first. The extensionsInitialised() call will query the GL setup for the ability to use GL programs, which should preferably be known before we realise the OpenGLShader objects. |
Affected Issues 0005609 |
|
mod - radiantcore/rendersystem/OpenGLRenderSystem.cpp | Diff File | ||
master b83c3d72 13.05.2021 17:00 Details Diff |
0005607: In case the rendersystem is not realised yet the contained material in the Shader object will be empty - don't crash when rendering MD5 models in such a situation |
Affected Issues 0005607 |
|
mod - radiantcore/model/md5/MD5ModelNode.cpp | Diff File |