On Thu, Sep 27, 2007 at 10:19:57AM -0700, ron minnich wrote: > OK, I like signed offset, but let's let stefan weigh in. It's critical > that we get this right.
I don't have any particular opinion here, but I'll NACK the patch in this form. If everybody else agrees with the patch _and_ there are at least 3-4 lines of code comments which explain all the issues involved here, the drawbacks and advantages of this (and the alternative) solution(s) etc., I'll be happy to ACK :) As already stated, this is way too tricky and non-obvious to go in without a good code comment. Uwe. -- http://www.hermann-uwe.de | http://www.holsham-traders.de http://www.crazy-hacks.org | http://www.unmaintained-free-software.org
signature.asc
Description: Digital signature
-- linuxbios mailing list linuxbios@linuxbios.org http://www.linuxbios.org/mailman/listinfo/linuxbios