On Sat, Mar 6, 2010 at 4:30 PM, Øyvind Harboe <[email protected]> wrote:
>> Probably, a better patch should merge the code pattern above in a
>> single function, e.g. cfi_write_command_to_target(), that allocates
>> local array, sets it, writes it to target and frees it.
>> This would be less flexible, but would highlight the needed array size.
>
> Optimizing code for readability? I like that!
>
> We still want good performance though, but that's rarely a conflict.

Makes sense, and in this case I do not expect performance degradation.
I will re-elaborate the code and submit a new patch accordingly.

Best Regards,
Antonio Borneo
_______________________________________________
Openocd-development mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to