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
_______________________________________________
rtems-devel mailing list
rtems-devel@rtems.org
http://www.rtems.org/mailman/listinfo/rtems-devel

Reply via email to