Changesets: DarkRadiant
master 6217c990 18.06.2021 12:25 Details Diff |
0005643: Add tests covering removal of modified spawnargs as well the modification of removed spawnargs |
Affected Issues 0005643 |
|
mod - test/MapMerging.cpp | Diff File | ||
mod - test/resources/tdm/maps/threeway_merge_source_2.mapx | Diff File | ||
mod - test/resources/tdm/maps/threeway_merge_target_2.mapx | Diff File | ||
master e2fec672 18.06.2021 12:13 Details Diff |
0005643: Add a ConflictType enum to be able to specifically tell what a conflict is about. Expand unit tests by some value change conflicts. |
Affected Issues 0005643 |
|
mod - include/imapmerge.h | Diff File | ||
mod - libs/scene/merge/MergeAction.h | Diff File | ||
mod - libs/scene/merge/ThreeWayMergeOperation.cpp | Diff File | ||
mod - libs/scene/merge/ThreeWayMergeOperation.h | Diff File | ||
mod - test/MapMerging.cpp | Diff File | ||
mod - test/resources/tdm/maps/threeway_merge_source_2.mapx | Diff File | ||
mod - test/resources/tdm/maps/threeway_merge_target_2.mapx | Diff File | ||
master 23cc9213 18.06.2021 09:12 Details Diff |
0005643: Craft the unit test to check the name conflict resolution |
Affected Issues 0005643 |
|
mod - test/MapMerging.cpp | Diff File | ||
master 440a55df 18.06.2021 08:34 Details Diff |
0005643: Restructure the whole process into two phases. Phase 1 will detect any name conflicts that will occur during merge and will adjust the entity names accordingly (using Namespace algorithms). Phase 2 will create the actions necessary to bring the source changes into the target scene. Start preparing the corresponding unit test. |
Affected Issues 0005643 |
|
mod - libs/scene/merge/ThreeWayMergeOperation.cpp | Diff File | ||
mod - libs/scene/merge/ThreeWayMergeOperation.h | Diff File | ||
mod - test/MapMerging.cpp | Diff File | ||
mod - test/resources/tdm/maps/threeway_merge_base.mapx | Diff File | ||
mod - test/resources/tdm/maps/threeway_merge_source_2.mapx | Diff File | ||
mod - test/resources/tdm/maps/threeway_merge_target_2.mapx | Diff File | ||
master 0ab693c9 18.06.2021 08:12 Details Diff |
0005643: Extend INamespace interface by a method that only covers a subset of the to-be-imported scene. |
Affected Issues 0005643 |
|
mod - include/inamespace.h | Diff File | ||
mod - radiantcore/map/namespace/Namespace.cpp | Diff File | ||
mod - radiantcore/map/namespace/Namespace.h | Diff File | ||
master 5cd128a4 16.06.2021 06:05 Details Diff |
Fix a crash on closing the MaterialEditor due to the connection not being cleared. | ||
mod - radiant/ui/materials/editor/MaterialEditor.cpp | Diff File | ||
master 17ac50f5 16.06.2021 04:00 Details Diff |
0005634: Revert the change that auto-fills the clipboard with the picked material name. It is very unhandy when trying to copy&paste elements between maps. |
Affected Issues 0005634 |
|
mod - radiantcore/selection/shaderclipboard/ShaderClipboard.cpp | Diff File | ||
master 54b38fe9 16.06.2021 03:42 Details Diff |
0005643: Another test covering primitive handling of func_static entities |
Affected Issues 0005643 |
|
mod - libs/scene/merge/ThreeWayMergeOperation.cpp | Diff File | ||
mod - test/MapMerging.cpp | Diff File | ||
mod - test/algorithm/Scene.h | Diff File | ||
add - test/resources/tdm/maps/threeway_merge_source_2.mapx | Diff File | ||
add - test/resources/tdm/maps/threeway_merge_target_2.mapx | Diff File | ||
master cb185f94 15.06.2021 19:47 Details Diff |
Add test for NPOT DDS texture loading Load a 10x16 uncompressed DDS file and examine pixel data. This test passes, indicating that there isn't a fundamental inability to load NPOT DDS textures. |
||
mod - test/ImageLoading.cpp | Diff File | ||
add - test/resources/tdm/textures/dds/test_10x16_uncomp.dds | Diff File | ||
master 15b74d5c 15.06.2021 19:31 Details Diff |
Test compressed DDS loading Fix the DDSImage::isPrecompressed() method so it returns the correct result for uncompressed images (which is already known internally) rather than incorrectly assuming every DDS file is compressed. This method is now used in tests to distinguish between compressed and uncompressed images. |
||
mod - radiantcore/imagefile/dds.cpp | Diff File | ||
mod - test/ImageLoading.cpp | Diff File | ||
add - test/resources/tdm/textures/dds/test_128x128_dxt1.dds | Diff File | ||
master 8ddcae8c 15.06.2021 19:13 Details Diff |
Add test for DDS mipmap loading Add a new method Image::getLevels() which returns the number of mipmap levels in a given image. Use this to confirm the expected number of levels when loading DDS images with/without mipmaps. |
||
mod - include/iimage.h | Diff File | ||
mod - libs/RGBAImage.h | Diff File | ||
mod - radiantcore/imagefile/dds.cpp | Diff File | ||
mod - test/ImageLoading.cpp | Diff File | ||
add - test/resources/tdm/textures/dds/test_16x16_uncomp_mips.dds | Diff File | ||
master 3efa9b83 15.06.2021 19:09 Details Diff |
0005643: Add the only seemingly trivial test which performs a three-way merge with the same map used as source and target. Fixup the implementation to get a green test result. |
Affected Issues 0005643 |
|
mod - libs/scene/merge/ComparisonResult.h | Diff File | ||
mod - libs/scene/merge/GraphComparer.cpp | Diff File | ||
mod - libs/scene/merge/ThreeWayMergeOperation.cpp | Diff File | ||
mod - test/MapMerging.cpp | Diff File | ||
master f42738e0 15.06.2021 18:55 Details Diff |
Streamline pixel examination in unit test Introduce a new Pixelator helper class which retrieves individual pixels from an Image using X and Y coordinates, casting the result to a suitable pixel struct. |
||
mod - test/ImageLoading.cpp | Diff File | ||
master 0c2b0d0f 15.06.2021 16:04 Details Diff |
0005643: Non-conflicting changes mostly covered |
Affected Issues 0005643 |
|
mod - test/MapMerging.cpp | Diff File | ||
master c1fa2516 15.06.2021 11:04 Details Diff |
0005643: Test covering spawnarg modifications |
Affected Issues 0005643 |
|
mod - test/MapMerging.cpp | Diff File | ||
master 11ca5a5d 15.06.2021 10:50 Details Diff |
0005643: Fix implementation for merging primitive changes into the target map |
Affected Issues 0005643 |
|
mod - libs/scene/merge/MergeOperation.cpp | Diff File | ||
mod - libs/scene/merge/MergeOperation.h | Diff File | ||
mod - libs/scene/merge/MergeOperationBase.cpp | Diff File | ||
mod - libs/scene/merge/MergeOperationBase.h | Diff File | ||
mod - libs/scene/merge/ThreeWayMergeOperation.cpp | Diff File | ||
mod - libs/scene/merge/ThreeWayMergeOperation.h | Diff File | ||
mod - test/MapMerging.cpp | Diff File | ||
master 8bc51096 15.06.2021 09:50 Details Diff |
0005643: Verify that the changes to the target scene are preserved before and after the merge |
Affected Issues 0005643 |
|
mod - test/MapMerging.cpp | Diff File | ||
master 39f57ac8 15.06.2021 08:10 Details Diff |
0005643: Start setting up the unit tests for the three-way merge scenario |
Affected Issues 0005643 |
|
mod - libs/scene/merge/MergeOperationBase.cpp | Diff File | ||
mod - libs/scene/merge/MergeOperationBase.h | Diff File | ||
mod - libs/scene/merge/ThreeWayMergeOperation.cpp | Diff File | ||
mod - libs/scene/merge/ThreeWayMergeOperation.h | Diff File | ||
mod - test/MapMerging.cpp | Diff File | ||
add - test/resources/tdm/maps/threeway_merge_base.mapx | Diff File | ||
add - test/resources/tdm/maps/threeway_merge_source_1.mapx | Diff File | ||
add - test/resources/tdm/maps/threeway_merge_target_1.mapx | Diff File | ||
master 33d28ff9 15.06.2021 05:09 Details Diff |
0005643: Adjust CMakeLists.txt |
Affected Issues 0005643 |
|
mod - libs/scene/CMakeLists.txt | Diff File | ||
master a35d2f29 15.06.2021 05:05 Details Diff |
0005643: ConflictResolutionActions apply the changes only if explicitly told so by the client code |
Affected Issues 0005643 |
|
mod - libs/scene/merge/MergeAction.h | Diff File | ||
master 3a86bd8d 15.06.2021 04:47 Details Diff |
0005643: Handle primitive removals and addition targeting the same entity |
Affected Issues 0005643 |
|
mod - libs/scene/merge/GraphComparer.cpp | Diff File | ||
mod - libs/scene/merge/GraphComparer.h | Diff File | ||
mod - libs/scene/merge/NodeUtils.h | Diff File | ||
mod - libs/scene/merge/ThreeWayMergeOperation.cpp | Diff File | ||
master 10d3b0ab 15.06.2021 04:14 Details Diff |
0005643: Lots of refactorings and bits of implementation. |
Affected Issues 0005643 |
|
mod - include/imapmerge.h | Diff File | ||
mod - libs/scene/merge/MergeAction.h | Diff File | ||
mod - libs/scene/merge/MergeOperationBase.cpp | Diff File | ||
mod - libs/scene/merge/MergeOperationBase.h | Diff File | ||
mod - libs/scene/merge/ThreeWayMergeOperation.cpp | Diff File | ||
mod - libs/scene/merge/ThreeWayMergeOperation.h | Diff File | ||
master 26bef51e 15.06.2021 03:15 Details Diff |
0005643: Roughly lay out the code paths for the conflict detection code |
Affected Issues 0005643 |
|
mod - libs/scene/merge/GraphComparer.h | Diff File | ||
mod - libs/scene/merge/ThreeWayMergeOperation.cpp | Diff File | ||
mod - libs/scene/merge/ThreeWayMergeOperation.h | Diff File | ||
master 6ea48e27 14.06.2021 15:30 Details Diff |
0005643: Introduce MergeOperationBase class to host the shared code |
Affected Issues 0005643 |
|
mod - libs/scene/merge/MergeOperation.cpp | Diff File | ||
mod - libs/scene/merge/MergeOperation.h | Diff File | ||
add - libs/scene/merge/MergeOperationBase.cpp | Diff File | ||
add - libs/scene/merge/MergeOperationBase.h | Diff File | ||
mod - libs/scene/merge/ThreeWayMergeOperation.cpp | Diff File | ||
mod - libs/scene/merge/ThreeWayMergeOperation.h | Diff File | ||
mod - tools/msvc/scenelib.vcxproj | Diff File | ||
mod - tools/msvc/scenelib.vcxproj.filters | Diff File | ||
master 9d08ce23 14.06.2021 14:08 Details Diff |
0005635: Set focus on the command entry when the console panel is getting focused |
Affected Issues 0005635 |
|
mod - radiant/log/Console.cpp | Diff File | ||
mod - radiant/log/Console.h | Diff File |