On Sat, 07 Jul 2007 18:38:03 +0200, Hans de Goede wrote: > Jean Delvare wrote: > > I guess that the main problem here is that we have an artificial split > > of temp_print_info() outside of the generic temperature printing > > function, while we shouldn't. This explains in part why the this > > generic temperature printing function doesn't look quite good. > > Hmm, I don't think putting all the temp_print_info() stuff into the function > will make it better, maybe it will, but I don't know if a function becomes tom > large splitting of small sub functions can be a good idea. Agreed. But the number of arguments we are passing to temp_print_info() right now clearly shows that something is wrong. Having everything in the same file will make it easier to decide how to best split the code between functions. -- Jean Delvare