On Mon, Apr 04, 2016 at 02:33:39PM -0700, Jamil Said Jr. wrote:
> I have not looked at the code, but this approach seems to me to have the
> smaller change possible and still completely solve the issue:
> 
> At some point the code will compare the date of the version with the value
> that it has stored (I believe 18 months), and complain if the version is
> older. Why don't just change the value store to compare from 18 months to
> 1800000 months or so?

Look at the code. Even if you are not a programmer, it's easy to understand:

http://sources.debian.net/src/xscreensaver/5.30-1/driver/prefs.c/?hl=1739#L1739

Instead of "months > 18" you can just put "0", which is always false.

This would be equivalent to using an infinite amount of time in your example.

But this is not even necessary because you can just put a "return 0;"
as the very first line in the senescent_p function.

So yes, you are right that this annoying warning may be removed by
changing or adding a single line, a very small change indeed.

The "only" problem is that the author does not want anybody to do that
(read the comments at the beginning of the function).

Thanks.

Reply via email to