Alon Bar-Lev has posted comments on this change.

Change subject: core: log: add indirect filtering
......................................................................


Patch Set 1:

In python private/public is contract.

The interface of the object is the public method, signature is without _ prefix.

Unlike in other languages this is a contract and not enforced.

The list in this case is not part of the interface exposed to the word.

There is nothing wrong in related objects to access each other private 
members/functions, this is friend in C or package protected in Java.

There is no violation of OOP in this code.

-- 
To view, visit http://gerrit.ovirt.org/19782
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Id96a70f5450f9416583b7803dfe7a10aa6c98472
Gerrit-PatchSet: 1
Gerrit-Project: otopi
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Yedidyah Bar David <[email protected]>
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to