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

------------------------------------------------------------------------------
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