On 03/28/2011 08:28 AM, Yong Shen wrote:
On Sun, Mar 27, 2011 at 5:06 AM, Daniel Lezcanodaniel.lezcano@free.frwrote:
By moving the functions in the right order in the file, we can get ride of their definitions and we can set them static.
Technically, those function names in head files should be declarations, not definitions.
Right, I meant "declarations", thanks.
For this patch, it could be further split, since it is a little bit messed up, for example: for print_clock_info(), you moved it from the original place, but for others, it is really hard to see this. It looks like you removed it and then changed collapse_all_subclocks to a new print_clock_info somehow.
Yep, sometimes the changes are trivial (here just moving the functions up) but the diff result is hard to understand. I can split this patch into several but is it really worth to do that ?