Bug 200756

Comment 1 Brian Bauman 2007-08-22 00:43:49 EDT

I was actually going to fix that when I fixed the bidi problem.  So if no one takes it for this month's bugday, I will :)

Comment 2 Remy Suen 2007-08-28 07:46:45 EDT

Created attachment 77120 [details]
Patch to remove the class and the subsequent list of unused imports.

Comment 3 Brian Bauman 2007-08-28 10:23:53 EDT

Hi Remy, I will go ahead and assign you the bug since you attached a patch, but you might want to go back and look at the patch.  It seems there might have been some confusion some where as the patch submitted seems to be to a different bug :)

Comment 4 Remy Suen 2007-08-28 10:28:11 EDT

Created attachment 77135 [details]
Oh, here's the right one. :(

Comment 5 Chris Aniszczyk 2007-08-28 10:29:38 EDT

That's what Mylyn is for Remy ;)

Comment 6 Brian Bauman 2007-08-28 11:38:01 EDT

Thanks Remy!  I applied the patch as is.  

The only thing I added was removing a PDEUIMessages and pderesources.properties entry which the ExportPackageDialogLabelProvider referenced.  The entries were no longer referenced by anyone.

Good luck with the patch for the other bug ;-)

Also, we have a page with the photos of all the PDE UI Committers and Contributors (http://www.eclipse.org/pde/pde-ui/committers/committers.php).  Since you have now contributed to the component, we would love to put your picture up with everyone else's.  If you can, please send me a head shot with preferably a leafy background and we will make sure you make it up there :)