OASIS Mailing List ArchivesView the OASIS mailing list archive below
or browse/search using MarkMail.


Help: OASIS Mailing Lists Help | MarkMail Help

xacml-comment message

[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]

Subject: XACML 3.0 wd 8 (was: Re: [xacml-comment] XACML 3.0 wd 7)

Erik Rissanen schrieb:
> BTW, you are very welcome to give the new algorithms a thorough review.
> We really appreciate more eye balls which can spot mistakes. :-)

In the pseudo code of deny-overrides, there is a duplicate variable
called "decision". I would call the parameter "decisions", since it is
an array. And (as a really minor issue), I would put the array brackets
at the right of the type name, not the variable: Decision[] decisions.

I would also remove all those unnecessary curly braces. They only make
the code look more bloated than it really is.

To add some good words in between: I really like the decision to unify
the combining algorithms using the Indeterminate{...} notation. It feels
to me like the most elegant way to do it.


[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]