`gcvt` not present on some OS's

Electrux / ls_extended

ls with coloring and icons

BSD 3-Clause "New" or "Revised" License

201 stars 14 forks source link

`gcvt` not present on some OS's #4

Closed kaosatha closed 6 years ago

kaosatha commented 6 years ago

ls_extended calls gcvt at src/core.c:120. On some OS's (FreeBSD for instance) gcvt is unavailable. Is there any reason gcvt is used over snprintf? I have a fork with the change made, if this is something that would be considered for merging?

Electrux commented 6 years ago

oh I didn't know FreeBSD doesn't have gcvt! I would love to merge that change if you have made! Go ahead and Pull Request it. Thanks a lot 😄 👍

Electrux commented 6 years ago

fixed