-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 07/31/2011 02:51 PM, [email protected] wrote:
> Please apply!
Send patches using git-send-email. People cannot reply with review
comments to patches sent as attachments.
patch 1:
+ if (!gbm)
+ {
+ free(dri2_dpy);
+ return EGL_FALSE;
+ }
should be
+ if (!gbm) {
+ free(dri2_dpy);
+ return EGL_FALSE;
+ }
I think 'gbm == NULL' is the more common idiom in Mesa, but either way
works for most people.
It may be even better to fold this hunk with the following if-statement:
gbm = (struct gbm_device *) disp->PlatformDisplay;
- - if (strcmp(gbm_device_get_backend_name(gbm), "drm") != 0) {
+ if (!gbm || strcmp(gbm_device_get_backend_name(gbm), "drm") != 0) {
free(dri2_dpy);
return EGL_FALSE;
patch 2:
I don't think always linking with -lm is quite correct either. The
comment at the top says that backends should define GBM_LIBS, but then
nothing under src/gdb uses GBM_LIBS.
Also, I think everyone hates the commit message style of starting a
sentence in the "brief" line and continuing it in the body. A better
message is:
gbm: link gbm_gallium_drm.so against math library
This avoids the following runtime error with EGL on platforms that
require linking with libm for nontrivial math functions:
failed to load module: /xorg/lib64/gbm/gbm_gallium_drm.so: undefined
symbol: powf
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
iEYEARECAAYFAk425PsACgkQX1gOwKyEAw9JzgCeMG8fw2Qtz8PXhcXc1xzc0ubP
WOUAn1l0OjzsTA/c4lKpZcihgjVOqK4n
=Db6m
-----END PGP SIGNATURE-----
------------------------------------------------------------------------------
BlackBerry® DevCon Americas, Oct. 18-20, San Francisco, CA
The must-attend event for mobile developers. Connect with experts.
Get tools for creating Super Apps. See the latest technologies.
Sessions, hands-on labs, demos & much more. Register early & save!
http://p.sf.net/sfu/rim-blackberry-1
_______________________________________________
Mesa3d-dev mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mesa3d-dev
