patchWeeChat - Patches: patch #8301, Use strrchr() instead of rindex()


patch #8301: Use strrchr() instead of rindex()

Submitter:  None
Submitted:  Sat 01 Feb 2014 02:41:56 PM UTC
Category:  compilation Priority:  5 - Normal
Status:  Done Privacy:  Public
Assigned to:  flashcode Originator Email:  -email is unavailable-
Open/Closed:  Closed IRC nick:  fornwall
* Mandatory Fields

Add a New Comment Rich Markup

Sun 02 Feb 2014 08:02:56 AM UTC, comment #1: 

Thanks for the patch, committed.

I replaced another call to rindex() (in rmodifier plugin), and I removed the include of "strings.h", which should not be needed any more.

Sebastien Helleu <flashcode>
Group administrator
Sat 01 Feb 2014 02:41:56 PM UTC, original submission:  

Nowadays strrchr() is preferred over rindex() [1].

This fixes compilation with androids bionic libc, which has strrchr() but not rindex().




(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:

Attached Files
file #30432:  strrchr_insteadof_rindex.patch added by None (563B - application/octet-stream)


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -email is unavailable- added by flashcode (Posted a comment)
  • -email is unavailable- added by None (Submitted the item)

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.


    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2014-02-09 flashcode Open/ClosedOpen Closed
    2014-02-02 flashcode StatusNone Done
        Assigned toNone flashcode
    2014-02-01 None Attached File- Added strrchr_insteadof_rindex.patch, #30432

    Back to the top

    Powered by Savane 3.12