[devel] Q: Merge my apt bugfixes

Slava Semushin =?iso-8859-1?q?slava=2Esemushin_=CE=C1_gmail=2Ecom?=
Вт Июл 31 22:23:00 MSD 2007


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

Прошу кого-нибудь из членов 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


Подробная информация о списке рассылки Devel