diff options
| author | Vito Caputo <vcaputo@pengaru.com> | 2023-12-03 21:45:53 -0800 | 
|---|---|---|
| committer | Vito Caputo <vcaputo@pengaru.com> | 2023-12-03 21:45:53 -0800 | 
| commit | c84a122828d3f8be87be3af8a1f7b0a21cf8b456 (patch) | |
| tree | cf61d448483f7bca961a6518841dbd882c72c513 | |
| parent | 5586fd914abec44a1fe9eab3d64d83572edf306e (diff) | |
modules/spokes: uint setup->{thickness,iterations}
Let's not handle negative values for these in the setup, even if
it's all multiple choice options that don't include erroneous
negatives.  Technically you can feed in whatever using the ":"
force prefix...
| -rw-r--r-- | src/modules/spokes/spokes.c | 8 | 
1 files changed, 4 insertions, 4 deletions
| diff --git a/src/modules/spokes/spokes.c b/src/modules/spokes/spokes.c index 0aa8eed..2b5fbe3 100644 --- a/src/modules/spokes/spokes.c +++ b/src/modules/spokes/spokes.c @@ -25,9 +25,9 @@ typedef struct spokes_context_t {  typedef struct spokes_setup_t {  	til_setup_t             til_setup; -	int			iterations; +	unsigned		iterations;  	float                   twist; -	int			thickness; +	unsigned		thickness;  } spokes_setup_t;  static void spokes_draw_line(til_fb_fragment_t *fragment, int x1, int y1, int x2, int y2, uint32_t color, int thickness) @@ -331,13 +331,13 @@ int spokes_setup(const til_settings_t *settings, til_setting_t **res_setting, co  		if (!setup)  			return -ENOMEM; -		if (sscanf(iterations->value, "%i", &setup->iterations) != 1) +		if (sscanf(iterations->value, "%u", &setup->iterations) != 1)  			return til_setup_free_with_failed_setting_ret_err(&setup->til_setup, iterations, res_setting, -EINVAL);  		if (sscanf(twist->value, "%f", &setup->twist) != 1)  			return til_setup_free_with_failed_setting_ret_err(&setup->til_setup, twist, res_setting, -EINVAL); -		if (sscanf(thickness->value, "%i", &setup->thickness) != 1) +		if (sscanf(thickness->value, "%u", &setup->thickness) != 1)  			return til_setup_free_with_failed_setting_ret_err(&setup->til_setup, thickness, res_setting, -EINVAL);  		*res_setup = &setup->til_setup; | 
