Hi Jim, > It looks to me like the change below is equivalent to yours,
Ah, I see now what you mean. Fine with me. I wouldn't have chosen this solution because it tears apart the determination of the ac_cv_func_working_mktime into two parts, one before the AC_CACHE_CHECK and one inside it (making the logic more complex) and because the configure output will show checking for working mktime... (cached) no already at the first invocation of the macro. > Then the nesting/logic of the majority of lines in > that file doesn't change. I find it normal to reindent even large chunks of code, when needed for the outer logic. With cvs, I could do "cvs diff -w". With git, "git diff --color-words" can convince you quickly that only the indentation has changed. Anyway, it's your .m4 files, your decision. Bruno
