Hi Sergey, I understand about the UB, but currently just including <string> is causing this warning to appear, and there doesn't seem to be a way without doing irrelevant changes, so somewhat reluctantly I'm proposing plain C changes.
Regarding the questions: 1. Currently, we expect that the user provides us the file name with the path. The only correction we are doing is that, if there is no explicit mention of extension, we add ".log". We donot expect to read a path and create a file (if that is what you meant by handling the absent slash at the end of the path). 2. I made all the declarations auto. Here is the new webrev: http://cr.openjdk.java.net/~kaddepalli/8196681/webrev09 Thanks, Krishna -----Original Message----- From: Sergey Bylokhov Sent: Tuesday, December 11, 2018 2:37 AM To: Krishna Addepalli <krishna.addepa...@oracle.com>; Shashidhara Veerabhadraiah <shashidhara.veerabhadra...@oracle.com> Cc: Prasanta Sadhukhan <prasanta.sadhuk...@oracle.com>; awt-dev@openjdk.java.net Subject: Re: <AWT Dev> RFR: [12] JDK-8196681: Java Access Bridge logging and debug flags dynamically controlled Hi, Krishna. On 10/12/2018 06:38, Krishna Addepalli wrote: > The only way to avoid changes to make file is by using C string manipulation > API. > Here is the webrev: http://cr.openjdk.java.net/~kaddepalli/8196681/webrev08 , > although I would personally prefer using the C++ api. Throwing a c++ exception out of "extern C", when the method is called from the other language is UB, so we should not throw any exception and use c++, or use plain c. Comment about the current version: - Do we handle the absent of the slash at the end of the path in "JAVA_ACCESSBRIDGE_LOGFILE"? - in some place you use auto and in another same situation you use "size_t", please unify. -- Best regards, Sergey.
