[devel] [git update] packages/rpm: heads/master

[Slava Semushin]

2007/11/19, Alex V. Myltsev <avm / altlinux.org>:
> Update of /people/avm/packages/rpm.git
[...]
> --- a/build/reqprov.c
> +++ b/build/reqprov.c
[...]
> +#define PRIVATE_PREFIX "[private]"
> +
> +/* If there is a PRIVATE_PREFIX at the beginning of depName, skip it. */
> +__attribute__((visibility ("hidden")))
> +const char* rpmStripDepPrefix(const char* depName)
> +{
> +    if (!strncmp(depName, PRIVATE_PREFIX, strlen(PRIVATE_PREFIX))) {
> +       depName += strlen(PRIVATE_PREFIX);
> +    }
> +    return depName;
> +}
> +
[...]

Почему бы не заменить strlen(PRIVATE_PREFIX) на sizeof(PRIVATE_PREFIX)-1 ?

Читабельность кода чуток уменьшится, зато длинна строки оба раза будет
вычисляться при компиляции, а не в run-time.

Это я так, лишь предлагаю.

-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru