Steevee28 wrote: 
> I would -not- recommend switching to an unsigned value type
> `lua_Integer`
> 
The point being, I think, that, at least in C, checking that a signed
integer is negative is probably more readable, and possibly more
efficient, than checking that an unsigned integer is >= half MAX UINT +
1, or whatever.

> 
> That's a very valid thought that has been discussed before, but IMO,
> fixing bugs should not be prevented by the fear of finding more bugs ;-)
> 
> Anyway, as you said, a regular scheduled restart might definitely still
> be a very good idea for Radios.
> We should also keep in mind, that all Jive-based projects (like
> PiCorePlayer) are affected by this bug.

I shall put away my fear ! But not a priority at present.

First step will be to make some reasonable looking proof of concept LUA
script function(s) to deal with the LUA side of things. Second step will
be to trawl all C and LUA code for occurrences. I don't think that would
be too hard.

One other thing that would need attention (from memory) is the ordering
of the timer list.


------------------------------------------------------------------------
mrw's Profile: http://forums.slimdevices.com/member.php?userid=38299
View this thread: http://forums.slimdevices.com/showthread.php?t=111650

_______________________________________________
Radio mailing list
Radio@lists.slimdevices.com
http://lists.slimdevices.com/mailman/listinfo/radio

Reply via email to