On 2/15/21 2:14 PM, Martin Simmons wrote:
On Fri, 12 Feb 2021 17:47:52 -0500, Josh Fisher said:
On 2/12/21 12:53 PM, Martin Simmons wrote:

I don't understand the comment about "refactor" or "all instances" unless
there are other errors as well.

By refactor, he means that when the function declaration is changed from
round() to bround() to avoid the conflict, it has to be changed
everywhere else in the code where it is called. In bsnprintf.c, round()
is declared static, so this amounts to changing all calls to round()
that are made inside the bsnprintf.c source file from round() to bround().
OK, so refactor is just a way to make simple things sound complicated...I
would call that renaming the function and you obviously also have to change
the single use of it as well :-)

Well, it's DevOps terminology, so....yes.


__Martin


_______________________________________________
Bacula-users mailing list
Bacula-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bacula-users

Reply via email to