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.

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

feat: add values for m_ChangedBitFlag, populate unknown fields of FriendPersistentInfo for Friends (3DS) #36

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ZeroSkill1
Copy link
Contributor

Resolves #XXX

Changes:

Add values for the m_ChangedBitFlag enum in NintendoPresence and the remaining "Unknown" fields in MyProfile based on reverse engineering of the 3DS friends module.

Copy link
Member

@DaniElectra DaniElectra left a comment

Choose a reason for hiding this comment

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

Nice work! Do you also happen to know anything about the AccountExtraInfo or the NintendoCreateAccountData structures?

| Uint8 | Area |
| Uint8 | Language |
| Uint8 | Platform |
| Uint64 | Local friend code seed |
Copy link
Member

Choose a reason for hiding this comment

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

Is this the Local Friend Code Seed or the Local Friend Code? The terminology is different (The "Local Friend Code" includes seemingly random data on the upper 16 bits, whereas the "Local Friend Code Seed" doesn't)

Comment on lines +513 to +523
| Flag | Description |
| ----------------------------- | --------------------------------------- |
| 1 | [Game Key](#gamekey-structure) changed. |
| 2 | Game mode description changed. |
| 4 | Join availability flag changed. |
| 8 | Matchmake system type changed. |
| 16 | Join game ID changed. |
| 32 | Join game mode changed. |
| 64 | Owner [PID] changed. |
| 128 | Join group ID changed. |
| 256 | Application argument changed. |
Copy link
Member

Choose a reason for hiding this comment

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

Can you represent the flags in hex instead of decimal?

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