ALT Linux Team development discussions
 help / color / mirror / Atom feed
* [devel] errno.h
@ 2005-02-02  6:51 Konstantin Klimchev
  2005-02-03  7:37 ` Mikhail Zabaluev
  0 siblings, 1 reply; 3+ messages in thread
From: Konstantin Klimchev @ 2005-02-02  6:51 UTC (permalink / raw)
  To: devel

Добрый день!

Прошу совета (а то уже погряз в сомнениях "по самое
нихочу"):

есть фрюшный фрагмент кода

-----------------------------
report(LOG_ERR, "chroot %s: %s",
cf->chrootdir,strerror(*(__error())) );
-----------------------------
на котором, естественно, сборка спотыкается из-за
*(__error()) - на сколько я понял он есть только в libc.

Так вот. Как правильнее будет:

1) исправить  *(__error()) на errno

или

2) прописать в нужных местах то, что есть в libc:
extern 	int * 	__error();
#define errno	(* __error())

В принципе и так и так работает, но все таки...

-- 
Best Regards, Konstantin Klimchev 
(mailto:koka@atvc.ru    jabber:koka@jabber.atvc.ru)
ATK-Internet ISP, Arkhangelsk, Russia


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [devel] errno.h
  2005-02-02  6:51 [devel] errno.h Konstantin Klimchev
@ 2005-02-03  7:37 ` Mikhail Zabaluev
  2005-02-03  8:48   ` Konstantin Klimchev
  0 siblings, 1 reply; 3+ messages in thread
From: Mikhail Zabaluev @ 2005-02-03  7:37 UTC (permalink / raw)
  To: ALT Devel discussion list

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

В Срд, 02/02/2005 в 09:51 +0300, Konstantin Klimchev пишет:
> Добрый день!
> 
> Прошу совета (а то уже погряз в сомнениях "по самое
> нихочу"):
> 
> есть фрюшный фрагмент кода
> 
> -----------------------------
> report(LOG_ERR, "chroot %s: %s",
> cf->chrootdir,strerror(*(__error())) );
> -----------------------------
> на котором, естественно, сборка спотыкается из-за
> *(__error()) - на сколько я понял он есть только в libc.
> 
> Так вот. Как правильнее будет:
> 
> 1) исправить  *(__error()) на errno

Да. Заодно можете сделать доброе дело для апстрима, сделав код более
переносимым. Возможно, придется провести воспитательную беседу с
автором.


[-- Attachment #2: Эта часть сообщения подписана цифровой подписью --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [devel] errno.h
  2005-02-03  7:37 ` Mikhail Zabaluev
@ 2005-02-03  8:48   ` Konstantin Klimchev
  0 siblings, 0 replies; 3+ messages in thread
From: Konstantin Klimchev @ 2005-02-03  8:48 UTC (permalink / raw)
  To: ALT Devel discussion list

On Thu, 03 Feb 2005 10:37:33 +0300
Mikhail Zabaluev <mhz@altlinux.org> wrote:

> > 1) исправить  *(__error()) на errno
> 
> Да. 

Ага. "Сомнения его не обманули" (с) - вообщем на этом
решении и остановился. Буду терзать - а там посмотрим.

>Заодно можете сделать доброе дело для апстрима, сделав
> код более переносимым. Возможно, придется провести
> воспитательную беседу с автором.

Да это не апстрим, а патч, но патч очень сладкий. Или это
"патчевый апстрим" назвать :)

-- 
Best Regards, Konstantin Klimchev 
(mailto:koka@atvc.ru    jabber:koka@jabber.atvc.ru)
ATK-Internet ISP, Arkhangelsk, Russia


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2005-02-03  8:48 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-02-02  6:51 [devel] errno.h Konstantin Klimchev
2005-02-03  7:37 ` Mikhail Zabaluev
2005-02-03  8:48   ` Konstantin Klimchev

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