Luca Kellermann
Luca Kellermann
also relevant: https://github.com/discord/discord-api-docs/pull/7109 and https://github.com/discord/discord-api-docs/pull/6959 (`40018` and `40019`)
just have to add subscriptions (https://github.com/discord/discord-api-docs/pull/7109) and then this will finally be done
> just have to add subscriptions ([discord/discord-api-docs#7109](https://github.com/discord/discord-api-docs/pull/7109)) and then this will finally be done just missing in core now
> Would love to see this feature merged after a review, conflicts resolved and fixes i can take a look, but i'll be on vacation for a week first
also relevant: https://github.com/discord/discord-api-docs/pull/7090
also https://github.com/discord/discord-api-docs/pull/7050
> In line 149 of > > https://github.com/bitcoin/libbase58/blob/b1dd03fa8d1be4be076bb6152325c6b5cf64f678/base58.c#L149 > , it appears to me that `j` is not initialized and directly used in line 159 to initialize `high`: > >...
could you provide some examples how you think this all should be used (maybe also compared to the old cache api)? that way it will be easier to review because...
what i just realized is a problem with `Cache.filter` and `Cache.firstOrNull`: these will always have O(n) complexity because all objects have to be accessed to be filtered.
> Just consulted google and it should work unless it's the first character, don't know what I did wrong then the js name should probably just be prepended with `_`...