Initialized the variables root and ri.

Initialized the variables root and ri.

These were being flagged by gcc-4.6.0 using the Wmaybe-uninitialized warning.

BUG=none
TEST=trybot


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=124619

[asharif1, 2012-01-31 19:41:01]

PTAL. This will have to be patched in upstream Mongoose as well.

[kkania, 2012-01-31 20:41:44]

can you update the bottom of the README and the .diff file too?

[Joe, 2012-01-31 21:29:34]

I sent a patch to the team that maintains the Mongoose project.

On Tue, Jan 31, 2012 at 3:41 PM,  <kkania@chromium.org> wrote:
> can you update the bottom of the README and the .diff file too?
>
> http://codereview.chromium.org/9309008/

[Joe, 2012-01-31 22:03:16]

According to the mongoose team if next_option() returns non-NULL, it
guarantees to fill vector correctly.  It also seems that the current
checked in file is out of date, see:
     http://code.google.com/p/mongoose/source/list

On Tue, Jan 31, 2012 at 4:29 PM, Joe Mikhail <jmikhail@google.com> wrote:
> I sent a patch to the team that maintains the Mongoose project.
>
> On Tue, Jan 31, 2012 at 3:41 PM,  <kkania@chromium.org> wrote:
>> can you update the bottom of the README and the .diff file too?
>>
>> http://codereview.chromium.org/9309008/

[asharif1, 2012-02-01 05:22:29]

On 2012/01/31 22:03:16, Joe wrote:
> According to the mongoose team if next_option() returns non-NULL, it
> guarantees to fill vector correctly.  It also seems that the current
> checked in file is out of date, see:
>      http://code.google.com/p/mongoose/source/list
> 
> On Tue, Jan 31, 2012 at 4:29 PM, Joe Mikhail <mailto:jmikhail@google.com>
wrote:
> > I sent a patch to the team that maintains the Mongoose project.
> >
> > On Tue, Jan 31, 2012 at 3:41 PM, mailto: <kkania@chromium.org> wrote:
> >> can you update the bottom of the README and the .diff file too?
> >>
> >> http://codereview.chromium.org/9309008/

Perhaps we should update the whole mongoose to upstream?

Btw, I uploaded a new patch with updated README and the diff file.

[kkania, 2012-02-02 22:28:42]

LGTM; we will eventually update mongoose again, but that is a lower priority
item

[commit-bot: I haz the power, 2012-02-02 23:56:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/asharif@chromium.org/9309008/5003

[commit-bot: I haz the power, 2012-02-03 02:49:20]

Try job failure for 9309008-5003 (retry) (retry) on win_rel for step
"test_shell_tests".
It's a second try, previously, step "test_shell_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2012-02-03 03:55:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/asharif@chromium.org/9309008/5003

[commit-bot: I haz the power, 2012-02-03 05:32:17]

Try job failure for 9309008-5003 (retry) on win_rel for step
"content_unittests".
It's a second try, previously, step "content_unittests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2012-03-01 22:25:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/asharif@chromium.org/9309008/5003

[commit-bot: I haz the power, 2012-03-02 08:31:18]

Change committed as 124619