sysutils/lazygit
1
0
Fork 0
mirror of https://github.com/jesseduffield/lazygit.git synced 2025-07-31 14:24:25 +03:00
Code Activity

Cleanup: remove unused log parameter of patch_exploring.NewState

Browse Source
This commit is contained in:
Stefan Haller
2024-10-16 14:25:38 +02:00
parent 2f1564d288
commit da47498066
3 changed files with 4 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
pkg/gui/controllers/helpers/patch_building_helper.go
View File

@ -91,7 +91,7 @@ func (self *PatchBuildingHelper) RefreshPatchBuildingPanel(opts types.OnFocusOpt
oldState := context.GetState() oldState := context.GetState()
state := patch_exploring.NewState(diff, selectedLineIdx, oldState, self.c.Log) state := patch_exploring.NewState(diff, selectedLineIdx, oldState)
context.SetState(state) context.SetState(state)
if state == nil { if state == nil {
self.Escape() self.Escape()

4
pkg/gui/controllers/helpers/staging_helper.go
View File

@ -63,11 +63,11 @@ func (self *StagingHelper) RefreshStagingPanel(focusOpts types.OnFocusOpts) {
secondaryContext.GetMutex().Lock() secondaryContext.GetMutex().Lock()
mainContext.SetState( mainContext.SetState(
patch_exploring.NewState(mainDiff, mainSelectedLineIdx, mainContext.GetState(), self.c.Log), patch_exploring.NewState(mainDiff, mainSelectedLineIdx, mainContext.GetState()),
) )
secondaryContext.SetState( secondaryContext.SetState(
patch_exploring.NewState(secondaryDiff, secondarySelectedLineIdx, secondaryContext.GetState(), self.c.Log), patch_exploring.NewState(secondaryDiff, secondarySelectedLineIdx, secondaryContext.GetState()),
) )
mainState := mainContext.GetState() mainState := mainContext.GetState()

3
pkg/gui/patch_exploring/state.go
View File

@ -3,7 +3,6 @@ package patch_exploring
import ( import (
"github.com/jesseduffield/generics/set" "github.com/jesseduffield/generics/set"
"github.com/jesseduffield/lazygit/pkg/commands/patch" "github.com/jesseduffield/lazygit/pkg/commands/patch"
"github.com/sirupsen/logrus"
) )
// State represents the current state of the patch explorer context i.e. when // State represents the current state of the patch explorer context i.e. when
@ -29,7 +28,7 @@ const (
HUNK HUNK
) )
func NewState(diff string, selectedLineIdx int, oldState *State, log *logrus.Entry) *State { func NewState(diff string, selectedLineIdx int, oldState *State) *State {
if oldState != nil && diff == oldState.diff && selectedLineIdx == -1 { if oldState != nil && diff == oldState.diff && selectedLineIdx == -1 {
// if we're here then we can return the old state. If selectedLineIdx was not -1 // if we're here then we can return the old state. If selectedLineIdx was not -1
// then that would mean we were trying to click and potentiall drag a range, which // then that would mean we were trying to click and potentiall drag a range, which