cleaner & greener (was Re: A patch for RTEMS4.10.0 PowerPC heap space initialization)
Joel Sherrill
joel.sherrill at OARcorp.com
Fri May 13 18:23:15 UTC 2011
On 05/13/2011 01:13 PM, Kate Feng wrote:
> Peter Dufault wrote:
>> On May 13, 2011, at 1:41 , Kate Feng wrote:
>>
>>
>>> I agree so. More comments regarding the attached patch before the
>>>
>> I never like to see "externs" in C code, it bypasses interface checking. Is there a header you can declare "uintptr_t bsp_sbrk_init(uintptr_t, uintptr_t*);" in?
>>
> How about rtems/bspIo.h ?
>
No. This is not printk/IO related.
I would lean to bootcard.h since that's where most of this
type of stuff is prototyped.
Hmmm... no Doxygen there. Something else to do. :(
--joel
> Cheers,
> Kate
>> Whoever does the final commit can do that additional cleanup, though.
>>
>>
>> Peter
>> -----------------
>> Peter Dufault
>> HD Associates, Inc. Software and System Engineering
>>
>>
> _______________________________________________
> rtems-users mailing list
> rtems-users at rtems.org
> http://www.rtems.org/mailman/listinfo/rtems-users
--
Joel Sherrill, Ph.D. Director of Research& Development
joel.sherrill at OARcorp.com On-Line Applications Research
Ask me about RTEMS: a free RTOS Huntsville AL 35805
Support Available (256) 722-9985
More information about the users
mailing list