Golangのエラー処理について

Question

質問

例えば

func Hoge() error {
    // なにか
}

のようなerrorのみを返す関数があるとして，

これが他の関数内の最後で呼び出されるとして，
Hogeから返されたerrorをHugeのerrorとして返せば良いだけの場合以下の書き方で問題ないでしょうか？

func Huge() error {
    //なにか
    return Hoge()

それともこっちでしょうか？(このほうが読みやすい気はする)

func Huge() error {
    //なにか
    if err := Hoge(); err != nil {
        return err
    }
    return nil

(追記)
もちろんHogeの返り値によってなにか追加で処理を行ったり，
Hugeが複数の返り値を持っていたりしたら2つめだとは思うんですが，
あくまで上記のようにerrorを伝搬?すれば良いだけの場合は
2つめだと冗長と言われればそんな気もするし...って感じです

Answer 1

書き方として問題があるかというと、無いと思います。下記のように error を返す関数で、return に関数があり、その呼び出し結果を return する書き方はあります。

func Huge() error {
    //なにか
    return Hoge()
}

例えば net.http の http.Get 関数は以下のようになっています。

https://github.com/golang/go/blob/master/src/net/http/client.go#L446-L448

func Get(url string) (resp *Response, err error) {
	return DefaultClient.Get(url)
}