search

Electrux / ls_extended

ls with coloring and icons
BSD 3-Clause "New" or "Revised" License
201 stars 14 forks source link

Mostly cosmetics #12

Closed pczajkowski closed 6 years ago

pczajkowski commented 6 years ago

src/core.c - this is tricky as previous formatting was giving me a warning on 2/3 of my systems. include/ls.h - this change may be a bit far fetched, but block seems redundant. build.sh - probably better to use gnu99 with functions like strcasecmp.

Electrux commented 6 years ago

hmm, the uint64_t should use PRIu64 macro. Source: https://stackoverflow.com/questions/9225567/how-to-print-a-int64-t-type-in-c

I have changed the code accordingly. Could you please check if it works for you? :)

And thanks for all those changes :)

pczajkowski commented 6 years ago

Yes, it works fine now. Cheers!

Electrux commented 6 years ago

awesome! thanks and cheers! :D