search

locationtech / udig-platform

uDig parent project containing all core components. More plugins can be found in community repos: http://github.com/uDig-Community
http://udig.refractions.net
189 stars 133 forks source link

Use SubMonitor in udig.project and udig.project.ui #591

Closed sschulz92 closed 2 years ago

sschulz92 commented 2 years ago

With https://github.com/locationtech/udig-platform/pull/590 and this PR, I managed to get all SubProgressMonitor places to be updated :)

image

I assume this part to be done with this PR beeing merged.

sschulz92 commented 2 years ago

May I ask: how long are those strings there? Do you think this kind of change will be done sooner than later?

In Eclipse warnings are shown which I want to get rid of. I would prefer TODO comments, to externalize those strings. What do you think?

fgdrf commented 2 years ago

May I ask: how long are those strings there? Do you think this kind of change will be done sooner than later?

In Eclipse warnings are shown which I want to get rid of. I would prefer TODO comments, to externalize those strings. What do you think?

Hmm, thats why this are warnings in the IDE, I guess no need to add aditional TODO right there. I guess the challange is to identify those messages to export and those that are not visible in UI