[devel] Q: Merge my apt bugfixes

[devel] Q: Merge my apt bugfixes

[Slava Semushin]

Здравствуйте!

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

Спасибо за внимание.

-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] Q: Merge my apt bugfixes

[Alexey Tourbin]

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

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 три раза
> дублируется один и тот же кусок кода (что дурно пахнет). Было бы лучше
> выделить его в локальную ф-цию и именно её вызывать в трёх местах.
> 
> Спасибо за внимание.

Щас попробую сделать.

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