bugGNUstep - Bugs: bug #38393, NSBezierPath does not have a...

Group
 
 

bug #38393: NSBezierPath does not have a complete implementation of keyed archive coding

Submitter:  Chris Armstrong <carmstrong>
Submitted:  Sun 24 Feb 2013 06:28:42 AM 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
   

Tue 21 Jan 2014 08:07:00 AM UTC, comment #3: 

No comment in almost a year.

Fred Kiefer <FredKiefer>
Group Member
Mon 04 Mar 2013 04:52:44 PM UTC, comment #2: 

I commited your patch plus the required change to -elementAtIndex:associatedPoints: plus the corresponding changes to the callers.

Please give it a try.

Fred Kiefer <FredKiefer>
Group Member
Sun 24 Feb 2013 12:24:25 PM UTC, comment #1: 

Thank you for this great patch. I would really love to add that to the upcoming gui release, even though we are in a code freeze.

There is one interesting aspect to your patch. For "close" elements you are using two points returned by elementAtIndex:associatedPoints: Now that method in GNUstep doesn't return any points for "close" elements and according to the Apple documentation it should return one.

This means that we have an important issue here in the current GNUstep code and that your code has an issue as well. Your issue could be easily fixed by using the same point twice. But where should that point come from? I think the best way to fix it now is to moce some code from currentPoint into the elementAtIndex:associatedPoints: method and later one rethink the whole code once more.

Fred Kiefer <FredKiefer>
Group Member
Sun 24 Feb 2013 06:28:42 AM UTC, original submission:  

gnustep-gui's implementation of NSBezierPath initWithCoder: and encodeWithCoder: is incomplete for keyed archive coding (i.e. with NSKeyedArchiver/NSKeyedUnarchiver.

I've attached a patch that completes the implementation.

Chris Armstrong <carmstrong>

 

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

Attach Files:
   
   
Comment:
   

Attached Files

 

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 carmstrong (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
    2013-03-04 FredKiefer StatusNone Ready For Test
        Assigned toNone FredKiefer
        Open/ClosedOpen In Test
    2013-02-24 carmstrong Attached File- Added NSBezierPath-keyedcodingimplementation.diff, #27516

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code