Looks good!

I probably misunderstood something, but why is this using memchr everywhere? With the last param being 1, isn't it equivalent to just doing a character comparison? Especially as you are not using the returned pointer.

nikic, the memchr usage was a leftover and I have now removed it. Thanks for noticing!

I think we need an RFC for that anywy. @nikic what do you think?

I think an RFC is too much for this small addition, but it would be nice to at least bring this up on internals ;)

Alright, @mj could you bring that up on Internals with something like "if no one objects it’s gonna be merged in a week".

A flag for specifying the valid format would be useful.

@narfbg: Good point, I'll look into it.

@lstrojny: I'll start a thread on internals this week.

Also, a no-delimiter format might also be useful. :)

@mj as you said, you have anough karma. Go ahead. But please don't forget to squash the commits first.

Wouldn't the separator characters and/or formats be better off as flags?

ping @mj

I'm pretty busy at the moment but will look into it ASAP.

Has been merged.