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

[Alexey Tourbin]

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 для константных строк.

> Это я так, лишь предлагаю.
----------- следующая часть -----------
Было удалено вложение не в текстовом формате...
Имя     : =?iso-8859-1?q?=CF=D4=D3=D5=D4=D3=D4=D7=D5=C5=D4?=
Тип     : application/pgp-signature
Размер  : 189 байтов
Описание: =?iso-8859-1?q?=CF=D4=D3=D5=D4=D3=D4=D7=D5=C5=D4?=
Url     : <http://lists.altlinux.org/pipermail/devel/attachments/20071120/358ab7ba/attachment-0002.bin>