Skip to content

Commit 8977bb2

Browse files
committed
Update comments to clarify potential issues with movieLength input values
1 parent ca8a1e2 commit 8977bb2

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

Sprint-1/3-mandatory-interpret/2-time-format.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ console.log(result);
3838

3939
// f) Try experimenting with different values of movieLength. Will this code work for all values of movieLength? Explain your answer
4040
//It will do the math perfectly for most normal positive numbers, but there are a few situations where the code will act wierdly.
41-
//1. Visual problems
41+
//1. The "Single Digit" Visual Bug (Zero-Padding)
4242
//2. Negative numbers
43-
//3.Decimal numbers
43+
//3.Decimal numbers (floats and non integers)
44+
//4. invalid data types (like strings, objects, etc.)

0 commit comments

Comments
 (0)