1308105 - Replace PL_strpbrk with strpbrk

Status: RESOLVED FIXED

(Core :: XPCOM, defect, P5)

Description

[Chris Peterson [:cpeterson]]

We should replace NSPR functions with standard C functions when they available on all platforms.

1. Replace PL_strpbrk() calls with strpbrk():

http://searchfox.org/mozilla-central/search?q=%5CbPL_strn%3Fpr%3Fbrk%5Cb&regexp=true&path=.cpp%24

*** CAUTION: PL_strpbrk() will return without crashing when passed a null pointer, but strpbrk() will crash! Any calls to PL_strpbrk() that are changed should be audited to see if they might need to handle a null pointer.

2. Add #include <string.h> if it's needed for strpbrk()'s function declaration.

3. Remove #include "plstr.h" if it's no longer needed for other NSPR string function declarations.

Comment 1

[Nicklas Boman [:smurfd]]

Attachment: Bug 1308105 - Replace PL_strpbrk with strpbrk

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4844183d0b44
Replace PL_strpbrk with strpbrk r=xpcom-reviewers,kmag

Comment 3

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/4844183d0b44