refactor: rename L1FeeEnabled to FeeVaultEnabled

[HAOYUatHZ]

to address #262 (comment)

[colinlyguo]

I left a new comment in the last merged PR: blockTrace.WithdrawTrieRoot = withdrawtrie.ReadWTRSlot(rcfg.L2MessageQueueAddress, env.state) is not controlled by any config now. Is it compatible with other geth configs?

[HAOYUatHZ]

I left a new comment in the last merged PR: blockTrace.WithdrawTrieRoot = withdrawtrie.ReadWTRSlot(rcfg.L2MessageQueueAddress, env.state) is not controlled by any config now. Is it compatible with other geth configs?

I think we don't need to be compatible with other geth config.
and by default ReadWTRSlot will return 0 if we don't set up L2MessageQueue contract?

but yeah I do agree it will be great if it can be "controlled by some config"
we can leave it as a future improvement

[colinlyguo]

I left a new comment in the last merged PR: blockTrace.WithdrawTrieRoot = withdrawtrie.ReadWTRSlot(rcfg.L2MessageQueueAddress, env.state) is not controlled by any config now. Is it compatible with other geth configs?

I think we don't need to be compatible with other geth config. and by default ReadWTRSlot will return 0 if we don't set up L2MessageQueue contract?

but yeah I do agree it will be great if it can be "controlled by some config" we can leave it as a future improvement

oh, I see, then this modification makes sense since it's just used as a scroll geth.