* [devel] Q: Merge my apt bugfixes
@ 2007-07-31 18:23 Slava Semushin
2007-07-31 20:31 ` Alexey Tourbin
0 siblings, 1 reply; 2+ messages in thread
From: Slava Semushin @ 2007-07-31 18:23 UTC (permalink / raw)
To: ALT Devel discussion list
Здравствуйте!
Прошу кого-нибудь из членов 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
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [devel] Q: Merge my apt bugfixes
2007-07-31 18:23 [devel] Q: Merge my apt bugfixes Slava Semushin
@ 2007-07-31 20:31 ` Alexey Tourbin
0 siblings, 0 replies; 2+ messages in thread
From: Alexey Tourbin @ 2007-07-31 20:31 UTC (permalink / raw)
To: ALT Devel discussion list
[-- 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 --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2007-07-31 20:31 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-07-31 18:23 [devel] Q: Merge my apt bugfixes Slava Semushin
2007-07-31 20:31 ` Alexey Tourbin
ALT Linux Team development discussions
This inbox may be cloned and mirrored by anyone:
git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
public-inbox-index devel
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://lore.altlinux.org/org.altlinux.lists.devel
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git