headers
Chris Ball | 1 Oct 2003 03:55
Picon
Picon
Favicon

[PATCH] Remove stderr garbage from at-spi.

Hi,

Please don't leak to stderr like this.

Thanks,

- Chris.

Index: spi_main.c
===================================================================
RCS file: /cvs/gnome/at-spi/cspi/spi_main.c,v
retrieving revision 1.38
diff -u -r1.38 spi_main.c
--- spi_main.c  1 Sep 2003 12:35:54 -0000       1.38
+++ spi_main.c  1 Oct 2003 01:10:45 -0000
 <at>  <at>  -514,8 +514,6  <at>  <at> 

   cspi_cleanup ();

-  fprintf (stderr, "bye-bye!\n");
-
   return leaked;
 }

--

-- 
Chris Ball                                        <cjb <at> mrao.cam.ac.uk>
  Inference Group:            <http://www.inference.phy.cam.ac.uk/is/>
  The Dasher Project:     <http://www.inference.phy.cam.ac.uk/dasher/>
Permalink | Reply | 2 comments |
headers
Bill Haneman | 1 Oct 2003 17:03
Picon

Re: [PATCH] Remove stderr garbage from at-spi.

Hi Chris:

I suggest you file a bug regarding error messages in at-spi's cspi
bindings.  Though the message below is pretty useless, there's a broader
problem of where the diagnostics go, which can't be solved by just
removing all the fprintfs to stderr.  For the time being they are our
only means of getting some vital diagnostics (though you are right in
discouraging indiscriminate use of stderr).

regards

Bill

On Wed, 2003-10-01 at 02:55, Chris Ball wrote:
> Hi,
> 
> Please don't leak to stderr like this.
>  
> Thanks,
> 
> - Chris.
> 
> Index: spi_main.c
> ===================================================================
> RCS file: /cvs/gnome/at-spi/cspi/spi_main.c,v
> retrieving revision 1.38
> diff -u -r1.38 spi_main.c
> --- spi_main.c  1 Sep 2003 12:35:54 -0000       1.38
> +++ spi_main.c  1 Oct 2003 01:10:45 -0000
>  <at>  <at>  -514,8 +514,6  <at>  <at>
(Continue reading)

Permalink | Reply | 1 comment |
headers
Chris Ball | 1 Oct 2003 17:43
Picon
Picon
Favicon

Re: [PATCH] Remove stderr garbage from at-spi.

Bill,

On Wed, Oct 01, 2003 at 04:03:23PM +0100, Bill Haneman wrote:
> I suggest you file a bug regarding error messages in at-spi's cspi
> bindings. 

Thanks, will do.

> For the time being they are our only means of getting some vital 
> diagnostics (though you are right in discouraging indiscriminate 
> use of stderr).

Just to be more clear, I'm happy with the rest of the stderr messages; 
the leak count is useful, and I'm not seeing any cspi output to stderr 
during normal use of our code anymore -- just the 'bye-bye!' message.

- Chris. 
--

-- 
Chris Ball                                        <cjb <at> mrao.cam.ac.uk>
  Inference Group:            <http://www.inference.phy.cam.ac.uk/is/>
  The Dasher Project:     <http://www.inference.phy.cam.ac.uk/dasher/>
Permalink | Reply | 0 comments |
headers
Bill Haneman | 1 Oct 2003 17:46
Picon

Re: spi_streamablecontent.c ...

On Mon, 2003-09-29 at 13:00, Michael Meeks wrote:
...I attach (an un-tested, un-compiled) patch that
> fixes the most obvious bugs;

Hi Michael:

Feel free to compile and commit if no obvious breakage occurs, otherwise
I will commit (if I don't hear back from you in a day or so).

thanks

Bill
Permalink | Reply | 1 comment |
headers
Michael Meeks | 1 Oct 2003 18:40

Re: spi_streamablecontent.c ...


On Wed, 2003-10-01 at 16:46, Bill Haneman wrote:
> Feel free to compile and commit if no obvious breakage occurs, otherwise
> I will commit (if I don't hear back from you in a day or so).

	It'd be great if you could test/commit - thanks; just passing through
quickly for Matthew.

	Regards,

		Michael.

--

-- 
 michael <at> ximian.com  <><, Pseudo Engineer, itinerant idiot
Permalink | Reply | 0 comments |
headers
padraig o'briain | 2 Oct 2003 15:07
Picon

Accessibility of Workspace Switcher applet

Remus has pointed out to me that the Workspace Switcher applet is not accessible.

Do we need to log a bug about this or is it the case that there is a different accessible way to perform the
functionality provided by this applet?

Padraig
Permalink | Reply | 6 comments |
headers
Luis Villa | 2 Oct 2003 15:27

Re: Accessibility of Workspace Switcher applet

On Thu, 2003-10-02 at 09:07, padraig o'briain wrote:
> Remus has pointed out to me that the Workspace Switcher applet is not accessible.
> 
> Do we need to log a bug about this or is it the case that there is a different accessible way to perform the
functionality provided by this applet?

A bug was filed about this in early 2002, and that the conclusion at the
time was that it was sort of nonsensical to try to accessibilize it- the
'core' functionality (switching workspaces, moving windows) are
available from default metacity keybindings. The secondary functionality
(displaying graphically the location of windows in relationship to each
other) was judged to not be useful in context- if you're unable to see
the applet, semantic information like 'the window is in the upper left
hand corner' was judged not likely to be useful/important.

Luis
Permalink | Reply | 0 comments |
headers
remus draica | 2 Oct 2003 15:32
Picon

Re: Accessibility of Workspace Switcher applet


Hi,

The workspace switcher is accessible, but is no way to change the current 
selection using only keyboard. Perhaps I don't know how. I suceed to change 
it only using mouse.

Remus

On Thursday 02 October 2003 16:07, padraig o'briain wrote:
> Remus has pointed out to me that the Workspace Switcher applet is not
> accessible.
>
> Do we need to log a bug about this or is it the case that there is a
> different accessible way to perform the functionality provided by this
> applet?
>
>
> Padraig
> _______________________________________________
> Gnome-accessibility-devel mailing list
> Gnome-accessibility-devel <at> gnome.org
> http://mail.gnome.org/mailman/listinfo/gnome-accessibility-devel
Permalink | Reply | 4 comments |
headers
Bill Haneman | 2 Oct 2003 15:34
Picon

Re: Accessibility of Workspace Switcher applet

Hi Remus:

Control-Alt-arrow (left or right arrow) should change your current
workspace.  If you want to move windows from workspace to workspace, you
can do that from the metacity Alt-spacebar menu.

Is that what you meant?

- Bill

On Thu, 2003-10-02 at 14:32, remus draica wrote:
> 
> Hi,
> 
> The workspace switcher is accessible, but is no way to change the current 
> selection using only keyboard. Perhaps I don't know how. I suceed to change 
> it only using mouse.
> 
> Remus
> 
> On Thursday 02 October 2003 16:07, padraig o'briain wrote:
> > Remus has pointed out to me that the Workspace Switcher applet is not
> > accessible.
> >
> > Do we need to log a bug about this or is it the case that there is a
> > different accessible way to perform the functionality provided by this
> > applet?
> >
> >
> > Padraig
(Continue reading)

Permalink | Reply | 3 comments |
headers
David Bolter | 2 Oct 2003 15:43
Picon
Picon
Favicon

The Skipper

Interesting project we should be aware of:

http://skipper.sourceforge.net/

(Sorry about the cross post and no, I'm not involved with this project)

cheers

David
Permalink | Reply | 0 comments |

Gmane