diff --git a/pkg/commands/git_commands/main_branches.go b/pkg/commands/git_commands/main_branches.go index 8cf08840a..19f809a4f 100644 --- a/pkg/commands/git_commands/main_branches.go +++ b/pkg/commands/git_commands/main_branches.go @@ -21,7 +21,7 @@ type MainBranches struct { previousMainBranches []string cmd oscommands.ICmdObjBuilder - mutex *deadlock.Mutex + mutex deadlock.Mutex } func NewMainBranches( @@ -32,7 +32,6 @@ func NewMainBranches( c: cmn, existingMainBranches: nil, cmd: cmd, - mutex: &deadlock.Mutex{}, } } diff --git a/pkg/gui/context/filtered_list.go b/pkg/gui/context/filtered_list.go index 0724ecb3b..8164cfa16 100644 --- a/pkg/gui/context/filtered_list.go +++ b/pkg/gui/context/filtered_list.go @@ -16,14 +16,13 @@ type FilteredList[T any] struct { getFilterFields func(T) []string filter string - mutex *deadlock.Mutex + mutex deadlock.Mutex } func NewFilteredList[T any](getList func() []T, getFilterFields func(T) []string) *FilteredList[T] { return &FilteredList[T]{ getList: getList, getFilterFields: getFilterFields, - mutex: &deadlock.Mutex{}, } } diff --git a/pkg/gui/context/merge_conflicts_context.go b/pkg/gui/context/merge_conflicts_context.go index c05eeb614..2ab446c06 100644 --- a/pkg/gui/context/merge_conflicts_context.go +++ b/pkg/gui/context/merge_conflicts_context.go @@ -12,7 +12,7 @@ type MergeConflictsContext struct { types.Context viewModel *ConflictsViewModel c *ContextCommon - mutex *deadlock.Mutex + mutex deadlock.Mutex } type ConflictsViewModel struct { @@ -33,7 +33,6 @@ func NewMergeConflictsContext( return &MergeConflictsContext{ viewModel: viewModel, - mutex: &deadlock.Mutex{}, Context: NewSimpleContext( NewBaseContext(NewBaseContextOpts{ Kind: types.MAIN_CONTEXT, @@ -57,7 +56,7 @@ func (self *MergeConflictsContext) SetState(state *mergeconflicts.State) { } func (self *MergeConflictsContext) GetMutex() *deadlock.Mutex { - return self.mutex + return &self.mutex } func (self *MergeConflictsContext) SetUserScrolling(isScrolling bool) { diff --git a/pkg/gui/context/patch_explorer_context.go b/pkg/gui/context/patch_explorer_context.go index eb79dce86..c9144e1ab 100644 --- a/pkg/gui/context/patch_explorer_context.go +++ b/pkg/gui/context/patch_explorer_context.go @@ -15,7 +15,7 @@ type PatchExplorerContext struct { viewTrait *ViewTrait getIncludedLineIndices func() []int c *ContextCommon - mutex *deadlock.Mutex + mutex deadlock.Mutex } var ( @@ -36,7 +36,6 @@ func NewPatchExplorerContext( state: nil, viewTrait: NewViewTrait(view), c: c, - mutex: &deadlock.Mutex{}, getIncludedLineIndices: getIncludedLineIndices, SimpleContext: NewSimpleContext(NewBaseContext(NewBaseContextOpts{ View: view, @@ -137,7 +136,7 @@ func (self *PatchExplorerContext) NavigateTo(selectedLineIdx int) { } func (self *PatchExplorerContext) GetMutex() *deadlock.Mutex { - return self.mutex + return &self.mutex } func (self *PatchExplorerContext) ModelSearchResults(searchStr string, caseSensitive bool) []gocui.SearchPosition { diff --git a/pkg/gui/controllers/helpers/inline_status_helper.go b/pkg/gui/controllers/helpers/inline_status_helper.go index 2e1cf29e0..38a4e2cf7 100644 --- a/pkg/gui/controllers/helpers/inline_status_helper.go +++ b/pkg/gui/controllers/helpers/inline_status_helper.go @@ -15,7 +15,7 @@ type InlineStatusHelper struct { windowHelper *WindowHelper contextsWithInlineStatus map[types.ContextKey]*inlineStatusInfo - mutex *deadlock.Mutex + mutex deadlock.Mutex } func NewInlineStatusHelper(c *HelperCommon, windowHelper *WindowHelper) *InlineStatusHelper { @@ -23,7 +23,6 @@ func NewInlineStatusHelper(c *HelperCommon, windowHelper *WindowHelper) *InlineS c: c, windowHelper: windowHelper, contextsWithInlineStatus: make(map[types.ContextKey]*inlineStatusInfo), - mutex: &deadlock.Mutex{}, } } diff --git a/pkg/gui/gui.go b/pkg/gui/gui.go index c38685d3a..ac1be6601 100644 --- a/pkg/gui/gui.go +++ b/pkg/gui/gui.go @@ -118,7 +118,7 @@ type Gui struct { // is being pushed). At the moment the rule is to use an item operation when // we need to talk to the remote. itemOperations map[string]types.ItemOperation - itemOperationsMutex *deadlock.Mutex + itemOperationsMutex deadlock.Mutex PrevLayout PrevLayout @@ -684,8 +684,7 @@ func NewGui( InitialDir: initialDir, afterLayoutFuncs: make(chan func() error, 1000), - itemOperations: make(map[string]types.ItemOperation), - itemOperationsMutex: &deadlock.Mutex{}, + itemOperations: make(map[string]types.ItemOperation), } gui.PopupHandler = popup.NewPopupHandler(