-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
giggle: fix cross, crash #27179
giggle: fix cross, crash #27179
Conversation
Hmm how does the cross patch work to fix? What seems to be doing is removing stuff related to time? |
By not depends on |
srcpkgs/giggle/patches/cross.patch
Outdated
return tm; | ||
#else | ||
struct tm *tm = g_new0 (struct tm, 1); | ||
+ long ltime; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#include <stdio.h>
int main()
{
printf("%ld\n", sizeof(long));
}
on my rpi prints 4, so I think it'd be more correct to use long long
here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sense, the original code is cursed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Anyway, correct format specifier for size_t
is zu
:-p
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Heh, true
Close #27151
@ndowens