Re: [PATCH v2 2/8] unicode: Create utf8_check_strict_name
From: Theodore Ts'o
Date: Tue Sep 03 2024 - 07:39:04 EST
I'd suggest using the one-line summary:
unicode: create the helper function utf8_check_strict_name()
so that it's a bit more descriptive.
On Mon, Sep 02, 2024 at 07:55:04PM -0300, André Almeida wrote:
> +/**
> + * utf8_check_strict_name - Check if a given name is suitable for a directory
> + *
> + * This functions checks if the proposed filename is suitable for the parent
> + * directory. That means that only valid UTF-8 filenames will be accepted for
> + * casefold directories from filesystems created with the strict enconding flags.
> + * That also means that any name will be accepted for directories that doesn't
> + * have casefold enabled, or aren't being strict with the enconding.
I also suggest wrapping with a fill column of 72 characters, instead
of 80.
- Ted