[RFC][PATCH][icedtea-web]: Added support for signed JNLP file- Updated Patch

Saad Mohammad smohammad at redhat.com
Mon Jul 25 11:02:53 PDT 2011


On 07/21/2011 09:51 AM, Jiri Vanek wrote:
> On 07/21/2011 03:31 PM, Dr Andrew John Hughes wrote:
>> On 09:21 Thu 21 Jul     , Jiri Vanek wrote:
> ..snip...
>>>>
>>>> The use of new String and casting seems wrong.  I'm not sure what 
>>>> e.nextElement is returning
>>>> but that enumeration should be using generics.
>>>
>>> This method was refactored to return pure list<String>  and is now 
>>> working correctly.
>>>
>>
>> So where's the new patch?
>>
>>>>
> ..snip...
>
> You are right. This method is relict. It was fixed in patch by correct 
> version  but this relict remains.
> This method you are mentioning is in commented section. I do not know 
> why there is such a big commented section (some kind of "class backup" 
> ) (If Saad did this it must be fixed)
> I will try to find an guilty one;)
>
> J. 
I will save u trouble! It wasn't me! :D.
I didn't add the commented section. It was there from the very beginning 
(rev 0). Should I remove the commented section on my next patch? I don't 
see any relevance for it to be there.

-- 
Cheers,
Saad Mohammad




More information about the distro-pkg-dev mailing list