Subject: Re: [PATCH v3 3/5] Move offset_1st_component() to path.c

From: Johannes Sixt <[email-masked]>
Date: Mon, 15 Feb 2010 20:43:41 +0100

Nguyễn Thái Ngọc Duy schrieb:
> +int offset_1st_component(const char *path)
> +{
> +	if (has_dos_drive_prefix(path))
> +		return 2 + (path[2] == '/');
> +	return *path == '/';
> +}

I'd have expected that you future-proofed this function by using 
is_dir_sep() or even use your previous implementation of is_root_path 
(because this implementation is a bit cryptic).

But if the new callers of this function will only pass the results of 
normalize_path_copy() and getcwd() (both return only forward-slashes on 
Windows), then I'm fine with this version. Do they?

-- Hannes