New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

add method get age in ms for Message class #1296

Open

apirom9 wants to merge 1 commit into restlet:2.4 from apirom9:2.4-add-age-method-for-message

apirom9 commented Oct 13, 2017

add method get age in ms for Message class


          add method get age in ms for Message class

8bcca18

thboileau self-assigned this

thboileau force-pushed the 2.4 branch 4 times, most recently from 660ec16 to 423c818 Compare

October 26, 2017 20:23

Collaborator

Tembrel commented Jan 30, 2018

Does this really carry its weight? If so, how about:

  public long getAge(TimeUnit unit) {
      Date date = getDate();
      if (date == null) {
          return Long.MIN_VALUE; // Long.MAX_VALUE?
      } else {
          long ageMillis = System.currentTimeMillis() - date.getTime();
          return unit.convert(ageMillis, TimeUnit.MILLISECONDS);
      }
  }

You want to be able to distinguish the case of the age being 0 from the case where there is no date.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet