openfortivpn icon indicating copy to clipboard operation
openfortivpn copied to clipboard

Published 20 hours ago • verified publisher icon adrienverge

refactoring of ipv4_add_nameservers_to_resolv_conf

Open mrbaseman opened this issue 4 years ago • 1 comments

ipv4_add_nameservers_to_resolv_conf would benefit from some refactoring, see https://github.com/adrienverge/openfortivpn/pull/547#issuecomment-591893613

mrbaseman avatar Feb 27 '20 11:02 mrbaseman

Code Inspector suggests refactoring other long/complex functions. Might be of interest in some cases. On the other hand most issues raised by Code Inspector do not seem relevant to me.

DimitriPapadopoulos avatar Feb 28 '20 09:02 DimitriPapadopoulos