Code review comment for lp:~javierder/bzr-eclipse/send_command

Revision history for this message
Javier Derderyan (javierder) wrote :

Ok,
Based on your input I fixed the issues you mentioned.

About the cancel, I just changed the variable to allow cancel, is that enough or I have to do something else?

About the browse button, I'll add it to my list of things for the next month, because I'm still trying to be friend with Java and Eclipse and I have no idea how to do it.
It's first in my to-do list.

I'll check with canonical about the Copyright headers.

Thanks again!

« Back to merge proposal