remove unnecessary context branching (#294)

x/ibc-hooks/move-hooks/receive.go

@@ -124,16 +124,10 @@ func (h MoveHooks) execMsg(ctx sdk.Context, msg *movetypes.MsgExecute) (*movetyp
 	}
 
 	moveMsgServer := movekeeper.NewMsgServerImpl(h.moveKeeper)
-
-	// use cache context to prevent state changes if the message execution fails
-	cacheCtx, write := ctx.CacheContext()
-	res, err := moveMsgServer.Execute(cacheCtx, msg)
+	res, err := moveMsgServer.Execute(ctx, msg)
 	if err != nil {
 		return nil, err
 	}
 
-	// write the cache context only if the message execution was successful
-	write()
-
 	return res, nil
 }