On 24 Feb 2018, at 17:58, Miancai Li ***@***.***> wrote: Please answer these questions before submitting your issue. Thanks! 437 func (t Time) abs() uint64 { 438 l := t.loc 439 // Avoid function calls when possible. 440 if l == nil || l == &localLoc { 441 l = l.get() 442 } 443 sec := t.sec + internalToUnix 444 if l != &utcLoc { 445 if l.cacheZone != nil && l.cacheStart <= sec && sec < l.cacheEnd { 446 sec += int64(l.cacheZone.offset) 447 } else { 448 _, offset, _, _, _ := l.lookup(sec) 449 sec += int64(offset) 450 } 451 } 452 return uint64(sec + (unixToInternal + internalToAbsolute)) 453 } What version of Go are you using (go version)? all go version I have check, go 1.7, 1.8 and go 1.9 Does this issue reproduce with the latest release? yes What operating system and processor architecture are you using (go env)? nothing about th env What did you do? code review If possible, provide a recipe for reproducing the error. A complete runnable program is good. A link on play.golang.org is best. What did you expect to see? line 440, if l is nil, should not use l.get(). What did you see instead? // abs returns the time t as an absolute time, adjusted by the zone offset. // It is called when computing a presentation property like Month or Hour. I suggest to be 440 if l != nil && l == &localLoc { — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.