Skip to content

Maps/Vmaps and MMaps Digit change#360

Open
billy1arm wants to merge 10 commits into
masterfrom
Map_Digits_Change_3_to_4
Open

Maps/Vmaps and MMaps Digit change#360
billy1arm wants to merge 10 commits into
masterfrom
Map_Digits_Change_3_to_4

Conversation

@billy1arm

@billy1arm billy1arm commented Jun 8, 2026

Copy link
Copy Markdown
Member

all files will need to be extracted after this update


This change is Reviewable

Copilot AI review requested due to automatic review settings June 8, 2026 20:42
@codacy-production

codacy-production Bot commented Jun 8, 2026

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates map/vmap/mmap file naming conventions to use 4-digit zero-padded map IDs (e.g., 0001 instead of 001) across runtime loaders, generators, and GM commands to align with updated extraction outputs.

Changes:

  • Switched mmap .mmap and .mmtile filename formatting from 3-digit to 4-digit map IDs.
  • Switched terrain .map filename formatting from 3-digit to 4-digit map IDs.
  • Switched vmap .vmtree/.vmtile filename formatting from 3-digit to 4-digit map IDs.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
src/game/WorldHandlers/MoveMap.cpp Updates mmap .mmap/.mmtile filename patterns and related logs to 4-digit map IDs.
src/game/WorldHandlers/GridMap.cpp Updates terrain .map filename patterns to 4-digit map IDs.
src/game/vmap/VMapManager2.cpp Updates vmap runtime map tree filename padding to 4 digits.
src/game/vmap/TileAssembler.cpp Updates vmap extractor output filenames (.vmtree/.vmtile) to 4 digits.
src/game/vmap/MapTree.cpp Updates vmap tile filename generation to 4 digits.
src/game/ChatCommands/MMapCommands.cpp Updates .mmap loc output formatting to 4-digit map IDs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/game/WorldHandlers/MoveMap.cpp Outdated
Comment thread src/game/WorldHandlers/MoveMap.cpp Outdated
Comment thread src/game/WorldHandlers/MoveMap.cpp
Comment thread src/game/WorldHandlers/MoveMap.cpp Outdated
Comment thread src/game/WorldHandlers/MoveMap.cpp
Comment thread src/game/WorldHandlers/MoveMap.cpp Outdated
Comment thread src/game/WorldHandlers/MoveMap.cpp Outdated
Comment thread src/game/WorldHandlers/MoveMap.cpp Outdated
Comment thread src/game/WorldHandlers/GridMap.cpp Outdated
@AppVeyorBot

Copy link
Copy Markdown

@AppVeyorBot

Copy link
Copy Markdown

Copilot AI review requested due to automatic review settings June 9, 2026 08:15
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 7 comments.

Comment thread src/game/WorldHandlers/MoveMap.cpp Outdated
Comment thread src/game/WorldHandlers/MoveMap.cpp Outdated
Comment thread src/game/WorldHandlers/MoveMap.cpp Outdated
Comment thread src/game/ChatCommands/MMapCommands.cpp Outdated
Comment thread src/game/WorldHandlers/GridMap.cpp Outdated
Comment thread src/game/WorldHandlers/MoveMap.cpp Outdated
Comment thread src/game/WorldHandlers/MoveMap.cpp Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 9, 2026 08:20
billy1arm and others added 3 commits June 9, 2026 09:21
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 5 comments.

Comment thread src/game/WorldHandlers/MoveMap.cpp Outdated
Comment thread src/game/WorldHandlers/MoveMap.cpp Outdated
Comment thread src/game/WorldHandlers/MoveMap.cpp Outdated
{
--loadedTiles;
DEBUG_FILTER_LOG(LOG_FILTER_MAP_LOADING, "MMAP:unloadMap: Unloaded mmtile %03i[%02i,%02i] from %03i", mapId, x, y, mapId);
DEBUG_FILTER_LOG(LOG_FILTER_MAP_LOADING, "MMAP:unloadMap: Unloaded mmtile %04u[%02i,%02i] from %04i", mapId, x, y, mapId);
Comment on lines +877 to +879
int len = sWorld.GetDataPath().length() + strlen("maps/%04u%02u%02u.map") + 1;
char* tmp = new char[len];
snprintf(tmp, len, (char*)(sWorld.GetDataPath() + "maps/%03u%02u%02u.map").c_str(), mapid, gx, gy);
snprintf(tmp, len, (char*)(sWorld.GetDataPath() + "maps/%04u%02u%02u.map").c_str(), mapid, (unsigned)gx, (unsigned)gy);
Comment thread src/game/WorldHandlers/GridMap.cpp
@AppVeyorBot

Copy link
Copy Markdown

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 9, 2026 11:01
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 4 comments.

Comment thread src/game/WorldHandlers/MoveMap.cpp Outdated
{
--loadedTiles;
DEBUG_FILTER_LOG(LOG_FILTER_MAP_LOADING, "MMAP:unloadMap: Unloaded mmtile %03i[%02i,%02i] from %03i", mapId, x, y, mapId);
DEBUG_FILTER_LOG(LOG_FILTER_MAP_LOADING, "MMAP:unloadMap: Unloaded mmtile %04u[%02i,%02i] from %04i", mapId, x, y, mapId);
const int32 filenameTileY = y;

// load this tile :: mmaps/MMMYYXX.mmtile
// load this tile :: mmaps/MMMXXYY.mmtile
int len = sWorld.GetDataPath().length() + strlen("maps/%04u%02u%02u.map") + 1;
char* tmp = new char[len];
snprintf(tmp, len, (char*)(sWorld.GetDataPath() + "maps/%03u%02u%02u.map").c_str(), mapid, gx, gy);
snprintf(tmp, len, (char*)(sWorld.GetDataPath() + "maps/%04u%02u%02u.map").c_str(), mapid, (unsigned)gx, (unsigned)gy);
int len = sWorld.GetDataPath().length() + strlen("maps/%04u%02u%02u.map") + 1;
char* tmp = new char[len];
snprintf(tmp, len, (char*)(sWorld.GetDataPath() + "maps/%03u%02u%02u.map").c_str(), m_mapId, x, y);
snprintf(tmp, len, (char*)(sWorld.GetDataPath() + "maps/%04u%02u%02u.map").c_str(), m_mapId, x, y);
@AppVeyorBot

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants