[devel] synaptic и g++ 3.2.1 строки и итераторы

[Dmitry V. Levin]

On Wed, Sep 25, 2002 at 11:47:33AM +0200, Alexey Dyachenko wrote:
> Мне тут понадобился работающий графический клиент для apt и я немного
> похакал synaptic для сборки gcc 3.2.
> Патч прилагается.
> 
> Вот только при этом получил я ОЧЕНЬ неприятную новость.
> 
> std::string::iterator !=  const char *

Это уже очень давно не новость.

> То есть в gcc 3.2 basic_string::iterator объявлен как
> typedef __gnu_cxx::__normal_iterator<pointer, basic_string>  iterator; 
> 
> В stlport это объявлеятся как 
>   typedef const value_type*                const_iterator;                                          
>   typedef value_type*                      iterator;
> 
> и по моему здесь прав stlport.

Нет; итератор - это целиком и полностью implementation-defined.
Вообще говоря, по стандарту реализация std::string не гарантирует,
что std::string::iterator указывает на область памяти, содержащую строку
(да еще и NULL-terminated).

> Если прав stlport -- тогда срочно сяду собирать пакет для сизифа.
> Хотя это надо бы сделать в любом случае -- вещь полезная.

Конечно.

> --- synaptic-0.16/common/indexcopy.cc	2001-08-15 21:44:09 +0200
> +++ synaptic-0.16-gcc32/common/indexcopy.cc	2002-09-25 10:28:33 +0200
> @@ -402,7 +402,7 @@
>        return;
>     
>     // Not a dists type.
> -   if (stringcmp(Path.begin(),Path.begin()+strlen("dists/"),"dists/") != 0)
> +   if (stringcmp( &( *Path.begin() ),&*( Path.begin()+strlen("dists/"),"dists/" ) ) != 0)

скобки тут лишние (хотя и не вредные).
когда-нибудь этот код может перестать работать (равно как и код apt'а),
по причине использования негарантированного свойства итератора.


--
ldv
----------- следующая часть -----------
Было удалено вложение не в текстовом формате...
Имя     : =?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/20020925/03f55bec/attachment-0001.bin>