GeoAPI
  1. GeoAPI
  2. GEO-128

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

        Hide
        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
        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
        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
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: