Problem in Consts.java

[alitariqnet]

// ================== Time constants ==================
/** One minute time in seconds. /
public static final int MINUTE = 60;
/* One hour time in seconds. /
public static final int HOUR = 60 * MINUTE;
/* One day time in seconds. /
public static final int DAY = 24 * HOUR;
/* One week time in seconds. */
public static final int WEEK = 24 * HOUR;

Above lines are written in Consts.java in Cloudsim 5.0 pre-release
Do you think that the last line should be,
/** One week time in seconds. */
public static final int WEEK = 7 * DAY;
?

[lak-sh-mi]

Looking at the logic if it is to represent seconds as stated in comments week = 7 * day This bug may not be obvious of the constant week is not used in the implementations of the tool. Did we know it the usage if week?

…

On Mon, Feb 22, 2021, 12:02 AM alitariqnet ***@***.***> wrote: // ================== Time constants ================== /** One minute time in seconds. */ public static final int MINUTE = 60; /** One hour time in seconds. */ public static final int HOUR = 60 * MINUTE; /** One day time in seconds. */ public static final int DAY = 24 * HOUR; /** One week time in seconds. */ public static final int WEEK = 24 * HOUR; Above lines are written in Consts.java in Cloudsim 5.0 pre-release Do you think that the last line should be, /** One week time in seconds. */ public static final int WEEK = 7 * DAY; ? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#147>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ANMB32ZMFEODTGRYOHAJ77LTAFGUDANCNFSM4X7HU4CA> .