Skip to content

Commit 8a9c7d8

Browse files
committed
Merge pull request #27 from jotrk/master
Fix compiler warnings
2 parents 6c84d8f + 650af7d commit 8a9c7d8

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

bower.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222
"dependencies": {
2323
"purescript-eff": "^0.1.0",
2424
"purescript-functions": "^0.1.0",
25-
"purescript-maybe": "^0.3.0"
25+
"purescript-maybe": "^0.3.0",
26+
"purescript-exceptions": "^0.3.1"
2627
}
2728
}

src/Graphics/Canvas.purs

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -109,6 +109,7 @@ import Prelude
109109
import Data.Function
110110
import Data.Maybe
111111
import Control.Monad.Eff
112+
import Control.Monad.Eff.Exception.Unsafe (unsafeThrow)
112113

113114
-- | The `Canvas` effect denotes computations which read/write from/to the canvas.
114115
foreign import data Canvas :: !
@@ -134,7 +135,7 @@ foreign import data CanvasGradient :: *
134135
foreign import canvasElementToImageSource :: CanvasElement -> CanvasImageSource
135136

136137
-- | Wrapper for asynchronously loading a image file by path and use it in callback, e.g. drawImage
137-
foreign import withImage :: forall eff a. String -> (CanvasImageSource -> Eff eff Unit) -> Eff eff Unit
138+
foreign import withImage :: forall eff. String -> (CanvasImageSource -> Eff eff Unit) -> Eff eff Unit
138139

139140
foreign import getCanvasElementByIdImpl ::
140141
forall r eff. Fn3 String
@@ -403,13 +404,15 @@ textAlign ctx = unsafeParseTextAlign <$> textAlignImpl ctx
403404
unsafeParseTextAlign "center" = AlignCenter
404405
unsafeParseTextAlign "start" = AlignStart
405406
unsafeParseTextAlign "end" = AlignEnd
407+
unsafeParseTextAlign align = unsafeThrow $ "invalid TextAlign: " ++ align
408+
-- ^ dummy to silence compiler warnings
406409

407410
foreign import setTextAlignImpl :: forall eff. Context2D -> String -> (Eff (canvas :: Canvas | eff) Context2D)
408411

409412
-- | Set the current text alignment.
410413
setTextAlign :: forall eff. Context2D -> TextAlign -> Eff (canvas :: Canvas | eff) Context2D
411-
setTextAlign ctx textAlign =
412-
setTextAlignImpl ctx (show textAlign)
414+
setTextAlign ctx textalign =
415+
setTextAlignImpl ctx (show textalign)
413416

414417
-- | Text metrics:
415418
-- |

0 commit comments

Comments
 (0)