New webrev with "_simple_basename":
http://cr.openjdk.java.net/~erikj/8221851/webrev.05/
/Erik
On 2019-04-08 12:20, Erik Joelsson wrote:
On 2019-04-08 11:40, Kim Barrett wrote:
On Apr 8, 2019, at 10:28 AM, Erik Joelsson
<erik.joels...@oracle.com> wrote:
Hello,
On 2019-04-05 15:46, Kim Barrett wrote:
Assuming all that, consider instead putting this_file_helper in
exceptions.hpp (perhaps with a better name?), don't bother with
THIS_FILE, and define THREAD_AND_LOCATION as
#define THREAD_AND_LOCATION THREAD, this_file_helper(__FILE__),
__LINE__
Moved to exceptions.hpp, renamed to "basename", and removed the
THIS_FILE macro.
“basename” might not count as a “better name”, as it conflicts with a
POSIX function,
even though we don’t presently seem to be using that anywhere that I
could find.
How about "simple_basename" then? Or just prefix with an underscore?
The idea is to keep it internal to the headerfile, but I'm not
familiar with conventions in Hotspot to know how you usually
prefix/namespace things as private.
/Erik
