On Wed, Mar 6, 2013 at 9:22 AM, Joe Perches <joe@xxxxxxxxxxx> wrote: > There's no current way to know the version > of sparse. Add --version to see it. Hi Joe, Thanks for the patch. I agree that this the "--version" switch has its value. However, I don't like using "die" to print out the version string. I see no reason why sparse should exit with error for printing out version string. For example, gcc does not do that. If you don't want to recompile lib.c every time, you can make a version.c to store the version string. Another thing is nice to have, but not required. You consider to give "--" its own handing function. It is bit odd seeing the "-version" mix with other switch command. The patch is simpler. Either way is fine with me. The exit code need to be fixed. I will apply the patch once you fix the exit code. Chris -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html