beyond-all-reason / beyond-all-reason/RecoilEngine

`Spring.SetSquareBuildingMask` has poor x/z interface

Open
#1,186 3 comments 0 reactions 0 assignees View on GitHub
enhancement good first issue
Dominant language
C++
Stars
683
Forks
293
Avg merge
3d 2h
Merged PRs (30d)
40

Description

Bill of materials to be done before starting this ticket:
* awareness of #1579

----

It requires you to scale the x/z parameters by 16 (i.e. `Game.buildSquareSize`) manually. This requirement is obscure and mistake prone. You should be able to specify the actual x/z you want to affect without having to take internal implementation details into account.

Contributor guide

Open the contributing guide

Research direction

Start with the Spring.SetSquareBuildingMask entry point and inspect how its x/z parameters are converted using Game.buildSquareSize. Read the awareness item in #1579 before deciding the intended coordinate behavior. Done means callers can provide the actual x/z area without manually scaling values by 16, with the relevant behavior verified by the project’s existing tests or reproduction path.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
api, game-dev
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.