Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

Fix #2864: allow empty x-www-form-urlencoded bodies #2868

Merged
merged 3 commits into from Oct 8, 2015
Merged

Fix #2864: allow empty x-www-form-urlencoded bodies #2868

merged 3 commits into from Oct 8, 2015

Conversation

jcayzac
Copy link
Contributor

@jcayzac jcayzac commented Jul 30, 2015

No description provided.

@kcharwood
Copy link
Contributor

Would you be able to add tests demonstrating the fix?

@jcayzac
Copy link
Contributor Author

jcayzac commented Aug 4, 2015

@kcharwood done

@jcayzac
Copy link
Contributor Author

jcayzac commented Aug 26, 2015

@kcharwood: sorry if it sounds pushy, which is not my intent, but could you let me know what is blocking the merging of this PR (it's blocking an internal ticket we have here at my workplace, which is getting carried over to new sprints as they come in)? If it's just a lack of time, then no big deal :)

Cheers

@kcharwood
Copy link
Contributor

Hey @jcayzac

In Xcode 7, this is generating a warning:

/Users/kevin.harwood/GIT/AFNetworking/AFNetworking/AFURLRequestSerialization.m:533:45: Use of GNU ?: conditional expression extension, omitting middle operand

@jcayzac
Copy link
Contributor Author

jcayzac commented Aug 27, 2015

Hi @kcharwood!

Interesting! ?: seems to be used in other places, but forcedly-silenced using #pragma clang diagnostic ignored "-Wgnu" 😄

Oh well, I removed it and replaced it with a good old if. It even makes things slightly clearer.

@kcharwood kcharwood modified the milestones: 2.6.1, 3.0.0 Oct 8, 2015
kcharwood added a commit that referenced this pull request Oct 8, 2015
Fix #2864: allow empty x-www-form-urlencoded bodies
@kcharwood kcharwood merged commit 824f7c5 into AFNetworking:master Oct 8, 2015
@kcharwood
Copy link
Contributor

Sorry for the delay on getting this one.

Thanks for the PR!

🍻

@kcharwood
Copy link
Contributor

Updated in 3.0.0 with 9521690

@jcayzac jcayzac deleted the fix-2864 branch February 19, 2017 03:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
2 participants