perfview icon indicating copy to clipboard operation
perfview copied to clipboard

Hide Utilities namespace

Open andkamashev opened this issue 4 years ago • 2 comments

Lots of devs has Utilities static class(es) in project and after installing this nuget all calls of methods like Utilities.Sum(228,1488) throw error on compilation. It is quite easy to fix, but not if you have hundreds of projects.

andkamashev avatar Jun 23 '21 20:06 andkamashev

From a quick look at the Utilities namespace, it looks like this would not change any public surface area. Would you be interested in submitting a PR to merge the Utilities namespace with Microsoft.Diagnostics.Utilities?

brianrob avatar Jun 24 '21 03:06 brianrob

Sure, it will be nice, anything except Utilities namespace will be perfect (even Microsoft.Utilities is better)

andkamashev avatar Jun 24 '21 10:06 andkamashev

Any chance of getting these namespaces cleaned up, we are hitting this same issue as https://github.com/microsoft/perfview/issues/1049

JpWebster avatar Feb 07 '23 15:02 JpWebster

Fixed via #1805.

brianrob avatar Feb 09 '23 00:02 brianrob