fix(indexeddb): Fix implementation of SafeEncode for tuple of 5 elements

[zecakeh]

Regression introduced by merging commit 9707d73 after efdeba7.

Fixes #1489.

Signed-off-by: Kévin Commaille zecakeh@tedomum.fr

[codecov]

Codecov Report

Base: 72.81% // Head: 74.39% // Increases project coverage by +1.58% 🎉

Coverage data is based on head (304b17a) compared to base (b436918).
Patch coverage: 86.44% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1490      +/-   ##
==========================================
+ Coverage   72.81%   74.39%   +1.58%     
==========================================
  Files         121      124       +3     
  Lines       13300    13726     +426     
==========================================
+ Hits         9684    10212     +528     
+ Misses       3616     3514     -102     

Impacted Files	Coverage Δ
crates/matrix-sdk-base/src/rooms/normal.rs	73.27% <ø> (ø)
crates/matrix-sdk-base/src/session.rs	46.66% <ø> (+7.77%)	⬆️
crates/matrix-sdk-base/src/store/mod.rs	82.50% <ø> (+3.92%)	⬆️
crates/matrix-sdk-crypto/src/olm/account.rs	82.28% <ø> (ø)
crates/matrix-sdk-crypto/src/store/error.rs	0.00% <0.00%> (ø)
crates/matrix-sdk-crypto/src/store/mod.rs	69.18% <ø> (+4.44%)	⬆️
crates/matrix-sdk-sled/src/crypto_store.rs	88.88% <ø> (ø)
crates/matrix-sdk-sqlite/src/error.rs	0.00% <0.00%> (ø)
crates/matrix-sdk/src/client/builder.rs	75.42% <ø> (+4.22%)	⬆️
crates/matrix-sdk/src/client/mod.rs	88.01% <ø> (ø)
... and 37 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[jplatte]

Now you're running into #1489 😅

Maybe we should add some #![allow()]s to the affected crates.

[zecakeh]

But at least it's not my fault 😛 .

Maybe we should add some #![allow()]s to the affected crates.

Either that or we pin the rust-nightly version?

[jplatte]

… or we just wait for rust-lang/rust-clippy#10321 before we merge. If it's merged today, that should be fine 😄

[jplatte]

If you want to sort this out now, I think I'd prefer the attributes because they're more obvious and we still get clippy upgrades even if it takes long for the attributes to be removed again.

[zecakeh]

It's fine, I'll wait until we're tired of waiting 😁