Skip to content

Match RPSysPlayerMgr::getSpeakerFlag - #175

Open
jpeisach wants to merge 3 commits into
doldecomp:masterfrom
jpeisach:master
Open

jpeisach wants to merge 3 commits into
doldecomp:masterfrom
jpeisach:master

Conversation

@jpeisach

@jpeisach jpeisach commented Oct 5, 2026

Copy link
Copy Markdown

I'm a little worried about the logic here: if player < 0, get the player number? not the channel ID?

Co-authored-by: roeming <roeming@users.noreply.github.com>
Signed-off-by: Joshua Peisach <itzswirlz2020@outlook.com>
@decomp-dev

decomp-dev Bot commented Oct 15, 2026

Copy link
Copy Markdown

Report for RSPE01_01 (27fa94a - 5a09615)

📈 Matched code: 35.03% (+0.00%, +100 bytes)

✅ 1 new match
Unit Item Bytes Before After
main/Pack/RPSystem/RPSysPlayerMgr RPSysPlayerMgr::getSpeakerFlag(long) const +20 80.00% 100.00%

Comment thread src/Pack/RPSystem/RPSysPlayerMgr.cpp Outdated
@kiwi515

kiwi515 commented Oct 15, 2026

Copy link
Copy Markdown
Collaborator

the logic lets u pass -1 to default to the active controller

@jpeisach

Copy link
Copy Markdown
Author

the logic lets u pass -1 to default to the active controller

other way around I thought? if player is -1, then it gets the player number, otherwise it must be positive in which it can call getChannel()?

Co-authored-by: Kiwi <49212064+kiwi515@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants