Page MenuHomeWildfire Games

Atlas: Allow map to recenter during resize
ClosedPublic

Authored by Stan on Aug 22 2017, 1:10 AM.

Details

Summary

Allows maps to have the center changed during resizing.
For maps being shrunk, this allows the desired, remaining, portion of the map to be chosen:

For maps being enlarged, this allows the user to choose where the original map data should be placed:

Test Plan

Use provided maps to verify that function chooses the correct location and decal/unit mapping.
(should be placed in binaries/data/mods/public/maps/scenarios)

Diff Detail

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes
vladislavbelov added inline comments.Oct 5 2019, 12:13 AM
source/tools/atlas/GameInterface/Handlers/MapHandlers.cpp
402

No, I mean to initialize them in ctor by nullptr.

Stan updated this revision to Diff 10564.Dec 11 2019, 5:25 PM

Add nullptr initialization

Owners added subscribers: Restricted Owners Package, Restricted Owners Package, Restricted Owners Package.Dec 11 2019, 5:25 PM

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1253/display/redirect

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/737/display/redirect

Stan updated this revision to Diff 10565.Dec 11 2019, 5:27 PM

nullptr

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1254/display/redirect

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/738/display/redirect

Stan updated this revision to Diff 10566.Dec 11 2019, 5:32 PM

nullptr

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1255/display/redirect

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/739/display/redirect

Stan updated this revision to Diff 10567.Dec 11 2019, 5:57 PM

Fix include.

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1256/display/redirect

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/740/display/redirect

Years. Also it's important to have a test for the new terrain functionality. But I suggest to split the patch and change the CTerrain and tests in a separate patch.

source/graphics/Terrain.cpp
510

I think it should be std::abs(horizontalOffset) >= (size + m_MapSizePatches) / 2 (same for vertical) if we also want to have a little piece of the source.

549

I'm not sure that +1 is needed here.

554

I think src[0] is better than *src as it's more explicit.

556

src[(width * PATCH_SIZE + 1) - 1]

source/graphics/Terrain.h
103

Different arguments types in definition and implementation. I think we need to use ssize_t, since we do most calculations using ssize_t.

