bugGNUstep - Bugs: bug #35359, NSMutableURLRequest

Group
 
 

bug #35359: NSMutableURLRequest

Submitter:  Norbert Schneider <nschneider>
Submitted:  Tue 24 Jan 2012 08:50:16 AM UTC
   
 
Category:  Base/Foundation Severity:  3 - Normal
Item Group:  Bug Status:  Fixed
Privacy:  Public Assigned to:  None
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 26 Jan 2012 08:01:36 PM UTC, comment #2: 

Thanks ... that's fixed in svn trunk.

Richard Frith-Macdonald <CaS>
Group Member
Tue 24 Jan 2012 09:55:17 AM UTC, comment #1: 

The call to gnutls_transport_set_lowat() has already been removed in the current SVN code for more recent versions of gnutls. This will be released with the upcoming new GNUstep version.

The problem with NSMutableURLRequest is a bug in the code. In the designated initialiser we test for the value of URL being of class NSURL while in init we set it to nil. We need an extra test here to allow for this case.

I would like to leave this change to Richard as I am not that comfortable with this class. A short look at the mutableCopyWithZone: method left me rather surprised.

Fred Kiefer <FredKiefer>
Group Member
Tue 24 Jan 2012 08:50:16 AM UTC, original submission:  

n some code I had to deal with, there was this line:
NSMutableURLRequest *myRequest = [[NSMutableURLRequest alloc]init];

However, myRequest always was NULL. I changed as follows:
NSMutableURLRequest *myRequest = [NSMutableURLRequest alloc];

which seems to circumvent the problem.

I am using gnustep-startup-0.26.2

Is this a problem in Gnustep or in the Code I got?

I am using OpenSuse 12.1, gcc Version is 4.6.2

Note that gnustep-base did not build for me out of the box, because of this line in GSSocketStream.m:
linux-3n93:> diff GSSocketStream.m*
532c532
<   gnutls_transport_set_lowat (session, 0);
---

>   gnutls_transport_set_lowat (session, 0);


gnutls_transport_set_lowat is deprecated, the call can be removed altogether I think.

Sorry, I first posted this in support.

Norbert Schneider <nschneider>

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by CaS (Posted a comment)
  • -email is unavailable- added by FredKiefer (Posted a comment)
  • -email is unavailable- added by nschneider (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 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2012-01-26 CaS StatusConfirmed Fixed
        Open/ClosedOpen Closed
    2012-01-24 FredKiefer CategoryNone Base/Foundation
        Item GroupNone Bug
        StatusNone Confirmed

    Back to the top

    Powered by Savane 3.13-bb6a.
    Corresponding source code