[devel] Q: Merge my apt bugfixes

[Alexey Tourbin]

On Wed, Aug 01, 2007 at 01:23:00AM +0700, Slava Semushin wrote:
> Прошу кого-нибудь из членов APT Development Team посмотреть/проверить
> и смержить мои изменения с
> http://git.altlinux.org/people/php_coder/packages/apt.git
> 
> Я сделал всего три изменения:
>    - исправил опечатку в описании патча
>    - исправил опечатку в выводе команды help в apt-shell (#5400, висит
> с октября 2004 года)
>    - исправил неверное сообщение при попытке удалить пакет в apt-shell
> (#7618, висит с августа 2005 года)
> 
> Также есть несколько вопросов/предложений:
>    - Url не открывается. Нужно или удалить или изменить на другой
> (гуглом сходу не нашел нужный)
>    - at@ последним добавил патч apt-0.5.15lorg3-alt-exit-status.patch
> Откуда в имени lorg3, если apt у нас lorg2?

По пьяни наверно написал lorg3 вместо lorg2.

>    - мне кажется, что в патче
> apt-0.5.15lorg2-alt-pkgInitConfig-cpu.patch утекает память, которую
> никто не освобождает после strdup() Поскольку ф-ция CndSet() вторым
> аргументом принимает тип std::string, то лучше объявлять переменную
> cpu именно этого типа и избавиться от strdup() вообще.
>    - в патче apt-0.5.15cnc6-alt-PrintLocalFile.patch три раза
> дублируется один и тот же кусок кода (что дурно пахнет). Было бы лучше
> выделить его в локальную ф-цию и именно её вызывать в трёх местах.
> 
> Спасибо за внимание.

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