Frankly I think it's a good idea, it was just stupid to do it silently and by default especially on release builds. It didn't actually send data anywhere unless locally enabled, though.
This is mostly a storm in a teacup, IMO, and if undesired it sounds like it's just necessary to build with VS 2015 update 3. I don't personally see it as particularly different to inserting dtrace events or similar.