Obsolete LeakTrackingObjectPool and LeakTrackingObjectPoolProvider

[adityamandaleeka]

LeakTrackingObjectPool doesn't work on release builds, and nothing else depends on it, so we've chosen to mark it (and LeakTrackingObjectPoolProvider) obsolete.

Address #42524

[daiplusplus]

The text "was only intended for use in diagnostic builds" can be interpreted as referring to the user's diag builds of user-code, not Microsoft's builds of the CLR+EE.

I'd phrase it like this, perhaps:

"LeakTrackingObjectPool is only functional in diagnostic builds of the .NET CLR and EE. It does not function in GA/release builds of .NET used by the general public. This type may be removed in a future release."

[adityamandaleeka]

Thanks @Jehoel. It doesn't actually need a special CLR build to work, but I understand your feedback. How about:

LeakTrackingObjectPool<T> was only intended for use in diagnostic builds of .NET. It does not function in any publicly shipped .NET versions and may be removed in a future release.

[daiplusplus]

👍

[halter73]

Suggested change

	[Obsolete("LeakTrackingObjectPool<T> was only intended for internal use in diagnostic builds of .NET. It does not " +
	"function in any publicly shipped .NET versions and may be removed in a future release.")]
	[Obsolete("LeakTrackingObjectPool<T> was only intended for internal use in diagnostic builds of .NET. It never " +
	"functioned in any publicly shipped .NET version and may be removed in a future release.")]

[halter73]

Suggested change

	[Obsolete("LeakTrackingObjectPoolProvider was only intended for internal use in diagnostic builds of .NET. It does not " +
	"function in any publicly shipped .NET versions and may be removed in a future release.")]
	[Obsolete("LeakTrackingObjectPoolProvider was only intended for internal use in diagnostic builds of .NET. It never " +
	"functioned in any publicly shipped .NET version and may be removed in a future release.")]