Messages in this thread | | | Subject | Re: [RFC/PATCH 1/8] lib: string: Introduce strreplace | From | Joe Perches <> | Date | Thu, 04 Jun 2015 15:45:47 -0700 |
| |
On Thu, 2015-06-04 at 23:37 +0100, Al Viro wrote: > On Thu, Jun 04, 2015 at 03:56:07AM -0700, Joe Perches wrote: > > Seems sensible, but the name maybe could be be more > > explicit as strreplace seems like it should more like > > a string substitution rather than a char substitution. > > > > Maybe strsubstchr or something like it (strtranschr?) > > Why not go the full monty and call it strtrtsstrrrtst(), with > strtrtssstrrtst() doing almost, but not quite the same thing?
Presumably the extra r is for reverse and the extra s in for insensitive, just because...
Naming does matter. Consistency too.
And for the consistency bit, the char arguments should probably be int.
| |