[patch] staging: panel: pass correct lengths to keypad_send_key()
w at 1wt.eu
Sat Dec 1 08:08:06 UTC 2012
On Thu, Nov 29, 2012 at 05:38:35PM +0300, Dan Carpenter wrote:
> We changed the sizeof() statements in 429ccf058b "staging:panel: Fixed
> coding conventions." so that they could fit inside the 80 character
> line limit. Unfortunately, the new sizeof() statements are a smaller
> size. This reverts it.
> There isn't a nice way to stay within the 80 character limit without
> a re-work so I've gone over.
Don't bother too much about the limit anyway, it's more of a guideline,
and the first time the code was adapted to fit, it became totally unreadable.
And BTW, I think that the only reason for the bug you found precisely is
the introduction of these repeat_str/release_str local variables that seem
to be there just to reduce line length...
> Signed-off-by: Dan Carpenter <dan.carpenter at oracle.com>
> Untested. The ->u.kbd.press_str and friends are funny data types.
I agree, it looks like the initial intent was to have the strings
fit well within the union, but this is all but obvious.
> not sure how this code, works but the stuff we have is wrong and I've
> reverted it back to the original.
Acked-by: Willy Tarreau <w at 1wt.eu>
More information about the devel