It doesn't work for "if with a statement"

[y7ls8i]

if err := veryLongFunctionWithManyLongArguments(veryLongArgument1, veryLongArgument2, veryLongArgument3, veryLongArgument4, veryLongArgument5); err != nil {
	// ... ... ...
}

should have been broken down into:

if err := veryLongFunctionWithManyLongArguments(
	veryLongArgument1,
	veryLongArgument2,
	veryLongArgument3,
	veryLongArgument4,
	veryLongArgument5,
); err != nil {
	// ... ... ...
}

but it doesn't.

[shuaiyy]

the same case

if err = n.db.Unscoped().Model(&models.User{}).Where("id = ?", id).First(&item).Error; err == nil {
}

but below is worked:

err = n.db.Unscoped().Model(&models.NotebookEnv{}).Where("id = ?", id).First(&item).Error
if err == nil {
}

[matthewhughes934]

The IfStmt.Init node isn't considered for shortening in the block:

golines/shortener.go

Line 460 in 13c3f0d

case *dst.IfStmt:

, but even if it was the shortening Decoration is on the IfStmt node and not the Init node so even if we did s.formatStmt(st.Init) in that first block linked the hasAnnotation call in formatStmt will return false"

golines/shortener.go

Line 424 in 13c3f0d

shouldShorten := HasAnnotation(stmt)

I suspect a fix would require adding something like the force parameter like formatExpr has:

golines/shortener.go

Line 485 in 13c3f0d

func (s *Shortener) formatExpr(expr dst.Expr, force bool, isChain bool) {