>>>>> "Matthew" == Matthew Wringe <[EMAIL PROTECTED]> writes:

Matthew> If this patch is approved, I can commit it.

Matthew> -    // If there is no property "service.algorithm"
Matthew> -    if (provider.getProperty(service + "." + algorithm) == null)

There's something here I don't understand.

Looking at java.security.Provider, I see that it tries to
canonicalize the keys -- see toCanonicalKey().

Would just changing this to use get() fix the problem?
Or perhaps it would be better to override getProperty in Provider?

Matthew> +    if (!algorithmFound) {

The "{" goes on its own line here.

Tom

Reply via email to