Convert all calls to __ASSERT_() to assert_()

[khwilliamson]

The former symbol is undefined behavior in C and C++.

• This set of changes does not require a perldelta entry.

[bulk88]

https://github.com/search?q=%22define+__assert_%28%22&type=code 133 hits with current/old macro

https://github.com/search?q=%22define+assert_%28%22&type=code 696 hits with future proposed macro

the new preferred macro has much higher chance of a collision with 3rd party C headers

update:

Although a C compiler vendor should be calling the actual proc termination func

extern void _assert(const char * msg);

I have a suspicion this is a popular, not that brilliant choice, either in the past, or current, for C compiler authors, or random C library code bases.

extern void assert_(const char * msg);

[bulk88]

@mauke so as a professional developer, did you analyze which spelling of the token has the least likely hood of conflicts with 3rd party headers or not?

[tonycoz]

the new preferred macro has much higher chance of a collision with 3rd party C headers

The __ASSERT_() name is reserved to the implementation, which is probably why few others use it.

Although a C compiler vendor should be calling the actual proc termination func

extern void _assert(const char * msg);

_assert() is windows specific and irrelevant.

[mauke]

@mauke so as a professional developer, did you analyze which spelling of the token has the least likely hood of conflicts with 3rd party headers or not?

Yes, it's q_ac6f10cd_382f_4077_88e9_8e3e2c291ff3.

[khwilliamson]

As a factoid, assert_() was added in 5.18. If its expansion conflicts with another definition, there would be a compiler warning.