lcm-proj / lcm

Lightweight Communications and Marshalling
GNU Lesser General Public License v2.1
944 stars 385 forks source link

Replace tmpnam with mkstemp #388

Closed ihilt closed 1 year ago

ihilt commented 1 year ago

This addresses the linker warning:

CMakeFiles/test-c-client.dir/common.c.o:common.c:function make_tmpnam: warning: the use of `tmpnam' is dangerous, better use `mkstemp'

I'm assuming that g_mkstemp is a suitable replacement on the Windows platform but I'm not currently setup to test. Since this only impacts the eventlog tests, I thought this was a low risk change.