refactor live interface

1) i would expect that the first "used" gps position after non-gps event
is mMinLiveLogDelayMillis after...wouldn't you ?

2) if you agree about semantics, then do you agree that you need to move
the assignment
to get that behaviour ?

/Jonas

On Thu, Nov 13, 2014 at 10:03 PM, Niklas Weidemann <notifications@github.com
> wrote:

> In app/src/org/runnerup/export/RunnerUpLive.java:
>
> > @@ -145,11 +152,19 @@ private int translateType(int type) {
> >      }
> >
> >      @Override
> > -    public void liveLog(Context context, Location location, int type, double mElapsedDistanceMeter,
> > -            double mElapsedTimeMillis) {
> > +    public void liveLog(GpsTracker gpsTracker, int type) {
> > +
> > +        if (type == DB.LOCATION.TYPE_GPS) {
> > +            if (System.currentTimeMillis()-mTimeLastLog < mMinLiveLogDelayMillis) {
> > +                return;
> > +            }
> > +            mTimeLastLog = System.currentTimeMillis();
>
> Why?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jonasoreland/runnerup/pull/121/files#r20322755>.
>