moved nolint hints

Author: ShishkovEM

internal/conn/conn.go

@@ -158,8 +158,7 @@ func (c *conn) setState(ctx context.Context, s State) State {
 	return s
 }
 
-//nolint:ifshort //here a pointer is intentionally saved under the lock
-func (c *conn) Unban(ctx context.Context) State {
+func (c *conn) Unban(ctx context.Context) State { //nolint:ifshort
 	var newState State
 	c.mtx.RLock()
 	cc := c.grpcConn

internal/decimal/decimal.go

@@ -54,23 +54,23 @@ func Err() *big.Int { return big.NewInt(0).Set(err) }
 //
 // If given bytes contains value that is greater than given precision it
 // returns infinity or negative infinity value accordingly the bytes sign.
-func FromBytes(bts []byte, precision, scale uint32) *big.Int {
+func FromBytes(bts []byte, precision, scale uint32) *big.Int { //nolint:ifshort
 	v := big.NewInt(0)
 	if len(bts) == 0 {
 		return v
 	}
 
 	v.SetBytes(bts)
-
-	if neg := bts[0]&negMask != 0; neg {
+	neg := bts[0]&negMask != 0
+	if neg {
 		// Given bytes contains negative value.
 		// Interpret is as two's complement.
 		not(v)
 		v.Add(v, one)
 		v.Neg(v)
 	}
 	if v.CmpAbs(pow(ten, precision)) >= 0 {
-		if neg := bts[0]&negMask != 0; neg {
+		if neg {
 			v.Set(neginf)
 		} else {
 			v.Set(inf)
@@ -88,9 +88,7 @@ func FromInt128(p [16]byte, precision, scale uint32) *big.Int {
 
 // Parse interprets a string s with the given precision and scale and returns
 // the corresponding big integer.
-//
-//nolint:ifshort
-func Parse(s string, precision, scale uint32) (*big.Int, error) {
+func Parse(s string, precision, scale uint32) (*big.Int, error) { //nolint:ifshort
 	if scale > precision {
 		return nil, precisionError(s, precision, scale)
 	}
@@ -192,9 +190,7 @@ func Parse(s string, precision, scale uint32) (*big.Int, error) {
 
 // Format returns the string representation of x with the given precision and
 // scale.
-//
-//nolint:ifshort //Can't place variable inside if scope because it's used below
-func Format(x *big.Int, precision, scale uint32) string {
+func Format(x *big.Int, precision, scale uint32) string { //nolint:ifshort
 	switch {
 	case x.CmpAbs(inf) == 0:
 		if x.Sign() < 0 {
@@ -290,8 +286,7 @@ func BigIntToByte(x *big.Int, precision, scale uint32) (p [16]byte) {
 	return p
 }
 
-//nolint:ifshort //Can't place variable inside if scope because it's used one more time below
-func put(x *big.Int, p []byte) {
+func put(x *big.Int, p []byte) { //nolint:ifshort
 	neg := x.Sign() < 0
 	if neg {
 		x = complement(x)

internal/topic/topicwriterinternal/writer_reconnector.go

@@ -327,8 +327,7 @@ func (w *WriterReconnector) Flush(ctx context.Context) error {
 	return w.queue.WaitLastWritten(ctx)
 }
 
-//nolint:ifshort //putting flushErr into if scope may cause random fails during closing topicwriter with flush
-func (w *WriterReconnector) Close(ctx context.Context) error {
+func (w *WriterReconnector) Close(ctx context.Context) error { //nolint:ifshort
 	reason := xerrors.WithStackTrace(errStopWriterReconnector)
 	w.queue.StopAddNewMessages(reason)