Error locations should be 1-based instead of 0-based (#18)

index.ts

@@ -57,7 +57,7 @@ function handleErrors(diagnostics: typescript.Diagnostic[], compiler: typeof typ
         var messageText = compiler.flattenDiagnosticMessageText(diagnostic.messageText, os.EOL);
         if (diagnostic.file) {
             var lineChar = diagnostic.file.getLineAndCharacterOfPosition(diagnostic.start);
-            outputFn(`  ${diagnostic.file.fileName.blue} (${lineChar.line.toString().cyan},${lineChar.character.toString().cyan}): ${messageText.red}`)
+            outputFn(`  ${diagnostic.file.fileName.blue} (${(lineChar.line+1).toString().cyan},${(lineChar.character+1).toString().cyan}): ${messageText.red}`)
         }
         else {
             outputFn(`  ${"unknown file".blue}: ${messageText.red}`)

test/errorlocation/app.ts

@@ -0,0 +1 @@
+var a == 0; 

test/errorlocation/webpack.config.js

@@ -0,0 +1,16 @@
+module.exports = {
+    context: __dirname,
+    entry: './app.ts',
+    output: {
+        path: __dirname,
+        filename: 'bundle.js'
+    },
+    resolve: {
+        extensions: ['', '.js', '.ts']
+    },
+    module: {
+        loaders: [
+            { test: /\.ts$/, loader: '../../index.js?instance=errorlocation' }
+        ]
+    }
+}

test/run.js

@@ -4,6 +4,7 @@ var path = require('path');
 var newLine = require('os').EOL;
 var tsLoader = require('../index');
 var webpack = require('webpack');
+var colors = require('colors');
 
 function handleErrors(err, stats, done) {
     if (err) { 
@@ -89,6 +90,21 @@ describe('externals', function() {
     })
 })
 
+describe('errorlocation', function() {
+    it('should report correct error location', function(done) {
+        webpack(require('./errorlocation/webpack.config')).run(function(err, stats) {
+            if (err) return done(err)
+
+            var errors = stats.toJson().errors;
+
+            assert.equal(errors.length, 2, 'Exactly two errors should be reported');
+            assert.ok(colors.strip(errors[0]).indexOf(" (1,7): ") != -1, 'The error reported was in the wrong location');
+
+            done();
+        })
+    })
+})
+
 describe('instance', function() {
     it('should error when incorrectly configured', function(done) {
         webpack(require('./instance/bad.config')).run(function(err, stats) {
@@ -260,4 +276,4 @@ describe('tsconfigSearch', function() {
             }
         })
     })
-})
+})