dynein icon indicating copy to clipboard operation
dynein copied to clipboard

Refactor: Reduce clone of context object

Open StoneDot opened this issue 3 years ago • 0 comments

Issue #, if available: This PR reduces the use of clone. There are primary and accompanying goals such as:

  • Reduce clone to improve performance.
  • Currently, there is not a good way to update context cache without quitting the program. It causes incoherent cache on shell mode. Mutable reference is a way to solve this problem.
  • Give future ability into context to hold another cache like "item cache," which is required to implement completions.

This PR is related to #6.

I am welcome the discussion to achieve the above goals.

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

StoneDot avatar Nov 29 '21 10:11 StoneDot