> + char *tmp = *msg, cur_sev = MCE_NO_SEVERITY, sev; Should cur_sev and sev be of type "int" (since that's the type returned by mce_severity())? It doesn't matter today since the list of return value does fit into "char", but it is setting up to fail if that should ever change. -Tony
