search

debrouxl / tilibs

TILP (formerly GtkTiLink) can transfer data between Texas Instruments graphing calculators and a computer. It works with all link cables (parallel, serial, Black/Gray/Silver/Direct Link) and it supports the TI-Z80 series (73..86), the TI-eZ80 series (83PCE, 84+CE), the TI-68k series (89, 92, 92+, V200, 89T) and the Nspire series (Nspire Clickpad / Touchpad / CX, both CAS and non-CAS)
http://lpg.ticalc.org/prj_tilp
67 stars 22 forks source link

libtifiles: Fix tifiles_create_table_of_entries #62

Closed Vogtinator closed 2 years ago

Vogtinator commented 2 years ago

The port from strcpy to strncpy errorneously used sizeof on a heap allocated array, which doesn't have the expected result. This broke e.g. saving variables to files, which were empty beyond the header.

+ a typo fix + fix ODR warnings

debrouxl commented 2 years ago

LGTM. Looks like I should proofread my occurrences of strncpy() again... and why not do the same in libtifiles as I did for libticalcs in aed916e7a57b5a9064652316a139ff2e277a32f2 .

debrouxl commented 2 years ago

Thanks :)