Typo in latest commit

[mm765]

BSON.js deserialize: toStrring() (double r),
other than that, it works for my very small test, thanks.

[mm765]

Push -
This is the one i mentioned in the decaf-thread, but maybe this is fixed in bower, too, so i didn't see it ?!

[mschwartz]

Can you tell me what line number in the file is wrong, what it says and what it should say?

Thanks

[mschwartz]

Bower simply does something like a git clone when you do "bower install."
When you do "bower update" it will do the equivalent of a git pull. If
you are not using bower, then I can't help more without knowing how exactly
you installed things.

On Mon, Jan 18, 2016 at 10:55 AM, mm765 [email protected] wrote:

Push -
This is the one i mentioned in the decaf-thread, but maybe this is fixed
in bower, too, so i didn't see it ?!

—
Reply to this email directly or view it on GitHub
#1 (comment)
.

[mm765]

Sorry for the sloppy description.
In BSON.js line 61 reads
return String(o.toStrring());
it should be
return String(o.toString());

[mm765]

As to how i installed it - i simply cloned the repositories from github.
Now when i do a git pull it says i am up to date (for this and also for the decafjs repository) and also github does show the last date of change to be about 10 days ago - so it seems the github repository has not been updated.

[mschwartz]

I don't see the mistake :)

On Mon, Jan 18, 2016 at 2:34 PM, mm765 [email protected] wrote:

Sorry for the sloppy description.
In BSON.js line 61 reads
return String(o.toStrring());
it should be
return String(o.toString());

—
Reply to this email directly or view it on GitHub
#1 (comment)
.

[mschwartz]

NVM, I found it and pushed the typo fix.

On Mon, Jan 18, 2016 at 3:03 PM, Mike Schwartz [email protected] wrote:

I don't see the mistake :)

On Mon, Jan 18, 2016 at 2:34 PM, mm765 [email protected] wrote:

Sorry for the sloppy description.
In BSON.js line 61 reads
return String(o.toStrring());
it should be
return String(o.toString());

—
Reply to this email directly or view it on GitHub
#1 (comment)
.

[mm765]

:) there are two "r"s in the toString
Double rr
Hope this helps :)

[mm765]

Ok, and this one also shows up on github and was sent to me on a git pull.

[mschwartz]

I thought so :)

On Mon, Jan 18, 2016 at 3:11 PM, mm765 [email protected] wrote:

Ok, and this one also shows up on github and was sent to me on a git pull.

—
Reply to this email directly or view it on GitHub
#1 (comment)
.