Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

d_attention #701

Draft
wants to merge 18 commits into
base: main
Choose a base branch
from
Draft

d_attention #701

wants to merge 18 commits into from

Conversation

Mm2PL
Copy link

@Mm2PL Mm2PL commented Mar 9, 2025 •
edited
Loading

TODO List:

  • Merge latest changes
  • Finish ::Draw() - instructions don't exactly match
  • Deal with magic numbers for mFlags
  • Deal with magic numbers for mLockOnState
  • Dealing with unnamed fields would be nice
  • rebase for nice history

@Mm2PL Mm2PL force-pushed the d_attention branch 3 times, most recently from 865af82 to a9658f9 Compare March 9, 2025 23:31
Mm2PL added 16 commits March 12, 2025 16:35
@Mm2PL
Add helper functions for d_attention
79cc7e3
@Mm2PL
Copy and adapt code from TP
9a2a987
@Mm2PL
Move for better matching layout
b25afbd
@Mm2PL
Clean up includes for d_attention.cpp
9166b1a
@Mm2PL
Give dAttention_c::nextAttention a return
aaf1e2d
@Mm2PL
Use enums in dAttention_c::getActionBtnB and slight reformat
8feaa82
@Mm2PL
Change return types for comparison code to match
16c0f21 
I don't love this
@Mm2PL
Implement dAttList_c::calcWeight
350e3ab
@Mm2PL
Implement dAttention_c::sortList
51c25d4
@Mm2PL
Implement dAttDraw_c::draw
e221e7e
@Mm2PL
Implement dAttDraw_c::setAnm
426d2e3
@Mm2PL
Implement dAttention_c::chaseAttention
9fe6ca6
@Mm2PL
Implement judgement related functions
10ecdf2
@Mm2PL
Implement dAttention_c::Run and runDrawProc
c862bf6
@Mm2PL
Implement dAttention_c::Draw
7b6afef 
Actually finish Draw
@Mm2PL
Looks like dAttention_c::stockAttention had an unused return value
623925d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Mm2PL