miconda left a comment (kamailio/kamailio#4221)

Thanks for the PR!

It would be good to keep coherence for local variable naming style, e.g., using 
lowercase -- now there are `Result` and `len`, or `FullBuffer` and `totallen`, 
or `sizeToAllocate`.

Also, declare the variables at the beginning of the block (e.g., `size_t 
name_len = strlen(dir->d_name);` is in the middle).

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4221#issuecomment-2829744891
You are receiving this because you are subscribed to this thread.

Message ID: <kamailio/kamailio/pull/4221/c2829744...@github.com>
_______________________________________________
Kamailio - Development Mailing List -- sr-dev@lists.kamailio.org
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org
Important: keep the mailing list in the recipients, do not reply only to the 
sender!

Reply via email to