You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
BoxFuture and TryFuture are already in lib, but why not BoxTryFuture? The same goes for BoxTryStream and TryStream. I'm declaring both of them quite often, maybe I'm not alone with this problem, so why not add them to futures after all?
The text was updated successfully, but these errors were encountered:
Only 4 seem to be unique (tide, tinychain, txn_lock, tokio-tide, and tokio-tide's code is in comment). All the others seem to be forks of tide and tinychain.
You're right. So you consider this extra for now? I still think that this is quite useful.
Also, the search can be extended by looking for the usage of BoxFuture<Result*.
I can see that the usage of BoxTryFuture<Ok, Err> instead of BoxFuture<Result<Ok, Err>> could be controversial. Especially in cases when Error is hidden in a type alias like BoxFuture<Result<Ok>>, because for someone the last one is more convenient. But refactoring this:
BoxFuture
andTryFuture
are already in lib, but why notBoxTryFuture
? The same goes forBoxTryStream
andTryStream
. I'm declaring both of them quite often, maybe I'm not alone with this problem, so why not add them tofutures
after all?The text was updated successfully, but these errors were encountered: