From 46b8c8c9131d8c8d2582d7f40453f83940052c11 Mon Sep 17 00:00:00 2001
From: nick black <dankamongmen@gmail.com>
Date: Fri, 3 Jul 2020 21:09:38 -0400
Subject: [PATCH] ncdirect_render_image(): get it working for all geometries
 #759

---
 src/lib/blit.c     |  1 +
 src/lib/direct.cpp | 28 +++++++++++++++-------------
 2 files changed, 16 insertions(+), 13 deletions(-)

diff --git a/src/lib/blit.c b/src/lib/blit.c
index e9c511d33..ef54b1f15 100644
--- a/src/lib/blit.c
+++ b/src/lib/blit.c
@@ -280,6 +280,7 @@ quadrant_blit(ncplane* nc, int placey, int placex, int linesize,
   int dimy, dimx, x, y;
   int total = 0; // number of cells written
   ncplane_dim_yx(nc, &dimy, &dimx);
+//fprintf(stderr, "quadblitter %dx%d -> %d/%d+%d/%d\n", leny, lenx, dimy, dimx, placey, placex);
   // FIXME not going to necessarily be safe on all architectures hrmmm
   const unsigned char* dat = data;
   int visy = begy;
diff --git a/src/lib/direct.cpp b/src/lib/direct.cpp
index af808be3f..65dbec450 100644
--- a/src/lib/direct.cpp
+++ b/src/lib/direct.cpp
@@ -250,13 +250,11 @@ ncdirect_dump_plane(ncdirect* n, const ncplane* np, int xoff){
   const int toty = ncdirect_dim_y(n);
   int dimy, dimx;
   ncplane_dim_yx(np, &dimy, &dimx);
-fprintf(stderr, "rasterizing %dx%d\n", dimy, dimx);
+//fprintf(stderr, "rasterizing %dx%d+%d\n", dimy, dimx, xoff);
+  assert(dimx + xoff <= totx);
   for(int y = 0 ; y < dimy ; ++y){
-    if(ncdirect_cursor_move_yx(n, -1, xoff)){
-      return -1;
-    }
-    for(int x = 0 ; x < xoff ; ++x){
-      if(putchar(' ') == EOF){
+    if(xoff){
+      if(ncdirect_cursor_move_yx(n, -1, xoff)){
         return -1;
       }
     }
@@ -270,21 +268,20 @@ fprintf(stderr, "rasterizing %dx%d\n", dimy, dimx);
       ncdirect_fg(n, channels_fg(channels));
       ncdirect_bg(n, channels_bg(channels));
 //fprintf(stderr, "%03d/%03d [%s] (%03dx%03d)\n", y, x, egc, dimy, dimx);
-      if(printf("%s", strlen(egc) == 0 ? " " : egc) < 0){
+      if(fprintf(n->ttyfp, "%s", strlen(egc) == 0 ? " " : egc) < 0){
         return -1;
       }
     }
-    // FIXME mystifyingly, we require this cursor_left() when using 2x2, but must
-    // not have it when using 2x1 (we insert blank lines otherwise). don't paper
-    // over it with a conditional, but instead get to the bottom of this FIXME.
     if(dimx < totx){
       ncdirect_bg_default(n);
-      if(putchar('\n') == EOF){
+      if(putc('\n', n->ttyfp) == EOF){
         return -1;
       }
     }
     if(y == toty){
-      ncdirect_cursor_down(n, 1);
+      if(ncdirect_cursor_down(n, 1)){
+        return -1;
+      }
     }
   }
   return 0;
@@ -325,7 +322,7 @@ nc_err_e ncdirect_render_image(ncdirect* n, const char* file, ncalign_e align,
 //fprintf(stderr, "render: %d+%d of %d/%d stride %u %p\n", leny, lenx, ncv->rows, ncv->cols, ncv->rowstride, ncv->data);
   struct ncplane* faken = ncplane_create(NULL, NULL,
                                          disprows / encoding_y_scale(bset),
-                                         dispcols,// / encoding_x_scale(bset),
+                                         dispcols / encoding_x_scale(bset),
                                          0, 0, NULL);
   if(faken == NULL){
     return NCERR_NOMEM;
@@ -341,7 +338,12 @@ nc_err_e ncdirect_render_image(ncdirect* n, const char* file, ncalign_e align,
   if(ncdirect_dump_plane(n, faken, xoff)){
     return NCERR_SYSTEM;
   }
+  while(fflush(stdout) == EOF && errno == EAGAIN){
+    ;
+  }
   free_plane(faken);
+  ncdirect_fg_default(n);
+  ncdirect_bg_default(n);
   return NCERR_SUCCESS;
 }