> It seems to be sufficient to add "intel" as an additional vendor to the list 
> in the X11GLFactory class. Tests pass and
> my own application also works with the new build.

Michael Paus has updated the pull request incrementally with one additional 
commit since the last revision:

  Remove duplicate GLGPUInfo according to review comment
  
  My original intention was to keep two entries in order to make it clearer 
that there are currently two distinct vendor
  strings for Intel floating around. But having just one entry which covers 
both cases is ok for me too.

-------------

Changes:
  - all: https://git.openjdk.java.net/jfx/pull/243/files
  - new: https://git.openjdk.java.net/jfx/pull/243/files/759af32c..4142adb3

Webrevs:
 - full: https://webrevs.openjdk.java.net/jfx/243/webrev.01
 - incr: https://webrevs.openjdk.java.net/jfx/243/webrev.00-01

  Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod
  Patch: https://git.openjdk.java.net/jfx/pull/243.diff
  Fetch: git fetch https://git.openjdk.java.net/jfx pull/243/head:pull/243

PR: https://git.openjdk.java.net/jfx/pull/243

Reply via email to