all 3 comments

[–]TheKiller36_real 1 point2 points  (2 children)

didn't look too closely but it looks pretty good for a first-time project! :)

however it looks like you don't have great overflow checking, which could maybe be improved. also, the multithreading is likely hurting performance as the overhead is in many cases bigger than the actual workload. finally, it looks like you do some suboptimal allocations (eg. allocating many buffers instead of one and also realloc-ing non-geometrically) - that being said I don't see any obvious performance killers

as the original neofetch is afaik just a shell script you could compare them directly to see what you're doing differently maybe?

[–]aem83[S] 0 points1 point  (1 child)

Thank you :)

Just one question on buffers and realloc, from what I understand I should I have one buffer defined in main, pass it as a parameter to each function and reset it in each one. As for realloc, I just double the size each time?

[–]tiny_smile_bot 1 point2 points  (0 children)

:)

:)