On 07/16/2013 11:35 AM, Zakaria Kasmi wrote:
> In sys/shell/commands/sc_heap.c:
>
>> +extern void heap_stats(void);
>> +
>> +void _heap_handler(char* unnused){
>> +	heap_stats();
>
> one “Line Of Production code” follows the same scheme as in “sc_ps.c”, “sc_id.c.”, etc.
This might be correct, but the *commits* introducing “sc_ps.c”, 
“sc_id.c.”, etc. didn't introduce "Makefile updates".

If we pull your one-liner together with the makefile updates, there'll 
be a "merge commit". Now imagine your one line has a problem somewhere 
else, and we revert the merge commit. That would also unmerge your 
Makefile "fixes".

Christian proposed to seperate the makefile fixes from your added 
functionality.