From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Wed, 25 Sep 2002 18:57:39 +0200 From: Alexey Dyachenko To: ALT Linux Mailing List Subject: Re: [devel] synaptic =?koi8-r?Q?=C9__g++_3?= =?koi8-r?B?LjIuMSDT1NLPy8kgySDJ1MXSwdTP0tk=?= Message-ID: <20020925165738.GA24525@deepblue.prokent.de> Mail-Followup-To: ALT Linux Mailing List References: <20020925094733.GB31139@deepblue.prokent.de> <20020925134215.GA31165@basalt.office.altlinux.ru> Mime-Version: 1.0 Content-Type: text/plain; charset=koi8-r Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20020925134215.GA31165@basalt.office.altlinux.ru> Sender: devel-admin@altlinux.ru Errors-To: devel-admin@altlinux.ru X-BeenThere: devel@altlinux.ru X-Mailman-Version: 2.0.13 Precedence: bulk Reply-To: devel@altlinux.ru X-Reply-To: Alexey Dyachenko List-Unsubscribe: , List-Id: List-Post: List-Help: List-Subscribe: , List-Archive: Archived-At: List-Archive: List-Post: On Wed, Sep 25, 2002 at 05:42:15PM +0400, Dmitry V. Levin wrote: > On Wed, Sep 25, 2002 at 11:47:33AM +0200, Alexey Dyachenko wrote: > > Вот только при этом получил я ОЧЕНЬ неприятную новость. > > > > std::string::iterator != const char * > > Это уже очень давно не новость. Увы, следить за всем, что происходит вокруг просто не успеваю. Сейчас занялся изучением Tcl и от плюсов немного отошел. > > То есть в gcc 3.2 basic_string::iterator объявлен как > > typedef __gnu_cxx::__normal_iterator iterator; > > > > В stlport это объявлеятся как > > typedef const value_type* const_iterator; > > typedef value_type* iterator; > > > > и по моему здесь прав stlport. > > Нет; итератор - это целиком и полностью implementation-defined. > Вообще говоря, по стандарту реализация std::string не гарантирует, > что std::string::iterator указывает на область памяти, содержащую строку > (да еще и NULL-terminated). Достал таки стандарт, действительно -- implementation defined. Извиняюсь за ложную тревогу, пора снова садиться за книги. > > > Если прав 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'а), > по причине использования негарантированного свойства итератора. Если можно -- объясни поподробней, где негарантированное свойство? вроде * для итератора -- самый что ни на есть гарантированный оператор, возвращающий объект. Для string итератор вроде должен быть random_access (хотя я в этом сейчас поклясться не могу) и к нему + тоже должен быть применим. По хорошему -- конечно все это надо переписать нормально, но мне нужен был быстрый результат, так что сделал "hack" в его оригинальном смысле "hack 1. /n./ Originally, a quick job that produces what is needed, but not well." Надеюсь, мантейнерам synaptic мой патч облегчит работу. С уважением, Алексей Дьяченко.