Skip to content

Commit b166343

Browse files
committed
refactor passing by ref to by val
1 parent 89e89f1 commit b166343

File tree

2 files changed

+3
-12
lines changed

2 files changed

+3
-12
lines changed

pkg/systemdtime/systemdtime.go

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -118,20 +118,11 @@ func ToDuration(raw string) (time.Duration, error) {
118118
}
119119

120120
// AdjustTime takes a systemd time adjustment string and uses it to modify a time.Time
121-
func AdjustTime(original *time.Time, adjustment string) (time.Time, error) {
121+
func AdjustTime(original time.Time, adjustment string) (time.Time, error) {
122122
duration, err := ToDuration(adjustment)
123123
if err != nil {
124124
return time.Time{}, err
125125
}
126126

127-
return adjustTimeByDuration(original, duration), nil
128-
}
129-
130-
func adjustTimeByDuration(original *time.Time, adjustment time.Duration) time.Time {
131-
if original == nil {
132-
rightNow := time.Now()
133-
original = &rightNow
134-
}
135-
136-
return original.Add(adjustment)
127+
return original.Add(duration), nil
137128
}

pkg/systemdtime/systemdtime_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ func TestToDuration(t *testing.T) {
125125

126126
func TestAdjustTime(t *testing.T) {
127127
time1 := time.Date(2012, time.May, 12, 5, 0, 0, 0, time.UTC)
128-
time1Mod, err := AdjustTime(&time1, " 4 days 2 hr")
128+
time1Mod, err := AdjustTime(time1, " 4 days 2 hr")
129129
if err != nil {
130130
t.Error(err)
131131
}

0 commit comments

Comments
 (0)