199791 - parseInt used instead of Math.floor in ChatZilla and Venkman utilities

Status: RESOLVED FIXED

(Other Applications Graveyard :: ChatZilla, defect)

Description

[Steve Chapel]

In the chrome files extensions/venkman/resources/content/venkman-utils.js and
extensions/irc/js/lib/utils.js the following lines appear:

    var seconds = roundTo(offset % 60, 2);
    var minutes = parseInt(offset / 60);
    var hours = parseInt(minutes / 60);
    minutes = minutes % 60;
    var days = parseInt(hours / 24);
    hours = hours % 24;

The parseInt function calls should be replaced by calls to Math.floor instead.
This will make the code faster and easier to read.

Comment 1

[timeless]

.

Comment 2

[mmc]

Attachment: patch

Change occurrences of parseInt() to Math.floor().

Comment 3

[Robert Ginda]

I've got these changes in my local tree, they'll show up in the next chatzila
and venkman XPIs.  There were a few other places in venkman-utils.js that I
fixed as well.

Comment 4

[Robert Ginda]

fixed