2010/10/25 Jakub Narebski <jnareb@xxxxxxxxx>: > On Sun, 24 Oct 2010, Giuseppe Bilotta wrote: > > [...] >> +# Group output by placing it in a DIV element and adding a header. >> +# Options for start_div() can be provided by passing a hash reference as the >> +# first parameter to the function. >> +# Options to git_print_header_div() can be provided by passing an array >> +# reference. This must follow the options to start_div if they are present. >> +# The content can be a scalar, which is output as-is, a scalar reference, which >> +# is output after html escaping, an IO handle passed either as *handle or >> +# *handle{IO}, or a function reference. In the latter case all following >> +# parameters will be taken as argument to the content function call. >> +sub git_print_section { > > Very nice API. Good work! I'm glad you like it. And thank you very much for the reviews that took me to design it properly. Shall I take this as an ack for the series, modulo the minor suggestions for the other two patches? (Just to know if I can include Junio in the next posting) -- Giuseppe "Oblomov" Bilotta -- 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