<AWT Dev> RFR: [12] JDK-8196681: Java Access Bridge logging and debug flags dynamically controlled

[Shashidhara Veerabhadraiah]

Hi Krishna, I have imported the patch and used it for some testing. The debug logs helped with debug information of the actions that I performed with a demo application. Below are few suggestions:

1. The Windows log file did not had any extensions whereas the java log had the file extension. May be a default extension could be built in.
2. Another thing I felt was that the windows log file was populated by the access bridge module causing it to write it for any version of java usage and whereas Java log file will be written for the built java module. Since both of them are behaving differently can we have 2 separate variables for it instead of one? Since the windows log file can grow to larger size and for debugging java log is sufficient at times, we can keep it separate.

Otherwise the changes looks good to me.

Thanks and regards,
Shashi

-----Original Message-----
From: Krishna Addepalli 
Sent: Wednesday, October 17, 2018 1:04 AM
To: Sergey Bylokhov <sergey.bylokhov at oracle.com>; Prasanta Sadhukhan <prasanta.sadhukhan at oracle.com>; awt-dev at openjdk.java.net
Subject: Re: <AWT Dev> RFR: [12] JDK-8196681: Java Access Bridge logging and debug flags dynamically controlled

Hi Sergey,

Per our conversation, I have made the following changes:
1. Add logging support to WindowsAccessBridge as well.
2. Added timestamp info in the log file.
3. Added INFO/WARN/ERROR tags as appropriate in all the places where the logging function is called.
4. JavaAccessBridge will generate the file suffixed with "_java_access_bridge", whereas WindowsAccessBridge generates file suffixed with "_windows_access_bridge".

Here is the updated webrev: http://cr.openjdk.java.net/~kaddepalli/8196681/webrev04/

Thanks,
Krishna

-----Original Message-----
From: Sergey Bylokhov
Sent: Tuesday, September 18, 2018 3:26 AM
To: Prasanta Sadhukhan <prasanta.sadhukhan at oracle.com>; Krishna Addepalli <krishna.addepalli at oracle.com>; awt-dev at openjdk.java.net
Subject: Re: <AWT Dev> RFR: [12] JDK-8196681: Java Access Bridge logging and debug flags dynamically controlled

I guess that it is a good thing to use nullptr, but I think it is better to follow the style used in the file, and replace NULL to nullptr in the whole file in some separate fix.

On 14/09/2018 05:18, Prasanta Sadhukhan wrote:
> one question: why nullptr is used? I see we used NULL in other places 
> in this file, why cant we use the same?
> 
> Regards
> Prasanta
> On 14-Sep-18 4:05 PM, Krishna Addepalli wrote:
>>
>> Thanks for the comments. Here is the new webrev: 
>> http://cr.openjdk.java.net/~kaddepalli/8196681/webrev02/
>> <http://cr.openjdk.java.net/%7Ekaddepalli/8196681/webrev02/>
>>
>> --Krishna
>>
>> *From:*Prasanta Sadhukhan
>> *Sent:* Friday, September 14, 2018 3:23 PM
>> *To:* Krishna Addepalli <krishna.addepalli at oracle.com>; 
>> awt-dev at openjdk.java.net
>> *Subject:* Re: <AWT Dev> RFR: [12] JDK-8196681: Java Access Bridge 
>> logging and debug flags dynamically controlled
>>
>> ok. The formatting is screwed up, needs to be rectified. Also l194, 
>> there should be a space before {
>>
>> Regards
>> Prasantaa
>>
>> On 14-Sep-18 2:45 PM, Krishna Addepalli wrote:
>>
>>     Thanks for the review Prasanta. Although there is not much
>>     difference between using fprintf and vfprintf, I have changed all
>>     the calls to vfprintf.
>>
>>     Here is the new webrev:
>>     http://cr.openjdk.java.net/~kaddepalli/8196681/webrev01/
>>     <http://cr.openjdk.java.net/%7Ekaddepalli/8196681/webrev01/>
>>
>>     Krishna
>>
>>     *From:*Prasanta Sadhukhan
>>     *Sent:* Friday, September 14, 2018 11:38 AM
>>     *To:* Krishna Addepalli <krishna.addepalli at oracle.com>
>>     <mailto:krishna.addepalli at oracle.com>; awt-dev at openjdk.java.net
>>     <mailto:awt-dev at openjdk.java.net>
>>     *Subject:* Re: <AWT Dev> RFR: [12] JDK-8196681: Java Access Bridge
>>     logging and debug flags dynamically controlled
>>
>>     One thing, since you are passing va_list as the last parameter,
>>     shouldn't all fprintf be actually vfprintf?
>>
>>     Regards
>>     Prasanta
>>
>>     On 14-Sep-18 12:55 AM, Krishna Addepalli wrote:
>>
>>         Hi All,
>>
>>         Enhancement: https://bugs.openjdk.java.net/browse/JDK-8196681
>>
>>         Webrev:
>>         http://cr.openjdk.java.net/~kaddepalli/8196681/webrev00/
>>         <http://cr.openjdk.java.net/%7Ekaddepalli/8196681/webrev00/>
>>
>>         Please review an enhancement for supporting the logging of
>>         Java Access Bridge, so that customers/users facing problems
>>         with Accessibility can turn on/off logging to debug problems
>>         with this feature.
>>
>>         The proposed solution is to have the user/developer define an
>>         environment variable "*JAVA_ACCESSBRIDGE_LOGFILE*" and filling
>>         with the path to the log file.
>>
>>         The JavaAccessBridge will read the variable, and write to the
>>         file in the path provided.
>>         Note that the implementation is simplistic, and doesnot have
>>         any way to handle multiple applications, since it is a debug
>>         only feature.
>>
>>         Thanks,
>>
>>         Krishna
>>
> 


--
Best regards, Sergey.