Interestingly, this is deprecated in Android too... There is even a comment that says that it does nothing. The code, however, actually does do something, just not very will. I just got through fixing it...
Blake Meike
added a comment - Interestingly, this is deprecated in Android too... There is even a comment that says that it does nothing. The code, however, actually does do something, just not very will. I just got through fixing it...
Deprecated should still work. It just means we can remove the API in some future release, so I hope the fix was making the code do the right thing and not nothing.
Daniel Petersen
added a comment - Deprecated should still work. It just means we can remove the API in some future release, so I hope the fix was making the code do the right thing and not nothing.
Yeah.... It does something plausible, now. It interacts in a very weird way, with the other logging controls we have. ... but it works.
Blake Meike
added a comment - Yeah.... It does something plausible, now. It interacts in a very weird way, with the other logging controls we have. ... but it works.
I remember having a discussion about this in which we agreed that while not ideal, this method should be a no-op. In C# this is both a compiler warning and a runtime logging message. There is no way for us to get it to function 100% like it did before in C# because this also controlled the file logging level (something the other platforms didn't have at the time), and the new file logging API does not use domains.
Jim Borden
added a comment - I remember having a discussion about this in which we agreed that while not ideal, this method should be a no-op. In C# this is both a compiler warning and a runtime logging message. There is no way for us to get it to function 100% like it did before in C# because this also controlled the file logging level (something the other platforms didn't have at the time), and the new file logging API does not use domains.
I like that suggestion. I doubt, very seriously, that any of our users will be able to use this function to do anything expected and useful.
Blake Meike
added a comment - I like that suggestion. I doubt, very seriously, that any of our users will be able to use this function to do anything expected and useful.
Interestingly, this is deprecated in Android too... There is even a comment that says that it does nothing. The code, however, actually does do something, just not very will. I just got through fixing it...