On Fri, 4 May 2007, Dana How wrote: > Now that I'm awake your original quote at the top suggests: > (a) zlib_compression_level = > isset(core.loosecompression) ? core.loosecompression : > isset(core.compression) ? core.compression : Z_BEST_SPEED; > (b) pack_compression_level = > isset(pack.compression) ? pack.compression : > isset(core.compression) ? core.compression : Z_DEFAULT_COMPRESSION; > > Your later reaction to my quoted (a)/(b) table suggests: > (a) zlib_compression_level = > isset(core.compression) ? core.compression : Z_BEST_SPEED; > (b) pack_compresion_level = > isset(pack.compression) ? pack.compression : Z_DEFAULT_COMPRESSION; My final comment is that I think the former looks more coherent, while the later is simpler to implement. Nicolas - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html