Simplify code

The readCell function was introduced in e659668 only for this purpose. We
don't need it because:
- the extra check for c.chr == 0 is not needed; if there is already a null cell
  at the line end, we needn't write it again. There is no difference between a
  null cell and a printable cell in this regard
- the only check that's left is for the coordinates; we know that y is valid,
  and x is not less than 0, so the only check we need is for x against the
  length of the line.

Author: stefanhaller

view.go

@@ -726,14 +726,6 @@ func (v *View) writeCells(x, y int, cells []cell) {
 	v.lines[y] = line[:newLen]
 }
 
-// readCell gets cell at specified location (x, y)
-func (v *View) readCell(x, y int) (cell, bool) {
-	if y < 0 || y >= len(v.lines) || x < 0 || x >= len(v.lines[y]) {
-		return cell{}, false
-	}
-	return v.lines[y][x], true
-}
-
 // Write appends a byte slice into the view's internal buffer. Because
 // View implements the io.Writer interface, it can be passed as parameter
 // of functions like fmt.Fprintf, fmt.Fprintln, io.Copy, etc. Clear must
@@ -764,7 +756,7 @@ func (v *View) writeRunes(p []rune) {
 
 	finishLine := func() {
 		v.autoRenderHyperlinksInCurrentLine()
-		if c, ok := v.readCell(v.wx, v.wy); !ok || c.chr == 0 {
+		if v.wx >= len(v.lines[v.wy]) {
 			v.writeCells(v.wx, v.wy, []cell{{
 				chr:     0,
 				fgColor: 0,