source/gui/ObjectTypes/CMiniMap.cpp
77 ↗(On Diff #10567)

Is CMinimap or CCmpPathfinder present on Atlas start? Because only they add that validator.

source/tools/atlas/AtlasUI/CustomControls/MapResizeDialog/MapResizeDialog.cpp
23

I think this empty line isn't necessary.

53

Instead of wxString(s["Name"]) we use wxString::FromUTF8(s["Name"]) (see Map.cpp or Terrain.cpp).

54

For size we use (*s["Tiles"]).getLong().

97

Maybe?

wxString str = static_cast<wxStringClientData*>(evt.GetClientObject())->GetData();
m_NewSize = str.getLong();
99

Not necessary parenthesis.

source/tools/atlas/AtlasUI/CustomControls/MapResizeDialog/MapResizeDialog.h
22

Empty line between system and game headers.

33

Empty line I think.

source/tools/atlas/AtlasUI/CustomControls/MapResizeDialog/PseudoMiniMapPanel.cpp
61

Not needed line.

65

Maybe imageBytes = std::move(*qryBackground.imageBytes)?

68

We use new, no?

96

We have duplication here. We need only one place to parse the size.

source/tools/atlas/AtlasUI/CustomControls/MapResizeDialog/PseudoMiniMapPanel.h
52

I think ssize_t instead of int like in other places.

source/tools/atlas/GameInterface/Handlers/MapHandlers.cpp
285

std::vector<u8> imageBytes(buf_size);.

341
msg->imageBytes = std::move(imageBytes);
397
cResizeMap() : m_Heightmap(nullptr), m_Patches(nullptr)
{
}
Stan marked an inline comment as done.Jan 15 2020, 4:23 PM
Stan added inline comments.
source/graphics/Terrain.h
103

I think I tried that, and it failed, so I reverted to int.

source/gui/ObjectTypes/CMiniMap.cpp
77 ↗(On Diff #10567)

Pathfinder I assume for the overlay you can enable. Minimap only if you start the simulation.

source/tools/atlas/AtlasUI/CustomControls/MapResizeDialog/MapResizeDialog.cpp
97

Advantages?

99

?

source/tools/atlas/AtlasUI/CustomControls/MapResizeDialog/PseudoMiniMapPanel.cpp
65

Can you explain? I'm always struggling with this.

68

Dunno? You're the expert.

96

Duplication with where?

source/tools/atlas/AtlasUI/CustomControls/MapResizeDialog/PseudoMiniMapPanel.h
52

See comment above. Or am I dumb and maybe I used size_t which is not signed hence breaking the feature?

Stan updated this revision to Diff 11050.Jan 16 2020, 1:01 PM
Stan marked 23 inline comments as done.

Fix notes

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/157/display/redirect

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1579/display/redirect

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/1061/display/redirect

Stan updated this revision to Diff 11051.Jan 16 2020, 2:26 PM

Try to fix build

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/158/display/redirect

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1580/display/redirect

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/1062/display/redirect

Stan updated this revision to Diff 11052.Jan 16 2020, 2:36 PM

int → ssize_t

source/tools/atlas/AtlasUI/CustomControls/MapResizeDialog/MapResizeDialog.cpp
97

Had to tweak it.

source/tools/atlas/AtlasUI/CustomControls/MapResizeDialog/PseudoMiniMapPanel.cpp
65

Reverted because

../../../source/tools/atlas/AtlasUI/CustomControls/MapResizeDialog/PseudoMiniMapPanel.cpp:63:36: warning: moving a temporary object prevents copy elision [-Wpessimizing-move]

        std::vector<uint8_t> imageBytes = std::move(*qryBackground.imageBytes);

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/1063/display/redirect

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/159/display/redirect

Successful build - Chance fights ever on the side of the prudent.

Linter detected issues:
Executing section Source...

source/graphics/Terrain.h
|   1| /*·Copyright·(C)·2019·Wildfire·Games.
|    | [NORMAL] LicenseYearBear:
|    | License should have "2020" year instead of "2019"

source/graphics/Terrain.h
|  51| class·CTerrain
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classCTerrain{' is invalid C code. Use --std or --language to configure the language.

source/tools/atlas/AtlasUI/ScenarioEditor/Sections/Terrain/Terrain.h
|   1| /*·Copyright·(C)·2019·Wildfire·Games.
|    | [NORMAL] LicenseYearBear:
|    | License should have "2020" year instead of "2019"

source/tools/atlas/AtlasUI/ScenarioEditor/Sections/Terrain/Terrain.h
|  23| {
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classSidebar:' is invalid C code. Use --std or --language to configure the language.

source/tools/atlas/AtlasUI/ScenarioEditor/Sections/Terrain/Terrain.cpp
|   1| /*·Copyright·(C)·2019·Wildfire·Games.
|    | [NORMAL] LicenseYearBear:
|    | License should have "2020" year instead of "2019"

source/tools/atlas/AtlasUI/ScenarioEditor/Sections/Terrain/Terrain.cpp
| 166| BEGIN_EVENT_TABLE(TexturePreviewPanel,·wxPanel)
|    | [MAJOR] CPPCheckBear (unknownMacro):
|    | There is an unknown macro here somewhere. Configuration is required. If BEGIN_EVENT_TABLE is a macro then please configure it.

source/graphics/Terrain.cpp
|   1| /*·Copyright·(C)·2019·Wildfire·Games.
|    | [NORMAL] LicenseYearBear:
|    | License should have "2020" year instead of "2019"

source/gui/ObjectTypes/CMiniMap.h
|   1| /*·Copyright·(C)·2019·Wildfire·Games.
|    | [NORMAL] LicenseYearBear:
|    | License should have "2020" year instead of "2019"

source/gui/ObjectTypes/CMiniMap.h
|  29| class·CMiniMap·:·public·IGUIObject
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classCMiniMap:' is invalid C code. Use --std or --language to configure the language.

source/tools/atlas/AtlasUI/CustomControls/MapResizeDialog/PseudoMiniMapPanel.h
|  25| class·PseudoMiniMapPanel·:·public·wxPanel
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classPseudoMiniMapPanel:' is invalid C code. Use --std or --language to configure the language.

source/gui/ObjectTypes/CMiniMap.cpp
|   1| /*·Copyright·(C)·2019·Wildfire·Games.
|    | [NORMAL] LicenseYearBear:
|    | License should have "2020" year instead of "2019"

source/tools/atlas/AtlasUI/ScenarioEditor/Sections/Map/Map.h
|   1| /*·Copyright·(C)·2019·Wildfire·Games.
|    | [NORMAL] LicenseYearBear:
|    | License should have "2020" year instead of "2019"

source/tools/atlas/AtlasUI/ScenarioEditor/Sections/Map/Map.h
|  23| 
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classSidebar:' is invalid C code. Use --std or --language to configure the language.

source/tools/atlas/AtlasUI/CustomControls/MapResizeDialog/MapResizeDialog.h
|  25| class·MapResizeDialog·:·public·wxDialog
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classPseudoMiniMapPanel:' is invalid C code. Use --std or --language to configure the language.

source/tools/atlas/GameInterface/Messages.h
|   1| /*·Copyright·(C)·2019·Wildfire·Games.
|    | [NORMAL] LicenseYearBear:
|    | License should have "2020" year instead of "2019"

source/tools/atlas/GameInterface/Messages.h
|  25| #include·<vector>
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'namespaceAtlasMessage{' is invalid C code. Use --std or --language to configure the language.

source/tools/atlas/GameInterface/Messages.h
| 211| »   »   ((std::vector<std::string>,·data))
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Unmatched '}'. Configuration: 'MESSAGESSETUP_NOTFIRST'.

source/tools/atlas/AtlasUI/ScenarioEditor/Sections/Map/Map.cpp
|   1| /*·Copyright·(C)·2019·Wildfire·Games.
|    | [NORMAL] LicenseYearBear:
|    | License should have "2020" year instead of "2019"

source/tools/atlas/GameInterface/Handlers/MapHandlers.cpp
|   1| /*·Copyright·(C)·2019·Wildfire·Games.
|    | [NORMAL] LicenseYearBear:
|    | License should have "2020" year instead of "2019"

source/tools/atlas/GameInterface/Handlers/MapHandlers.cpp
| 544| »   »   »   »   m_DeletedObjects.push_back(deleted);
|    | [MAJOR] CPPCheckBear (uninitStructMember):
|    | Uninitialized struct member: deleted.owner

source/tools/atlas/GameInterface/Handlers/MapHandlers.cpp
| 211| »   »   cmpTerrain->ReloadTerrain();
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Unmatched '}'. Configuration: 'MESSAGESSETUP_NOTFIRST'.
Executing section JS...
Executing section cli...

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1581/display/redirect

Stan updated this revision to Diff 11812.May 8 2020, 11:20 AM

Rebase after D2724

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/670/display/redirect

Stan updated this revision to Diff 11813.May 8 2020, 11:36 AM

Fix a warning introduced by D2724 / rP23639

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/671/display/redirect

Successful build - Chance fights ever on the side of the prudent.

Linter detected issues:
Executing section Source...

source/tools/atlas/AtlasUI/CustomControls/MapResizeDialog/MapResizeDialog.h
|  25| class·MapResizeDialog·:·public·wxDialog
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classPseudoMiniMapPanel:' is invalid C code. Use --std or --language to configure the language.

source/tools/atlas/AtlasUI/ScenarioEditor/Sections/Map/Map.h
|  23| 
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classSidebar:' is invalid C code. Use --std or --language to configure the language.

source/gui/ObjectTypes/CMiniMap.h
|   1| /*·Copyright·(C)·2019·Wildfire·Games.
|    | [NORMAL] LicenseYearBear:
|    | License should have "2020" year instead of "2019"

source/gui/ObjectTypes/CMiniMap.h
|  29| class·CMiniMap·:·public·IGUIObject
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classCMiniMap:' is invalid C code. Use --std or --language to configure the language.

source/tools/atlas/AtlasUI/ScenarioEditor/Sections/Terrain/Terrain.cpp
| 166| BEGIN_EVENT_TABLE(TexturePreviewPanel,·wxPanel)
|    | [MAJOR] CPPCheckBear (unknownMacro):
|    | There is an unknown macro here somewhere. Configuration is required. If BEGIN_EVENT_TABLE is a macro then please configure it.

source/tools/atlas/AtlasUI/CustomControls/MapResizeDialog/PseudoMiniMapPanel.h
|  25| class·PseudoMiniMapPanel·:·public·wxPanel
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classPseudoMiniMapPanel:' is invalid C code. Use --std or --language to configure the language.

source/gui/ObjectTypes/CMiniMap.cpp
|   1| /*·Copyright·(C)·2019·Wildfire·Games.
|    | [NORMAL] LicenseYearBear:
|    | License should have "2020" year instead of "2019"

source/tools/atlas/AtlasUI/ScenarioEditor/Sections/Terrain/Terrain.h
|  23| {
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classSidebar:' is invalid C code. Use --std or --language to configure the language.

source/tools/atlas/GameInterface/Messages.h
|   1| /*·Copyright·(C)·2019·Wildfire·Games.
|    | [NORMAL] LicenseYearBear:
|    | License should have "2020" year instead of "2019"

source/tools/atlas/GameInterface/Messages.h
|  25| #include·<vector>
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'namespaceAtlasMessage{' is invalid C code. Use --std or --language to configure the language.

source/tools/atlas/GameInterface/Messages.h
| 211| »   »   ((std::vector<std::string>,·data))
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Unmatched '}'. Configuration: 'MESSAGESSETUP_NOTFIRST'.

source/tools/atlas/GameInterface/Handlers/MapHandlers.cpp
| 545| »   »   »   »   m_DeletedObjects.push_back(deleted);
|    | [MAJOR] CPPCheckBear (uninitStructMember):
|    | Uninitialized struct member: deleted.owner

source/tools/atlas/GameInterface/Handlers/MapHandlers.cpp
| 211| »   CmpPtr<ICmpTerrain>·cmpTerrain(*g_Game->GetSimulation2(),·SYSTEM_ENTITY);
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Unmatched '}'. Configuration: 'MESSAGESSETUP_NOTFIRST'.
Executing section JS...
Executing section cli...

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/2085/display/redirect

Successful build - Chance fights ever on the side of the prudent.

Linter detected issues:
Executing section Source...

source/tools/atlas/AtlasUI/CustomControls/MapResizeDialog/MapResizeDialog.h
|  25| class·MapResizeDialog·:·public·wxDialog
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classPseudoMiniMapPanel:' is invalid C code. Use --std or --language to configure the language.

source/tools/atlas/AtlasUI/CustomControls/MapResizeDialog/PseudoMiniMapPanel.h
|  25| class·PseudoMiniMapPanel·:·public·wxPanel
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classPseudoMiniMapPanel:' is invalid C code. Use --std or --language to configure the language.

source/tools/atlas/AtlasUI/ScenarioEditor/Sections/Terrain/Terrain.cpp
| 166| BEGIN_EVENT_TABLE(TexturePreviewPanel,·wxPanel)
|    | [MAJOR] CPPCheckBear (unknownMacro):
|    | There is an unknown macro here somewhere. Configuration is required. If BEGIN_EVENT_TABLE is a macro then please configure it.

source/tools/atlas/GameInterface/Messages.h
|   1| /*·Copyright·(C)·2019·Wildfire·Games.
|    | [NORMAL] LicenseYearBear:
|    | License should have "2020" year instead of "2019"

source/tools/atlas/GameInterface/Messages.h
|  25| #include·<vector>
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'namespaceAtlasMessage{' is invalid C code. Use --std or --language to configure the language.

source/tools/atlas/GameInterface/Messages.h
| 211| »   »   ((std::vector<std::string>,·data))
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Unmatched '}'. Configuration: 'MESSAGESSETUP_NOTFIRST'.

source/tools/atlas/AtlasUI/ScenarioEditor/Sections/Terrain/Terrain.h
|  23| {
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classSidebar:' is invalid C code. Use --std or --language to configure the language.

source/tools/atlas/AtlasUI/ScenarioEditor/Sections/Map/Map.h
|  23| 
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classSidebar:' is invalid C code. Use --std or --language to configure the language.

source/gui/ObjectTypes/CMiniMap.h
|   1| /*·Copyright·(C)·2019·Wildfire·Games.
|    | [NORMAL] LicenseYearBear:
|    | License should have "2020" year instead of "2019"

source/gui/ObjectTypes/CMiniMap.h
|  29| class·CMiniMap·:·public·IGUIObject
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classCMiniMap:' is invalid C code. Use --std or --language to configure the language.

source/gui/ObjectTypes/CMiniMap.cpp
|   1| /*·Copyright·(C)·2019·Wildfire·Games.
|    | [NORMAL] LicenseYearBear:
|    | License should have "2020" year instead of "2019"

source/tools/atlas/GameInterface/Handlers/MapHandlers.cpp
| 545| »   »   »   »   m_DeletedObjects.push_back(deleted);
|    | [MAJOR] CPPCheckBear (uninitStructMember):
|    | Uninitialized struct member: deleted.owner

source/tools/atlas/GameInterface/Handlers/MapHandlers.cpp
| 211| »   CmpPtr<ICmpTerrain>·cmpTerrain(*g_Game->GetSimulation2(),·SYSTEM_ENTITY);
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Unmatched '}'. Configuration: 'MESSAGESSETUP_NOTFIRST'.
Executing section JS...
Executing section cli...

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/2086/display/redirect

Stan updated this revision to Diff 12589.Jul 8 2020, 9:19 AM

Rebase following rP23807

Vulcan added a comment.Jul 8 2020, 9:38 AM

Successful build - Chance fights ever on the side of the prudent.

Linter detected issues:
Executing section Source...

source/tools/atlas/AtlasUI/CustomControls/MapResizeDialog/MapResizeDialog.h
|  25| class·MapResizeDialog·:·public·wxDialog
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classPseudoMiniMapPanel:' is invalid C code. Use --std or --language to configure the language.

source/gui/ObjectTypes/CMiniMap.h
|   1| /*·Copyright·(C)·2019·Wildfire·Games.
|    | [NORMAL] LicenseYearBear:
|    | License should have "2020" year instead of "2019"

source/gui/ObjectTypes/CMiniMap.h
|  29| class·CMiniMap·:·public·IGUIObject
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classCMiniMap:' is invalid C code. Use --std or --language to configure the language.

source/tools/atlas/AtlasUI/ScenarioEditor/Sections/Map/Map.h
|  23| 
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classSidebar:' is invalid C code. Use --std or --language to configure the language.

source/tools/atlas/AtlasUI/ScenarioEditor/Sections/Terrain/Terrain.cpp
| 166| BEGIN_EVENT_TABLE(TexturePreviewPanel,·wxPanel)
|    | [MAJOR] CPPCheckBear (unknownMacro):
|    | There is an unknown macro here somewhere. Configuration is required. If BEGIN_EVENT_TABLE is a macro then please configure it.

source/tools/atlas/GameInterface/Messages.h
|   1| /*·Copyright·(C)·2019·Wildfire·Games.
|    | [NORMAL] LicenseYearBear:
|    | License should have "2020" year instead of "2019"

source/tools/atlas/GameInterface/Messages.h
|  25| #include·<vector>
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'namespaceAtlasMessage{' is invalid C code. Use --std or --language to configure the language.

source/tools/atlas/GameInterface/Messages.h
| 211| »   »   ((std::vector<std::string>,·data))
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Unmatched '}'. Configuration: 'MESSAGESSETUP_NOTFIRST'.

source/tools/atlas/AtlasUI/ScenarioEditor/Sections/Terrain/Terrain.h
|  23| {
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classSidebar:' is invalid C code. Use --std or --language to configure the language.

source/tools/atlas/AtlasUI/CustomControls/MapResizeDialog/PseudoMiniMapPanel.h
|  25| class·PseudoMiniMapPanel·:·public·wxPanel
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Code 'classPseudoMiniMapPanel:' is invalid C code. Use --std or --language to configure the language.

source/tools/atlas/GameInterface/Handlers/MapHandlers.cpp
| 545| »   »   »   »   m_DeletedObjects.push_back(deleted);
|    | [MAJOR] CPPCheckBear (uninitStructMember):
|    | Uninitialized struct member: deleted.owner

source/tools/atlas/GameInterface/Handlers/MapHandlers.cpp
| 211| »   CmpPtr<ICmpTerrain>·cmpTerrain(*g_Game->GetSimulation2(),·SYSTEM_ENTITY);
|    | [MAJOR] CPPCheckBear (syntaxError):
|    | Unmatched '}'. Configuration: 'MESSAGESSETUP_NOTFIRST'.
Executing section JS...
Executing section cli...

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/2597/display/redirect

This revision was not accepted when it landed; it landed in state Needs Review.Jul 21 2020, 4:09 AM
This revision was automatically updated to reflect the committed changes.

@Clockwork-Muse, sorry for the long review and thank you for the patch.