lines().count() should warn in case of Err

What it does

As stated here, Lines can enter an infinite loop if nothing like map_while is used when used on directory. Therefore something like lines().count() may enter an infinite loop. Therefore, it would be good if a warn can be added to fix like : lines().map_while(Result::ok).count().

Advantage

• It will avoid entering a loop when used on directory in certain platforms.

Drawbacks

• Some users may be interested in getting the count with lines with errors included.

Example

let linecount = BufReader::new(File::open("some-path")?).lines().count();

Could be warned and this used instead :

let linecount = BufReader::new(File::open("some-path")?).lines().map_while(Result::ok).count();

Start by reviewing the linked map_while lint documentation and the issue's lines().count() example. Determine how the warning should handle line-reading errors and the case where callers intentionally count them; done means the requested case is covered by a Clippy warning with an appropriate suggested alternative.