bugGNUstep - Bugs: bug #37799, NSOpenGLView -initWIthCoder does...

Group
 
 

bug #37799: NSOpenGLView -initWIthCoder does not deserialize the NSPixelFormat

Submitter:  Clint Smullen <daemonae>
Submitted:  Fri 23 Nov 2012 07:18:29 PM UTC
   
 
Category:  Gui/AppKit Severity:  3 - Normal
Item Group:  Bug Status:  Fixed
Privacy:  Public Assigned to:  FredKiefer
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 26 Dec 2012 07:23:36 PM UTC, comment #1: 

I commited you patch with small changes. I was a bit reluctant to change the type of the first argument of getValues:... from long* to int*, but you are right on most systems GLint gets mapped to a 32bit integer. Still this change may cause problems on code that gets converted over from Macs and uses long instead of GLint.

Thank you very much for your patch

Fred Kiefer <FredKiefer>
Group Member
Fri 23 Nov 2012 07:18:29 PM UTC, original submission:  

A NSOpenGLView present in a NIB cannot be used immediately due to the missing pixel format (this technique works fine under OSX).

I have implemented a fix which has NSOpenGLPixelFormat properly declare its implementation of NSCoder and implement the necessary methods, marshaling the attribute data before it is handed to the backend via initWithAttributes.

I tested under Ubuntu 12.10 using GNUstep locally built with Clang 3.1 using an Xcode 4.5 generated XIB.

https://github.com/Daemonae/gnustep-gui/commit/c61b143b08d67c119a11bd4800ec14488f1b1c8a

Clint Smullen <daemonae>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by FredKiefer (Posted a comment)
  • -email is unavailable- added by daemonae (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2014-01-21 FredKiefer StatusReady For Test Fixed
        Open/ClosedIn Test Closed
    2012-12-26 FredKiefer StatusNone Ready For Test
        Assigned toNone FredKiefer
        Open/ClosedOpen In Test
    2012-11-23 daemonae Attached File- Added 0001-Add-and-use-NSCoder-in-NSOpenGLPixelFormat.patch, #26964

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code