ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: "Slava Semushin" <slava.semushin@gmail.com>
To: "ALT Devel discussion list" <devel@lists.altlinux.org>
Subject: [devel] Q: Merge my apt bugfixes
Date: Wed, 1 Aug 2007 01:23:00 +0700
Message-ID: <75e139a00707311123w36e56372jec70bfcc5f83cc88@mail.gmail.com> (raw)

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

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

             reply	other threads:[~2007-07-31 18:23 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-31 18:23 Slava Semushin [this message]
2007-07-31 20:31 ` Alexey Tourbin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=75e139a00707311123w36e56372jec70bfcc5f83cc88@mail.gmail.com \
    --to=slava.semushin@gmail.com \
    --cc=devel@lists.altlinux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

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