On 05/03/2014 01:03, Mark Cave-Ayland wrote:
On 04/03/14 20:57, Olivier Danet wrote:
Just push the one-liner Forth patch I submitted two months ago...
Okay... I think I understand this a bit better now. According to grep there are only 2 Forth references to next-property in the source tree:
forth/admin/devices.fs forth/system/ciface.fs
Now we know next-property in ciface.fs can never be called with a non-existent property, since the check further up the code will detect this first and exit.
Similarly the loop in devices.fs (.properties) starts from the first property in the package and loops through them in turn, so again it is impossible to call next-property with a non-existent property here.
On this basis, given that the patch causes no regressions in my SPARC32 test images I've applied it to SVN trunk - thanks for the patch, and apologies for the delay in getting it applied.
ATB,
Mark.
Never mind. And thank you for your hard work !
Olivier.