mbasti-rh commented on a pull request

"""
I really don't like to move definitions how to split params from classes to 
single function
```
def split_rrparam(name, value):
```

It doesn't look safe for me or easy to understand and maintain. When I want to 
add new DNS type, I have to check 3 different files, I'm sure I will overlook 
something.


"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/34#issuecomment-243112068
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to