Bug 1637 - switch default handling is missing, generates warning.
Summary: switch default handling is missing, generates warning.
Status: RESOLVED FIXED
Alias: None
Product: xprint
Classification: Unclassified
Component: Server: Other (show other bugs)
Version: unspecified
Hardware: x86 (IA32) Linux (All)
: high normal
Assignee: Roland Mainz
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-10-14 11:01 UTC by Egbert Eich
Modified: 2004-12-12 02:31 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
Fix (757 bytes, patch)
2004-10-14 11:04 UTC, Egbert Eich
no flags Details | Splinter Review
[FIXED_X11R68x] Slightly different patch which issues a warning when an unknown |XpOidNotify| value is used. (2.93 KB, patch)
2004-10-15 14:41 UTC, Roland Mainz
roland.mainz: 6.8-branch+
Details | Splinter Review

Description Egbert Eich 2004-10-14 11:01:50 UTC
Xprint/Oid.c:XpOidNotifyString() has no default case. 
Fix will be attached.
Comment 1 Egbert Eich 2004-10-14 11:04:42 UTC
Created attachment 1109 [details] [review]
Fix
Comment 2 Roland Mainz 2004-10-15 14:41:45 UTC
Created attachment 1114 [details] [review]
[FIXED_X11R68x] Slightly different patch which issues a warning when an unknown |XpOidNotify| value is used.
Comment 3 Roland Mainz 2004-10-15 14:44:10 UTC
Patch checked-in...

/cvs/xorg/xc/ChangeLog,v  <--  ChangeLog
new revision: 1.464; previous revision: 1.463
/cvs/xorg/xc/programs/Xserver/Xprint/Oid.c,v  <--  Oid.c
new revision: 1.3; previous revision: 1.2
Mailing the commit message to xorg-commit@pdx.freedesktop.org...

... marking bug as FIXED.
Comment 4 Roland Mainz 2004-11-09 21:33:34 UTC
Comment on attachment 1114 [details] [review]
[FIXED_X11R68x] Slightly different patch which issues a warning when an unknown |XpOidNotify| value is used.

Requesting approval for X11R6.8.x...
Comment 5 Roland Mainz 2004-11-19 07:37:06 UTC
Comment on attachment 1114 [details] [review]
[FIXED_X11R68x] Slightly different patch which issues a warning when an unknown |XpOidNotify| value is used.

Approved for the X11R6.8.x branch in the 2004-11-17 release-wranglers phone
call.
Please don't commit it yourself, I'll handle that once the CVS service is
available again.
Comment 6 Roland Mainz 2004-12-12 21:31:36 UTC
Comment on attachment 1114 [details] [review]
[FIXED_X11R68x] Slightly different patch which issues a warning when an unknown |XpOidNotify| value is used.

Patch checked-in into X11R6.8.x stable branch:

/cvs/xorg/xc/ChangeLog,v  <--  ChangeLog
new revision: 1.365.2.45; previous revision: 1.365.2.44
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/Xprint/Oid.c,v  <--  Oid.c
new revision: 1.2.4.1; previous revision: 1.2
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...


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.