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

Fix Song detection and update Offsets #8

Merged
merged 15 commits into from Sep 29, 2023

Conversation

PrincessAkira
Copy link
Contributor

@PrincessAkira PrincessAkira commented Sep 29, 2023

32-Bit is currently throwing a Nullpointer Exception with garbled TrackID
image

Investigating atm.

EDIT:
So its able to find the title name of the song in the 32-Bit with a 64-bit offset.
IT will also print true to the trackID of the 64-Bit offset in a 32-Bit application
image

Atleast, it lasted 2 weeks this time haha

PrincessAkira and others added 3 commits September 29, 2023 11:33
# Conflicts:
#	src/main/java/de/labystudio/spotifyapi/platform/windows/api/spotify/SpotifyProcess.java
@PrincessAkira
Copy link
Contributor Author

PrincessAkira commented Sep 29, 2023

This was brought up to me as concern.
Does my Character Checking include Japanese, chinese etc?

image

It also works with russian and korean
image

Sadly, im unable to test with another Windows Language, as that may affect it?
Unsure though since i dont own a Russian Locale or Japanese Locale system

@PrincessAkira PrincessAkira changed the title Update to latest Offsets Fix Song detection and update Offsets Sep 29, 2023
# Conflicts:
#	src/main/java/de/labystudio/spotifyapi/platform/windows/api/spotify/SpotifyProcess.java
&& new MemoryPlaybackAccessor(this, address).isValid();
});
}

if (addressPlayBack == -1) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Can be squashed into the line above 121 - 128

@LabyStudio LabyStudio merged commit 200ca52 into LabyStudio:master Sep 29, 2023
1 check passed
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.

None yet

2 participants