Celix
  1. Celix
  2. CELIX-28

celix_strerror in celix_errno.h does not give error correct error strings for errno.h errors

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0.0-incubating
    • Component/s: None
    • Labels:
      None

      Description

      When using celix_strerror to print a user friendly version of CELIX_ENOMEM (ENOMEM) you get the message "No message" while i expected "Out of memory".

      1. celix_errorcodes.c
        2 kB
        Nicolae Florin Petrovici
      2. errcodes.patch.txt
        2 kB
        Nicolae Florin Petrovici

        Activity

        Hide
        Nicolae Florin Petrovici added a comment -

        Hello,

        I have made a fix. Should work.

        Cheers,
        Florin

        Show
        Nicolae Florin Petrovici added a comment - Hello, I have made a fix. Should work. Cheers, Florin
        Hide
        Nicolae Florin Petrovici added a comment -

        Forgot to include it as a patch

        Show
        Nicolae Florin Petrovici added a comment - Forgot to include it as a patch
        Hide
        Alexander Broekhuis added a comment -

        Thanks for the patch. I changed the solution a bit, does this seem ok? It now also calls the APR strerror method for other error codes.

        ----------------
        Author: abroekhuis
        Date: Tue Oct 1 11:43:57 2013
        New Revision: 1528049

        URL: http://svn.apache.org/r1528049
        Log:
        CELIX-28: Updated error codes. Used only part of the patch. Updated the strerror method to be similar to the strerror method of APR.

        Show
        Alexander Broekhuis added a comment - Thanks for the patch. I changed the solution a bit, does this seem ok? It now also calls the APR strerror method for other error codes. ---------------- Author: abroekhuis Date: Tue Oct 1 11:43:57 2013 New Revision: 1528049 URL: http://svn.apache.org/r1528049 Log: CELIX-28 : Updated error codes. Used only part of the patch. Updated the strerror method to be similar to the strerror method of APR.

          People

          • Assignee:
            Unassigned
            Reporter:
            Pepijn Noltes
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development