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

Timestamp support #187

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

TheComputerGuy96
Copy link

Fixes #100

The timestamp setting code currently doesn't use the file wrappers though (and is directly located in unshield.c)

These were originally present in the early unshield code but
for some reason they were removed later (this change is based
on the i5comp and i6comp struct layout)

Those struct members have a correct date with the Baldur's Gate
test (likely <IS6?) and GTA San Andreas (IS9)
This also adds a helper function to load the DOS date/time into a tm
structure and an library function to get the ISO 8601 date
This also adds an API function for getting an Unix timestamp
(and updates the man page)
@twogood
Copy link
Owner

twogood commented Dec 14, 2024

Very nice! What work is need to support it in the wrapper?

@twogood twogood self-assigned this Dec 14, 2024
@TheComputerGuy96
Copy link
Author

What work is need to support it in the wrapper?

I think it's as easy as adding the relevant callbacks (but I'm not sure if those 2 functions have alternative implementations; especially the utime() function)

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.

[Request] Timestamp preservation
2 participants