Dear Philip,
Thank you for the review comments.
AFAIK, I used the PlatformLogger which was commonly used in that module.
Just to understand,
1. Is/are there any bug(s) associated in getting rid of PlatformLogger
usage?
2. Is there any new logger design/development planned in the near future?
3. Any new common logger API(s) will be in place for the logging
purpose in the coming days?
Could you please let me know.
Best Regards,
K Suman Rajkumaar
On 21/07/2020 04:05, Philip Race wrote:
PS I also don't think statically initialising a logger even if it is
never needed is a good idea.
-phil.
On 7/20/20, 9:04 AM, Philip Race wrote:
No, not approved. This fix is using sun.util from inside Java,base.
We need to remove the existing cases of this usage. Not add new ones.
-phil.
On 7/20/20, 8:53 AM, Sergey Bylokhov wrote:
Firstly, as the issue was observed with JPEG flavor with alpha
component, I have tested with regular JPEG image (without alpha) and
the image was copied onto the clipboard and no exception was thrown
as expected.
Secondly, tested with JPEG flavor with alpha, there was no JPEG
flavor copied onto the clipboard as the support was removed/dropped
and the IOException is thrown as expected and is logged into the
logger (seen after enabling the logging). Please find the attached
file for reference. Please search for the log message "WARNING:
Registered service providers failed to encode".
Also, there are datatransfer tests available which was run and there
were no new failures observed.
This fix doesn't change any of the functionalities rather, cleans up
the code and logs the exception to the logger instead of printing
the same on the console.
Best Regards,
K Suman Rajkumaar