Factor out pluralisation checks in diagnostics

[varkor]

The following pattern is common in diagnostic messages in rustc:

format!("found {} thing{}", x, if x != 1 { "s" } else { "" })

It would be good to extract all of these pluralisation checks into a function or macro:

format!("found {} thing{}", x, pluralise(x))

There's already one in src/librustc/ty/error.rs, but it's not used anywhere else, so we could move it to somewhere in src/librustc_errors, and replace the occurrences of this pattern with the macro.

rust/src/librustc/ty/error.rs

Lines 85 to 89 in 0b97726

	macro_rules! pluralise {
	($x:expr) => {
	if $x != 1 { "s" } else { "" }
	};
	}

This issue has been assigned to @V1shvesh via this comment.

[V1shvesh]

I'd love to be assigned to this issue!

[Centril]

@rustbot assign @V1shvesh

[V1shvesh]

So, I have moved the macro under src/librustc_errors in a file called pluralise.rs. I have also located all the occurrences of the pattern.

I am a bit new to Rust, so I still am learning about the module system. How shall I pull the macro into the scope of all the relevant files?

[varkor]

@V1shvesh: I'd put it in src/librustc_errors/lib.rs, so we don't have one new file for a single macro. Then you can use it from use syntax::errors;.

[V1shvesh]

Thanks for pointing me in the right direction @varkor!