
12 May
2011
12 May
'11
5:08 p.m.
On Thursday, May 12, 2011 08:47:56 Sergei Shtylyov wrote:
On 11-05-2011 18:54, Mike Frysinger wrote:
int __post_hotkeys_pressed(void) { +#ifdef CONFIG_SYS_POST_HOTKEYS_GPIO
- return ret;
+#endif
You surely meant #else here?
return 0; /* No hotkeys supported */
not really what i was going for. it's fine either way though.
I don't think 2 returns in a row is fine...
if there were no CPP here, then obviously you'd be right. but there is, and i dont see much point now in adding a little more wrapping. gcc doesnt warn about it. -mike