From: Rickard Strandqvist <[email protected]> Date: Sat, 7 Jun 2014 01:25:30 +0200
> Removes an ifdef that is no longer available, > and that would cause a compile error if it was activated. > > This was partly found using a static code analysis program called cppcheck. > > Signed-off-by: Rickard Strandqvist <[email protected]> This indentation looks wrong, and if you scan up you'll see why. A different set of CPP guards create the openning curly brace(s): /* PROCESS DATA (two members) */ #ifdef CMX_CONF_DEBUG if (0) { #else if (members == 2) { #endif This is crazy, and deleting only the DSP_NEVER_DEFINED part makes it even more confusing because less sophisticated code analysis tools and editing mode will think that the braces are unbalanced now. I'm not applying this, it makes things worse not better, sorry. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
