[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