-
Notifications
You must be signed in to change notification settings - Fork 25
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
sprintf issues #349
Comments
The test patch |
With 8e56237 I was able to remove the %a and %A options from cyclone and include %p |
with 737fb0a I seem to have enabled "hi" and "hu" update download test patch |
with 9bc4a57 I'm now able to load %F update: download file: |
with d3c835a I'm now able to use "%lf" and "%lF" |
With e341338 I tried adding new formats that start with "%L", like "%Lf", etc. It only did work to prevent errors, but "%Lf" and "%LF" actually blow up PD, and the others just output zero... hence, I commented it out. These seem to be a bit buggy in Max, but with Pd 64 bits around the corner, I think long integers/float could be useful in Pd ;) |
I updated the help file with 71af21a Here's a review of which issues we still have:
references: |
btw, we've also been discussing [makefilename] in vanilla pure-data/pure-data#549 |
"#" doesn't work in Max |
"[sprintf]: slot skipped ('j' modifier not supported in a format pattern)" is not an error given in Max |
|
I guess I'll reboot this issue, cause it became too messy |
Update the help file
Revise the code...
For whatever '%' argument we create in max, it generates an inlet, no matter if it's a valid thing - that's not the case in Pd, where it verifies if it's a 'valid' argument. So it should get whatever it receives instead of doing this checking.
BTW, the code has this:
/* CHECKED: max creates as many inlets, as there are %-signs, no matter
if they are valid, or not -- if not, it prints ``can't convert'' errors
for any input... */
And here's a list of issues:
references:
https://www.tutorialspoint.com/c_standard_library/c_function_sprintf.htm
https://en.wikipedia.org/wiki/C_data_types
https://www.gnu.org/software/libc/manual/html_node/Table-of-Output-Conversions.html
The text was updated successfully, but these errors were encountered: