GeoAPI

ProgressListener getProgress()

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.2
  • Fix Version/s: 2.2-M1
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    0

Description

Started getting some hands on time with ProgressListener today; and was unable to make a sub ProgressListener (which is an important use case when you are trying to compose several steps and report progress back to the user...).

As such I need to add the following method to the api:

float getProgress()

This is used when starting a sub progress listener: new SubProgressListener( progress, 30.0f )

Basically you need to know the starting point; so as the sub progress listener contributes its 30% to the overall progress it can correctly produce a final percentage for the parent...

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Jody Garnett added a comment -

Change has been committed and deployed.

Show
Jody Garnett added a comment - Change has been committed and deployed.
Jody Garnett made changes -
Field Original Value New Value
Resolution Fixed [ 1 ]
Status Open [ 1 ] Closed [ 6 ]
Hide
Permalink
Martin Desruisseaux added a comment -

Looks fine. But maybe we should rename progress(float) as setProgress(float) for consistency? The old progress(float) method would need to be deprecated of course.

Just for archive, JavaDoc is there:
http://geoapi.sourceforge.net/2.1/javadoc/org/opengis/util/ProgressListener.html

Show
Martin Desruisseaux added a comment - Looks fine. But maybe we should rename progress(float) as setProgress(float) for consistency? The old progress(float) method would need to be deprecated of course. Just for archive, JavaDoc is there: http://geoapi.sourceforge.net/2.1/javadoc/org/opengis/util/ProgressListener.html
Hide
Permalink
Jody Garnett added a comment -

Up to you I don't mind either way - I am happy that I can now use this interface for what I need - docs here:

So far I have not found any code making use of InternationalString

Show
Jody Garnett added a comment - Up to you I don't mind either way - I am happy that I can now use this interface for what I need - docs here: http://docs.codehaus.org/display/GEOTDOC/ProgressListener So far I have not found any code making use of InternationalString
Martin Desruisseaux made changes -
Assignee Jody Garnett [ jgarnett ]
Status Closed [ 6 ] Reopened [ 4 ]
Resolution Fixed [ 1 ]
Martin Desruisseaux made changes -
Resolution Fixed [ 1 ]
Status Reopened [ 4 ] Closed [ 6 ]
Fix Version/s 2.2 [ 12386 ]
Fix Version/s 2.2-M1 [ 14472 ]

People

  • Assignee:
    Jody Garnett
    Reporter:
    Jody Garnett
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: