`<iomanip>`: consider removing `operator>>` for `istream >> setfill(c)`

According to the standard([this version](https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2023/n4944.pdf)), `std::setfill` is only for ostreams. Therefore, `istream >> setfill(c)` has no well-defined meaning and can lead to [confusion](https://stackoverflow.com/questions/42530114/stdsetfill-and-stdsetw-for-input-streams).
![image](https://github.com/microsoft/STL/assets/60953653/7d3b328c-88ae-4eed-9e7f-ee11a2d5b3f7)
The following piece of code should get removed.
https://github.com/microsoft/STL/blob/a62109595b6d89e08172fdf4beb75a2670fe0cc9/stl/inc/iomanip#L28-L35

	template <class _Elem2, class _Traits>
	friend basic_istream<_Elem2, _Traits>& operator>>(basic_istream<_Elem2, _Traits>& _Istr, const _Fillobj& _Manip) {
	// set fill character in input stream
	static_assert(is_same_v<_Elem, _Elem2>, "wrong character type for setfill");

	_Istr.fill(_Manip._Fill);
	return _Istr;
	}

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

`<iomanip>`: consider removing `operator>>` for `istream >> setfill(c)` #3724

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

<iomanip>: consider removing operator>> for istream >> setfill(c) #3724

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions

`<iomanip>`: consider removing `operator>>` for `istream >> setfill(c)` #3724