On 2013-01-04 21:12, Aaron Plattner wrote: > On 01/02/2013 02:38 PM, Peter Harris wrote: >> Some tests segfault if XT_DISPLAY is set and XT_DISPLAYHOST is not. >> >> Signed-off-by: Peter Harris <phar...@opentext.com> > > Reviewed-by: Aaron Plattner <aplatt...@nvidia.com> > > Which test?
Xlib3/XOpenDisplay > Tested-by: Aaron Plattner <aplatt...@nvidia.com> Thanks. I just noticed, however, that >> + my @displayhost = split(/:/, $::vars{"DISPLAY"}, 2); >> + $::vars{"XT_DISPLAYHOST"} = $displayhost[0]; can't work for IPv6 literal addresses. I need to split on the last :, not the first one. I'll redo the patch. Peter Harris -- Open Text Connectivity Solutions Group Peter Harris http://connectivity.opentext.com/ Research and Development Phone: +1 905 762 6001 phar...@opentext.com Toll Free: 1 877 359 4866 _______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel