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

Reply via email to