Last Comment Bug 708305 - Fix 2 PR_* to false/true in spdysession
: Fix 2 PR_* to false/true in spdysession
Product: Core
Classification: Components
Component: Networking: HTTP (show other bugs)
: 11 Branch
: x86_64 Linux
-- normal (vote)
: mozilla11
Assigned To: Patrick McManus [:mcmanus]
: Patrick McManus [:mcmanus]
Depends on:
  Show dependency treegraph
Reported: 2011-12-07 10:33 PST by Patrick McManus [:mcmanus]
Modified: 2012-02-01 13:57 PST (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch 0 (1.11 KB, patch)
2011-12-07 10:37 PST, Patrick McManus [:mcmanus]
honzab.moz: review+
Details | Diff | Splinter Review

Description User image Patrick McManus [:mcmanus] 2011-12-07 10:33:18 PST
2 uses of PR_TRUE and PR_FALSE snuck into spdysession.cpp. change them to true/false.
Comment 1 User image Patrick McManus [:mcmanus] 2011-12-07 10:37:15 PST
Created attachment 579741 [details] [diff] [review]
patch 0
Comment 2 User image Patrick McManus [:mcmanus] 2011-12-07 11:27:17 PST
Comment 3 User image Ed Morley [:emorley] 2011-12-08 08:54:09 PST
Comment 4 User image Ed Morley [:emorley] 2011-12-10 15:34:06 PST
Backed out (along with the rest of the SPDY landing) in order to stop us hitting the MSVC virtual address limit, so we can reopen the trees (bug 709193).

Sucks, but we don't really have any other choice here :-(
Comment 5 User image Ed Morley [:emorley] 2011-12-14 11:14:16 PST
Relanded on mozilla-central :-)

Note You need to log in before you can comment on or make changes to this bug.