Skip to content
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

Fix confusing Javadoc of memoizeWithExpiration #5812

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix confusing Javadoc of memoizeWithExpiration #5812

wants to merge 1 commit into from

Commits on Dec 6, 2021

  1. Fix confusing Javadoc of memoizeWithExpiration 

    The previous wording could be read such that the cache is cleared in
the background once the expiration time is reached. However this is
not the case, so we simply remove the following part of the Javadoc:
"and removes the cached value after the specified time has passed"
The rest of the Javadoc already accurately describes the behavior,
though the part with "After the expiration time" could again be
misinterpreted, so we rewrite this to clarify that things only
happen once the "get" method is called.
    @MartinSpiessl
    MartinSpiessl committed Dec 6, 2021
    Configuration menu
    Copy the full SHA
    4c49bfc View commit details
    Browse the repository at this point in the history