improvement to bresenham algoritm

LICENSE

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2016 @alvaropinot
+Copyright (c) 2016 @nquicenob
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

src/bresenham.js

@@ -19,15 +19,16 @@ function getInitValues(startPoint, finalPoint){
   }
 }
 
-function inLoop(sign, currentP, finalP){
-  return sign < 0 ? currentP >= finalP : currentP <= finalP;
+function getBreakFn(sign){
+  return sign < 0 ?
+    (current, final) => current >= final :
+    (current, final) => current <= final;
 }
 
 function calcMainCoordinates(absDiff){
   return absDiff.x > absDiff.y ? ['x', 'y'] : ['y', 'x'];
 }
 
-
 export function* bresenhamLine(point, finalPoint) {
   const { absDiff, sign } = getInitValues(point, finalPoint);
   const [ mainCoordinate, coordinate ] = calcMainCoordinates(absDiff);
@@ -40,12 +41,14 @@ export function* bresenhamLine(point, finalPoint) {
   const mainDiff = absDiff[mainCoordinate];
   const secondDiff = absDiff[coordinate];
 
+  const breakFn = getBreakFn(mainSign);
+
   let mainValue = point[mainCoordinate];
   let secondValue = point[coordinate];
 
   let eps = 0;
 
-  for( ; inLoop(mainSign, mainValue, final); mainValue += mainSign) {
+  for( ; breakFn(mainValue, final); mainValue += mainSign) {
     yield {
       [mainCoordinate]: mainValue,
       [coordinate]: secondValue

test.bresenham.js

@@ -11,6 +11,7 @@ function buildArray(fn, startPoint, finalPoint){
   return arr;
 }
 
+
 test('Bresenham\'s line algoritm ', (assert) => {
 
   [