nol888/parrot — comment on commit c447729f6978fb5c146534bca355e6cababa1a33
File:  line 0
By niner:

Why not call it Parrot_lib_fix_path_separator or even Parrot_lib_localize_path_separator? Such a name would still fit if we extend the function to say a platform which uses a colon as separator and at least to me it would be more clear what it does. With fix_path_slashes, I wouldn't have thought about forward/back slashes but more like fixing // to /.

Also a statement with three single letter variable names like if (c == f) c = r; is kinda hard to read.