Apply code review suggestions

ethereumjs · Sep 28, 2023 · 4d48796 · 4d48796
1 parent 29b3ac4
commit 4d48796
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/packages/statemanager/src/stateManager.ts b/packages/statemanager/src/stateManager.ts
@@ -962,7 +962,8 @@ export class DefaultStateManager implements EVMStateManagerInterface {
    * Both adoptions can be deactivated by setting `downlevelCaches` to
    * `false`.
    *
-   * Cache values are generally not copied along.
+   * Cache values are generally not copied along regardless of the
+   * `downlevelCaches` setting.
    */
   shallowCopy(downlevelCaches = true): DefaultStateManager {
     const common = this.common.copy()

diff --git a/packages/vm/src/vm.ts b/packages/vm/src/vm.ts
@@ -241,9 +241,11 @@ export class VM {
    *
    * Note that the returned copy will share the same db as the original for the blockchain and the statemanager.
    *
-   * Associated state and trie caches will be deleted and caches will be re-initialized for a more short-term
-   * focused usage, being less memory intense. To fine-tune this behavior (if the shallow copy has a longer life
-   * span e.g.) you can set the `downlevelCaches` option to `false`.
+   * Associated caches will be deleted and caches will be re-initialized for a more short-term focused
+   * usage, being less memory intense (the statemanager caches will switch to using an ORDERED_MAP cache
+   * datastructure more suitable for short-term usage, the trie node LRU cache will not be activated at all).
+   * To fine-tune this behavior (if the shallow-copy-returned object has a longer life span e.g.) you can set
+   * the `downlevelCaches` option to `false`.
    *
    * @param downlevelCaches Downlevel (so: adopted for short-term usage) associated state caches (default: true)
    */