Skip to content

Commit fbf4500

Browse files
committed
Merge pull request #17 from davidchambers/ternary
use ternary operator in JavaScript output
2 parents 1152224 + 75da016 commit fbf4500

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

src/Data/String.purs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,7 @@ module Data.String
2828
foreign import _charAt
2929
"""
3030
function _charAt(i, s, Just, Nothing) {
31-
if (i < 0 || i >= s.length) return Nothing;
32-
else return Just(s.charAt(i));
31+
return i >= 0 && i < s.length ? Just(s.charAt(i)) : Nothing;
3332
}
3433
""" :: forall a. Fn4 Number String (a -> Maybe a) (Maybe a) (Maybe Char)
3534

@@ -42,8 +41,7 @@ module Data.String
4241
foreign import _charCodeAt
4342
"""
4443
function _charCodeAt(i, s, Just, Nothing) {
45-
if (i < 0 || i >= s.length) return Nothing;
46-
else return Just(s.charCodeAt(i));
44+
return i >= 0 && i < s.length ? Just(s.charCodeAt(i)) : Nothing;
4745
}
4846
""" :: forall a. Fn4 Number String (a -> Maybe a) (Maybe a) (Maybe Number)
4947

0 commit comments

Comments
 (0)