jcarr
/
git2go
1
0
Fork 0
Code Issues 52 Pull Requests 31 Packages Projects Releases Wiki Activity

refactor: Rename methods with Get prefix 

It is not Go idiomatic to put Get into the getter's name.

https: //golang.org/doc/effective_go.html#Getters
Co-Authored-By: lhchavez <lhchavez@users.noreply.github.com>
This commit is contained in:
Suhaib Mujahid 2020-05-09 23:32:17 -04:00
parent 8b51d0db8e
commit cf6522c7fe
5 changed files with 21 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
diff.go
View File

@ -144,7 +144,7 @@ func (diff *Diff) NumDeltas() (int, error) {
return ret, nil
}
func (diff *Diff) GetDelta(index int) (DiffDelta, error) {
func (diff *Diff) Delta(index int) (DiffDelta, error) {
if diff.ptr == nil {
return DiffDelta{}, ErrInvalid
}
@ -154,6 +154,11 @@ func (diff *Diff) GetDelta(index int) (DiffDelta, error) {
return ret, nil
}
// deprecated: You should use `Diff.Delta()` instead.
func (diff *Diff) GetDelta(index int) (DiffDelta, error) {
return diff.Delta(index)
}
func newDiffFromC(ptr *C.git_diff, repo *Repository) *Diff {
if ptr == nil {
return nil

7
index.go
View File

@ -528,7 +528,7 @@ type IndexConflict struct {
Their *IndexEntry
}
func (v *Index) GetConflict(path string) (IndexConflict, error) {
func (v *Index) Conflict(path string) (IndexConflict, error) {
var cancestor *C.git_index_entry
var cour *C.git_index_entry
@ -553,6 +553,11 @@ func (v *Index) GetConflict(path string) (IndexConflict, error) {
return ret, nil
}
// deprecated: You should use `Index.Conflict()` instead.
func (v *Index) GetConflict(path string) (IndexConflict, error) {
return v.Conflict(path)
}
func (v *Index) RemoveConflict(path string) error {
cpath := C.CString(path)

2
merge_test.go
View File

@ -109,7 +109,7 @@ func TestMergeTreesWithoutAncestor(t *testing.T) {
if !index.HasConflicts() {
t.Fatal("expected conflicts in the index")
}
_, err = index.GetConflict("README")
_, err = index.Conflict("README")
checkFatal(t, err)
}

7
settings.go
View File

@ -93,10 +93,15 @@ func EnableCaching(enabled bool) error {
}
}
func GetCachedMemory() (current int, allowed int, err error) {
func CachedMemory() (current int, allowed int, err error) {
return getSizetSizet(C.GIT_OPT_GET_CACHED_MEMORY)
}
// deprecated: You should use `CachedMemory()` instead.
func GetCachedMemory() (current int, allowed int, err error) {
return CachedMemory()
}
func SetCacheMaxSize(maxSize int) error {
return setSizet(C.GIT_OPT_SET_CACHE_MAX_SIZE, maxSize)
}

4
settings_test.go
View File

@ -57,8 +57,8 @@ func TestEnableCaching(t *testing.T) {
checkFatal(t, err)
}
func TestGetCachedMemory(t *testing.T) {
current, allowed, err := GetCachedMemory()
func TestCachedMemory(t *testing.T) {
current, allowed, err := CachedMemory()
checkFatal(t, err)
if current < 0 {