On Thu, May 08, 2014 at 10:26:36PM +0200, Ruediger Meier wrote: > That's a bit ugly to fix without more refactoring. I've tried here to > address both issues: > https://github.com/karelzak/util-linux/pull/73 Merged, but note that code like: if (ctl->time_fmt > LAST_TIMEFTM_SHORT_CTIME) is fragile (as someone can modify LAST_TIMEFTM_* enum), it would be better to introduce any macro and keep it near to LAST_TIMEFTM_* definitions or fix the problem in a better way. Karel -- Karel Zak <kzak@xxxxxxxxxx> http://karelzak.blogspot.com -- To unsubscribe from this list: send the line "unsubscribe util-linux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
