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. errcodes.patch.txt
        2 kB
        Nicolae Florin Petrovici
      2. celix_errorcodes.c
        2 kB
        Nicolae Florin Petrovici

        Activity

        Pepijn Noltes created issue -
        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
        Nicolae Florin Petrovici made changes -
        Field Original Value New Value
        Attachment celix_errorcodes.c [ 12583159 ]
        Nicolae Florin Petrovici made changes -
        Attachment errcodes.patch.txt [ 12583166 ]
        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
        Alexander Broekhuis made changes -
        Fix Version/s 0.0.2-incubating [ 12323511 ]
        Alexander Broekhuis made changes -
        Assignee Alexander Broekhuis [ abroekhuis ]
        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.
        Alexander Broekhuis made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Alexander Broekhuis made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Alexander Broekhuis made changes -
        Resolution Fixed [ 1 ]
        Status Closed [ 6 ] Reopened [ 4 ]
        Alexander Broekhuis made changes -
        Status Reopened [ 4 ] Closed [ 6 ]
        Assignee Alexander Broekhuis [ abroekhuis ]
        Resolution Fixed [ 1 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        616d 21h 30m 1 Alexander Broekhuis 01/Oct/13 12:45
        Resolved Resolved Closed Closed
        76d 23h 29m 1 Alexander Broekhuis 17/Dec/13 11:15
        Closed Closed Reopened Reopened
        8m 49s 1 Alexander Broekhuis 17/Dec/13 11:24
        Reopened Reopened Closed Closed
        3m 21s 1 Alexander Broekhuis 17/Dec/13 11:27

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development