search

apple / servicetalk

A networking framework that evolves with your application
https://docs.servicetalk.io
Apache License 2.0
915 stars 180 forks source link

capacity-limiter-api: polish javadocs and minor code modifications #3031

Closed daschl closed 2 months ago

daschl commented 2 months ago

This changeset mainly polishes javadocs as well as cleans up the implementations slightly without changing their behavior.

daschl commented 2 months ago

These seem fine to me. Feel free to ignore: while you're here would you like to change the documentation for the meaning of the return values, specifically that the hints have meaning if they're non-negative? The only place I see it used in non-tests is here where only negative values are ignored (eg, 0 has meaning).

Will do this as a follow-up.