Proposal: Add option to memoize toString()/hashCode()

apollographql / apollo-kotlin

:rocket: A strongly-typed, caching GraphQL client for the JVM, Android, and Kotlin multiplatform.

https://www.apollographql.com/docs/kotlin

MIT License

3.76k stars 653 forks source link

Proposal: Add option to memoize toString()/hashCode() #420

Closed ZacSweers closed 7 years ago

ZacSweers commented 7 years ago

Since the models are immutable, memoizing hashCode/toString() would be nice. This is a common use case for AutoValue's @Memoized extension as well, and would make its implementation a good reference.

sav007 commented 7 years ago

@hzsweers is this issue duplicate of this one https://github.com/apollographql/apollo-android/issues/423

ZacSweers commented 7 years ago

Oh wat. Github gave me an error earlier so I opened this one in the process of trying again. Closed the other ¯_(ツ)_/¯