Re: [dev] [st] Division by zero

From: Paul Menzel <pmenzel_AT_molgen.mpg.de>
Date: Mon, 18 Jul 2016 14:54:51 +0200

Dear FRIGN,


On 07/18/16 14:49, FRIGN wrote:
> On Mon, 18 Jul 2016 14:45:44 +0200 Paul Menzel wrote:

>> If I am not mistaken, this is really a corner case. The user has to
>> set `actionfps` to zero in `config.def.h`.
>>
>> ```
>> config.def.h:static unsigned int actionfps = 30;
>> ```
>>
>> Even setting it to zero and rebuilding the package, I was unable to
>> trigger the issue.
>
> why would you set actionfps to 0? Thing is, what users do in config.h
> is their responsibility. I could also leave a struct empty and then
> be "surprised" about the program breaking. What you configure in
> config.h is your responsibility, that's it.

I can understand that reasoning. On the other hand, mistakes can happen,
so having checks for sensible values and inform the user about it, might
be a good idea.


Best regards,

Paul
Received on Mon Jul 18 2016 - 14:54:51 CEST