Closed Bug 742490 Opened 13 years ago Closed 6 years ago

WebGL's linkProgram should warn if compile/linking failed even if infolog is empty

Categories

(Core :: Graphics: CanvasWebGL, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED INCOMPLETE

People

(Reporter: ehsan.akhgari, Unassigned)

Details

(Whiteboard: webgl-conformance)

Attachments

(1 file)

(deleted), application/x-bzip2
Details
Attached file Testcase (deleted) —
See the testcase attached (sorry that it's not minimal). Look for _glGetShaderInfoLog, which ends up not being called.
Using webgl-conformance tracking bug until we switch to a keyword/whiteboard for this.
No longer blocks: webgl-conformance
Whiteboard: webgl-conformance
Ehsan, can you remind me what this bug is about? getShaderiv is not a WebGL function.
I'm starting to have doubts about where the bug is. I have to retry this.
Turns out that this is really a bug in the Emscripten implementation of glGetShaderiv, but what confused us was that compileShader is failing without letting anyone know. Perhaps we should log an error to the console when that happens?
Summary: getShaderiv(GL_INFO_LOG_LENGTH) returns 0 as the log length even when the log length is not zero → WebGL's compileShader function should log an error if the compilation failed
This bug is now almost a duplicate of 745292. Note that compileShader is not the right place to do that, as compilation can be completely delayed until linkProgram. So linkProgram it is. The only reason why this is not a dupe of 745292 is that my patch only generates a warning when infolog is non-empty. If some GL impl gave empty infologs on linkProgram failure, we wouldn't report anything, which would be misleading, hence, is worth fixing.
Summary: WebGL's compileShader function should log an error if the compilation failed → WebGL's linkProgram should warn if compile/linking failed even if infolog is empty
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: