Hi Pekka, I'll reformat the long lines.

I agree, there should only be one platform visible when the test is
executing.
It's easy enough to remove the iteration over platforms.


--Thanks!
Bryan


On Wed, Apr 3, 2013 at 3:24 AM, Pekka Jääskeläinen <
[email protected]> wrote:

> On 04/02/2013 12:55 PM, Bryan Bell wrote:
>
>> Comments?
>>
>
> Some overlong lines (let's try to cut at 80 chars),
> but otherwise LGTM.
>
> In the test it iterates over the platforms which
> seems unnecessary as the tests should be able to
> assume only one platform is visible when executed via
> an ICD dispatcher that supports the OCL_ICD_VENDORS
> override?
>
> But this seems to be the case also with the
> previous runtime lib tests.
>
> --
> --Pekka
>



-- 
-- Regards
Bryan
Phone:530-355-5145
http://www.google.com/profiles/Bryan.W.Bell
------------------------------------------------------------------------------
Minimize network downtime and maximize team effectiveness.
Reduce network management and security costs.Learn how to hire 
the most talented Cisco Certified professionals. Visit the 
Employer Resources Portal
http://www.cisco.com/web/learning/employer_resources/index.html
_______________________________________________
pocl-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/pocl-devel

Reply via email to