Re: [pdt-dev] Question about debug.daemon plugin.

[Dave Kelsey <d_kelsey@xxxxxxxxxx>]

Thanks for the response Michael, I will
make the change, test it and commit it.

Cheers
Dave Kelsey


pdt-dev-bounces@xxxxxxxxxxx wrote on 06/01/2009 13:37:45:

> The only reason for that IIRC was eliminating the need to load the
PHP
> stuff (debug plug-in has a dependency with PHP core) on startup. I
> don't understand why it's being done in PHPDebugPlugin instead of
> DaemonPlugin, I think the listeners startup code can be moved there.
> 
> 2009/1/6 Dave Kelsey <d_kelsey@xxxxxxxxxx>:
> >
> > I noticed a problem where the debug socket listeners were not
being started
> > in some cases when PDT was first started. It turns out that if
you don't
> > have a PHP file open or the PHP Debug perspective being listed
as an opened
> > perspective (not necessarily the active perspective) then the
socket
> > listeners are not started.
> >
> > Looking at the code I see that the debug.daemon (which contains
the code to
> > start the socket listeners) plugin registers an earlyStartup,
but the
> > starting of that plugin (earlyStartup() or start() ) actually
doesn't do
> > anything, so the earlyStartup is redundent. The code to start
the listeners
> > is actually found in the debug.core plugin in its start() method,
which
> > means that unless this plugin gets loaded the socket listeners
won't start
> > until it does get loaded and this can and does get delay loaded.
> >
> > The obvious answer to this would be that as the debug.daemon
has an
> > earlyStartup registered, that this should start the socket listeners,
not
> > the debug.core plugin and I would like to make this change. Does
anyone know
> > why the code did it this way or is there any good reason why
it shouldn't be
> > changed ? (An alternative would be to have the debug.core plugin
register an
> > earlyStartup and remove the one from the debug.daemon plugin).
> >
> > I would like to get this change into PDT 2.0.1 as well.
> >
> > Cheers
> > Dave Kelsey
> >
> >
> >
> >
> > ________________________________
> >
> > Unless stated otherwise above:
> > IBM United Kingdom Limited - Registered in England and Wales
with number
> > 741598.
> > Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire
PO6 3AU
> >
> >
> >
> >
> >
> > _______________________________________________
> > pdt-dev mailing list
> > pdt-dev@xxxxxxxxxxx
> > https://dev.eclipse.org/mailman/listinfo/pdt-dev
> >
> >
> 
> 
> 
> -- 
> Michael
> _______________________________________________
> pdt-dev mailing list
> pdt-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/pdt-dev

---

Unless stated otherwise above:
IBM United Kingdom Limited - Registered in England and Wales with number 741598.
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU