nim-lang / RFCs

A repository for your Nim proposals.
136 stars 26 forks source link

remove/disable "conversion from x to itself not needed" hint #471

Closed metagn closed 2 years ago

metagn commented 2 years ago

Abstract

Remove or disable the ConvFromXtoItselfNotNeeded hint by default.

Motivation

No response

Description

The only time this hint is acknowledged in practice is when people have to turn it off in macros. Using TaintedString when it still existed always gave this warning without taint mode on. It has an open issue: https://github.com/nim-lang/Nim/issues/10542.

It's not like this improves performance or anything. It's just a niche refactor.

Code Examples

No response

Backwards Compatibility

No response