Johannes Schindelin [off-list ref] writes:
It takes a little folding and knotting of the brain to understand that
this `!skip_dos_drive_prefix(&pos)` has *nothing* to do with the comment
`unc paths` nor with the test whether the paths starts with two directory
separators.
As a consequence, I would highly suggest to turn this into:
if (skip_dos_drive_prefix(&pos))
; /* absolute path with DOS drive prefix */
/* unc paths */
else if (is_dir_sep(pos[0]) && is_dir_sep(pos[1])) {
That makes the code a lot easier to understand, and as a consequence a lot
harder to mess up in the future.
Excellent. With or without "unc paths" comment, the separation does
make the logic more clear.