The fix removes unused DDC sections that caused misinterpretation of DDC data due to a missing break statement in a switch. Already committed to head. Submitted for nomination for 6.8.2.
Created attachment 1390 [details] [review] [FIXED_X11R68x] Fix For 6.8.2 nomination only.
Most of this patch does cosmetics. The real fix is here: - break; - case DS_ASCII_STR: - case DS_SERIAL: + break; case DS_RANGES: case DS_ASCII_STR and DS_SERIAL where processed like DS_RANGES due to a missing break. To prevent this in the future and to make the code more readable the cosmetics were done.
Comment on attachment 1390 [details] [review] [FIXED_X11R68x] Fix Approval for X11R6.8.x stable branch granted in the 2004-12-08 release-wranglers phone call. Please don't commit, I'll do that myself...
Comment on attachment 1390 [details] [review] [FIXED_X11R68x] Fix Patch checked-in into X11R6.8.x stable branch: /cvs/xorg/xc/ChangeLog,v <-- ChangeLog new revision: 1.365.2.75; previous revision: 1.365.2.74 cvs commit: Using deprecated info format strings. Convert your scripts to use the new argument format and remove '1's from your info file format strings. /cvs/xorg/xc/programs/Xserver/hw/xfree86/common/xf86Configure.c,v <-- xf86Configure.c new revision: 1.6.2.1; previous revision: 1.6 cvs commit: Using deprecated info format strings. Convert your scripts to use the new argument format and remove '1's from your info file format strings. Mailing the commit message to xorg-commit@lists.freedesktop.org...
Patch checked-in into Xorg trunk and X11R6.8.x stable branch... ... marking bug as FIXED.
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.