small improvement on the capabilities config comments

[garyyu]

John Tromp @tromp 16:23
it says Bit flags for FULL_NODE and then bits = 6
#7 = Bit flags for FULL_NODE, this structure needs to be changed
#internally to make it more configurable
[server.p2p_config.capabilities]
bits = 6

---

Gary Yu @garyyu 16:24
it says 7= … FULL_NODE. Confused?

---

John Tromp @tromp 16:25
6 is for FAST_SYNC_NODE
oh i see why im confused
the 7 is not separate from #

---

John Tromp @tromp 16:27
i read it as a garbled # bit flags for FULL_NODE

---

Gary Yu @garyyu 16:28
Ok, will give a minor change on this comment.

[antiochp]

Do we still have an archival_mode flag in the config or is this fully replaced with this value 6?

[jaspervdm]

According to comments.rs:88 it still exists

[garyyu]

Do we still have an archival_mode flag in the config or is this fully replaced with this value 6?

Yes. And archival_mode has higher priority to overwrite bits.
According to Ingo's explain, it's much easier for user to config archival_mode instead of bits, that's why we want to keep archival_mode.

[jaspervdm]

So if archival_mode takes priority over bits, why is bits configurable? Doesn't it make sense to set it automatically depending on archival_mode

[garyyu]

good point! I guess that's why there's a comment there 😄 a remaining todo item.

#This structure needs to be changed internally, to make it more configurable