Skip to content

Conversation

@krazynez
Copy link

Cross referenced with uOFW src

#endif
#ifdef F_SysclibForKernel_0040
IMPORT_FUNC "SysclibForKernel",0x909C228B,setjmp
IMPORT_FUNC "SysclibForKernel",0xC2145E80,snprintf
Copy link
Contributor

Choose a reason for hiding this comment

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

why was this switched ?

Copy link
Author

Choose a reason for hiding this comment

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

I can move it down, was putting it near sprintf, also need to update the Makefile.am to add the missing F_SysclibForKernel_ function for snprintf or the offset. Just noticed that.

Copy link
Contributor

Choose a reason for hiding this comment

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

usually keeping diffs minimal is the best course of action (i.e. leave the existing longjmp/setjmp blocks as they were)

Copy link
Author

Choose a reason for hiding this comment

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

Okay no prob I will append it to the bottom 👍

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