Changesets: DarkRadiant

map_merge 07ca6d94

23.05.2021 05:17

greebo


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

greebo


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

greebo


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

greebo


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

greebo


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

greebo


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

greebo


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

greebo


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

greebo


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

greebo


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

greebo


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

greebo


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

greebo


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

greebo


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

greebo


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

greebo


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

orbweaver


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

orbweaver


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

orbweaver


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

orbweaver


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

orbweaver


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

orbweaver


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

greebo


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

greebo


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

greebo


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
 First  Prev  1 2 3 ... 30 ... 60 ... 90 ... 120 ... 127 128 129 130 131 132 133 ... 150 ... 180 ... 210 ... 240 ... 261 262 263  Next  Last