You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
However, there is no non-null enforcement for the last 4 pointer arguments.
Then one could believe (like I did), that passing NULL for unwanted out argument is an option.
But it's not. 😅 ( dang, I just needed the height, nothing else) -wink-
If implementing the nullability of those args agrees to everyone's fancy, I'd be happy to do it.
The function will stay backward-compatible doing this, and the caller won't have its stack hogged with 3 more ushorts.
Originally, wouldn't it be more reasonable to pass those as references ? What motivated the choice of pointers here ?
The text was updated successfully, but these errors were encountered:
The current signature of Adafruit_GFX::getTextBounds are :
getTextBounds( [multiple_string_types] , int16_t , int16_t , int16_t*, int16_t* , uint16_t*, uint16_t*)
However, there is no non-null enforcement for the last 4 pointer arguments.
Then one could believe (like I did), that passing NULL for unwanted out argument is an option.
But it's not. 😅 ( dang, I just needed the height, nothing else) -wink-
If implementing the nullability of those args agrees to everyone's fancy, I'd be happy to do it.
The function will stay backward-compatible doing this, and the caller won't have its stack hogged with 3 more ushorts.
Originally, wouldn't it be more reasonable to pass those as references ? What motivated the choice of pointers here ?
The text was updated successfully, but these errors were encountered: