@rfuchs commented on this pull request.
>
typedef struct rtpengine_api
{
rtpengine_start_recording_f start_recording;
rtpengine_answer_f answer;
rtpengine_offer_f offer;
- rtpengine_delete_f delete;
+ rtpengine_delete_f _delete;
> Thanks for the review. I added the underscore to delete to avoid potential
> conflicts with the delete keyword in C++. While this code is written in C,
> there's a possibility that it might be included in a C++ project in the
> future. Using _delete ensures compatibility and prevents any issues if that
> happens. If you think it's unnecessary, I'm happy to revert it, but I thought
> it was a safer choice to avoid potential problems.
Ok, fair point, but I do think having the member name with an underscore looks
a bit odd. Perhaps use some alternative like `del` or `delet` then, or even
just `teardown` or something else.
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3956#discussion_r1734589089
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3956/review/2266116...@github.com>_______________________________________________
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org
