Bug 10573 - xmessage gets its widget hierarchy wrong [patch included]
Summary: xmessage gets its widget hierarchy wrong [patch included]
Status: RESOLVED NOTABUG
Alias: None
Product: xorg
Classification: Unclassified
Component: App/other (show other bugs)
Version: git
Hardware: Other All
: medium normal
Assignee: Matthieu Herrb
QA Contact: Xorg Project Team
URL: http://bugs.debian.org/cgi-bin/bugrep...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-04-09 08:50 UTC by Brice Goglin
Modified: 2010-05-11 07:56 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Brice Goglin 2007-04-09 08:50:40 UTC
According to M.Buckett in the Debian BTS 6 years ago, the following patch should be applied to xmessage (apparently still applies to git):

<       XtVaSetValues(text, XtNwidth, width, XtNheight, height, NULL);
---
>       XtVaSetValues(XtParent(text), XtNwidth, width, XtNheight, height, NULL);

The submitter's address isn't valid anymore, but I guess you people will be able to see whethr he was right or not. Here's the explanation is give.

xmessage [...] seemed to have a problem displaying a message. All I would
get was a white empty box and the Okay message but the box was the correct
size for the text. I discovered that this was due to my Xresources setup as
clearing them displayed the message correctly. No matter how I tried I
did not seem to be able to keep my Xresources and have xmessage work.

Looking at the source and comparing it with xmotd (both use asciiTextWidget) I
discovered that xmessage made a different call to set the values of the widget,
so I made xmessage the same and hey presto it works. I'm no X programmer so I'm
not sure how correct this is but it seems to work.
Comment 1 Matthieu Herrb 2007-04-15 09:47:12 UTC
Hmm this report really lack details. 
Can you at least give us an url to the Debian BTS entry?
Comment 2 Brice Goglin 2007-04-15 11:36:16 UTC
I entered it in the "URL" field above, I don't know whether I am actually supposed to do so :)
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=94713

However, I already copied almost everything from the Debian BTS into here. And since the submitter's address isn't invalid anymore, we won't be able to help you more.

I thought such a simple patch would look obviously wrong or obviously correct to somebody knowing Xt well...
Comment 3 Matthieu Herrb 2007-04-15 11:54:02 UTC
oops I did't look closely(In reply to comment #2)
> I entered it in the "URL" field above, I don't know whether I am actually
> supposed to do so :)
> http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=94713

Sorry I didn't look to the PR closely enough. 

> 
> However, I already copied almost everything from the Debian BTS into here. And
> since the submitter's address isn't invalid anymore, we won't be able to help
> you more.
> 
> I thought such a simple patch would look obviously wrong or obviously correct
> to somebody knowing Xt well...
> 

It looks strange to me, causes an extra vertical space to be added below the text in the few test cases I did, and I still don't understand what problem this patch is supposed to fix. Xt applications can behave stupidly with stupid resources, so I'm more inclined to label this as an user error than as a real problem that needs a patch.
Comment 4 Brice Goglin 2007-04-15 12:16:32 UTC
Ok, then, go ahead and close it. I'll close the Debian bug too. If the submitter ever wakes up or if anybody else gets the same problem, I will tell him to come here.

Sorry for the noise.

Brice
Comment 5 Julien Cristau 2010-05-11 07:56:12 UTC
Resolving as not a bug, per comment#3.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.