ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Alexey Tourbin <at@altlinux.ru>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: Re: [devel] PyGTK
Date: Fri, 24 Jul 2009 13:51:02 +0400
Message-ID: <20090724095102.GD11416@altlinux.org> (raw)
In-Reply-To: <1248428689.16801.34.camel@latitude.localdomain>

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

On Fri, Jul 24, 2009 at 01:44:49PM +0400, Alexey Rusakov wrote:
> В Птн, 24/07/2009 в 10:59 +0400, Alexey Tourbin пишет:
> > On Fri, Jul 24, 2009 at 02:53:35PM +0800, REAL wrote:
> > > Исходный код:
> > > strncpy((char *) &event->client.data, tmp,
> > 
> > А зачем тут адрес берётся?
> Адрес тут берётся, потому что event->client.data - это union.
> event->client - это GdkEventClient, который выглядит так:
> 
> typedef struct {
>   GdkEventType type;
>   GdkWindow *window;
>   gint8 send_event;
>   GdkAtom message_type;
>   gushort data_format;
>   union {
>     char b[20];
>     short s[10];
>     long l[5];
>   } data;
> } GdkEventClient;

В том-то и дело, что если копируем в массив, то адрес массива брать
не нужно (или не обязательно).  Поэтому сразу подозрение, что копируют
куда-то не туда, куда хотят.

> Соответственно, люди таким образом копируют строку в
> event->client.data.b . Не знаю, почему бы не использовать явное указание
> поля в объединении - может, с переносимостью этого какая проблема... Но
> у меня есть подозрение, что на 64-разрядной платформе размер этого
> union'а оказался больше, чем размер его поля. Проверить, к сожалению, в
> данный момент не на чем (а на 32 битах всё в порядке, sizeof data
> возвращает 20).

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

  reply	other threads:[~2009-07-24  9:51 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-24  3:39 REAL
2009-07-24  3:42 ` REAL
2009-07-24  3:51 ` REAL
2009-07-24  5:39   ` Eugene Ostapets
2009-07-24  5:58     ` REAL
2009-07-24  6:25       ` Eugene Ostapets
2009-07-24  6:53         ` REAL
2009-07-24  6:59           ` Alexey Tourbin
2009-07-24  7:22             ` REAL
2009-07-24  9:44             ` Alexey Rusakov
2009-07-24  9:51               ` Alexey Tourbin [this message]
2009-07-24 10:34   ` Sergey Vlasov
2009-07-27  9:27     ` REAL
2009-07-27 11:04       ` Yuri N. Sedunov
2009-07-28  5:08         ` REAL

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=20090724095102.GD11416@altlinux.org \
    --to=at@altlinux.ru \
    --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