[PATCH] rtems: Add rtems_status_code_description()
Chris Johns
chrisj at rtems.org
Thu Feb 6 21:47:14 UTC 2014
> On 2014-02-05 17:34, Joel Sherrill wrote:
>> This is redundant with rtems_status_text() which has been in the tree for
>> years. Chris wrote this years ago cpukit/libcsupport/src/error.c
>
> Its a matter of taste if you consider this as redundant. I think its an
> alternative.
We should avoid alternatives. They make APIs confusing because users see
what is the same thing and attempt to figure out differences that do not
exist.
I am happy to see rtems_status_text move from error.c to the sapi and
changed to be just the text.
> The rtems_status_text() returns strings with space characters.
I am fine with this becoming just the text of the errors.
> It pulls
> in a lot of other stuff (could be fixed easily).
Yeap lets fix this.
> The new function is
> more in line with rtems_fatal_source_description() and
> rtems_internal_error_description().
Now this has been raised I think all these functions are incorrectly
named. The rtems_status_text is currently a description and that should
change to be just the text and the ones you added are text. A
description is something you have in a user manual or man page.
Chris
More information about the devel
mailing list