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

Re: [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

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

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 962 bytes --]

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

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

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

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

[Денис Смирнов]

[-- Attachment #1: Type: text/plain, Size: 559 bytes --]

On Tue, Nov 20, 2007 at 02:18:01PM +0300, Алексей Турбин wrote:

>> Почему бы не заменить strlen(PRIVATE_PREFIX) на sizeof(PRIVATE_PREFIX)-1 ?
>> Читабельность кода чуток уменьшится, зато длинна строки оба раза будет
>> вычисляться при компиляции, а не в run-time.
AT> Кажется gcc умеет оптимизировать strlen для константных строк.

Причем даже с -O0.

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------
Буду пока не собирать ничего с GraphicsMagik.
		-- zerg in #10165

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]