Bug 1637

Summary: switch default handling is missing, generates warning.
Product: xprint Reporter: Egbert Eich <eich>
Component: Server: OtherAssignee: Roland Mainz <roland.mainz>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: high    
Version: unspecified   
Hardware: x86 (IA32)   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
Fix
none
[FIXED_X11R68x] Slightly different patch which issues a warning when an unknown |XpOidNotify| value is used. roland.mainz: 6.8-branch+

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.