On Tue, Nov 20, 2007 at 05:12:59PM +0600, Slava Semushin wrote:
> 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.

Кажется gcc умеет оптимизировать strlen для константных строк.

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