canis 0.0.11 → 0.0.13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ccb62a5de8425b056e27630bfbfdfd14a8d6dd70fbf7ed088edd99d28bbb00c2
-  data.tar.gz: 70e719a10733c1deb4d8a4a1b0101268cdc5a76d43b871ef7f9a1f89013226ef
+  metadata.gz: a5ff168e0396a8194dde1fef712068369b675a575648806182c9a5f5540198f7
+  data.tar.gz: 4b0b4a2e219ea2bfefcaff61a293bd47a3d5e423a4237c1d76be0f1c86857767
 SHA512:
-  metadata.gz: 295f2d4be95aaf57c7ba316b46a348ed3324e49cfa9ba8f7f48a15fbae0282fc8baa95850329e6d4525a13aa8c4a54950af6344f1783393e6578bf682e6ef6aa
-  data.tar.gz: 93c55b658895a685deb167b8f330b772ccb34ae6e76c5a7af510147d17d2ac2b23ca2476d1096800b09cb27288a5ff9c61d16cc902f4801bf557241eeee16d70
+  metadata.gz: f295681678c651300eeb93153df8ba6b6a751f5daacb86baa88f2e07b005159b93d933fb2c75e80ca0c7c620b92696dccd0a2237beaefef8e78b165c185b6943
+  data.tar.gz: f5191705d2292807e095c4f90e1740ed8b307a2e0c0a6f75f9049f44c8601009df3b0d9609f55d9145e33974ee26f8eccbeca86e1bc92cd4958ec4dace1f9bc9

data/README.md

@@ -1,7 +1,18 @@
 # Canis
 
+**2019-03-11**. Kindly note that `canis` is not, and will never be, production-ready.
+Very few ruby programmers are using ncurses so the usage of canis is
+very low if any. So kindly do not use for critical work.
 
-## Last update on 2018-03-05 for ruby 2.5 
+That is why I made a much simpler and stripped version of `canis` called `umbra`.
+I use `umbra` mostly, but that too for a screen with a few fields and a single table,
+and some popups.
+
+Umbra will be easier for you to go through, and hack (if you come across a problem) than canis. Having said that, I am perhaps the only user of `umbra`, so again it is at your own risk.
+
+If anyone wishes to take over and maintain `canis` please contact me.
+
+## Last update on 2018-03-05 for ruby 2.5
 
 Everything seems to be working fine without any changes.
 
@@ -23,7 +34,7 @@ document reader, `rigel` which is based on the code of `ribhu` (which was rbcurs
 ## Usage
 
 Until, we have better documentation, the best way to understand "canis" is to run the examples in the examples
-folder. This gives a good idea of what the library does. Now you can open the source and get a feel of the 
+folder. This gives a good idea of what the library does. Now you can open the source and get a feel of the
 structure of the code and what to study. One can also select an example that is close to the application one has in mind
 and use the source as a starting point.
 
@@ -51,8 +62,8 @@ TODO: Write usage instructions here
 
 Commonly used widgets and features:
 
-- Textpad - to display non-editable multiline text. Text can be colored. 
-- Listbox - identical to the textpad except that it allows single or multiple selection, and has 
+- Textpad - to display non-editable multiline text. Text can be colored.
+- Listbox - identical to the textpad except that it allows single or multiple selection, and has
     some extra events such as entering and leaving or row, and selection.
 - Field - user entry of a single line of data.
 - Label - readonly text
@@ -80,7 +91,7 @@ Lesser used widgets and features:
 Some Issues with rbcurse:
 
 Widgets required explicit coordinates. To that effect the App class allowed for `Stack` and `Flow` (idea borrowed from
-the **Shoes** project. This works well, but all stack and flow information is lost when controls are placed meaning that a 
+the **Shoes** project. This works well, but all stack and flow information is lost when controls are placed meaning that a
 change in the window size (if user resizes) does not (and cannot) resize the application.
 
 Canis has recently introduced Layout objects which have the ability to re-locate and resize objects when screen size
@@ -91,7 +102,7 @@ Currently there are three layout objects:
 - `StackLayout` : does only stacking of objects (vertical placement)
 - `FlowLayout` : does only horizontal placement of obects
 - `SplitLayout` : allows for multiple stacks and flows by means of splitting a split either horizontally or vertically
-  and either placing an object in it, or splitting it further. 
+  and either placing an object in it, or splitting it further.
 These are based on an `AbstractLayout` which can be used to derive further layouts.
 
 It is my intention to move usage over to these layouts since they are simpler, and allow for resizing (and to abandon
@@ -103,7 +114,7 @@ Issues canis would like to address:
   I would like to do this before reaching 1.0.
 
 - Keymapping. Currently, takes codes as integers, but i would have liked moving to strings as in vim.
-  Currently we have to map `?\C-a.getbytes(0)` or `[?g, ?g]`, whereas a string would allow us to map `"<C-x>s"` 
+  Currently we have to map `?\C-a.getbytes(0)` or `[?g, ?g]`, whereas a string would allow us to map `"<C-x>s"`
   or `"<F1>"` or `"gg"`. The issue is that there is too much rework within the library since each widget uses integer mappings.
   Mapping and matching multiple keys would be a lot easier if stored internally as a string, currently multiple
   mappings require a hash or tree at each level.
@@ -115,6 +126,9 @@ This tutorial needs to be updated for canis. Although, canis has diverged/forked
 There is some on-line documentation of classes at:
 http://rubydoc.info/gems/canis/0.0.8/frames
 
+Umbra/ncumbra is at:
+https://github.com/mare-imbrium/umbra
+
 ## Contributing
 
 - Please give suggestions on how to improve the documentation.

data/examples/testdb.rb

@@ -105,7 +105,7 @@ begin
     #file = "movies1000.txt"
 
     arr = get_data
-    tv = Canis::Table.new @form, :row => 1, :col => 0, :height => h, :width => w, :name => "tv", :suppress_borders => false do |b|
+    tv = Canis::Table.new @form, :row => 2, :col => 0, :height => h, :width => w, :name => "tv", :suppress_borders => false do |b|
 
       b.resultset $columns, arr
 

data/lib/canis/core/widgets/rwidget.rb

@@ -9,7 +9,7 @@
   * Author: jkepler (ABCD)
   * Date: 2008-11-19 12:49 
   * License: Same as Ruby's License (http://www.ruby-lang.org/LICENSE.txt)
-  * Last update: 2017-03-09 23:10
+  * Last update: 2018-05-16 12:40
 
   == CHANGES
   * 2011-10-2 Added PropertyVetoException to rollback changes to property
@@ -97,15 +97,14 @@ class Module  # dsl_accessor {{{
             oldvalue = @#{sym}
             tmp = val.size == 1 ? val[0] : val
             newvalue = tmp
-            if oldvalue.nil? || @_object_created.nil?
+            if @_object_created.nil?
                @#{sym} = tmp
             end
-            return(self) if oldvalue.nil? || @_object_created.nil?
+            return(self) if @_object_created.nil?
 
             if oldvalue != newvalue
               # trying to reduce calls to fire, when object is being created
                begin
-                 @property_changed = true
                  fire_property_change("#{sym}", oldvalue, newvalue) if !oldvalue.nil?
                  @#{sym} = tmp
                  @config["#{sym}"]=@#{sym}
@@ -909,9 +908,9 @@ module Canis
       # create a logger giving a path.
       def create_logger path
         #path = File.join(ENV["LOGDIR"] || "./" ,"canis14.log")
-        file   = File.open(path, File::WRONLY|File::TRUNC|File::CREAT) 
-        logg = Logger.new(path)
-        raise "Could not create logger  " unless logg
+        _path   = File.open(path, File::WRONLY|File::TRUNC|File::CREAT) 
+        logg = Logger.new(_path)
+        raise "Could not create logger: #{path}" unless logg
         # if not set, will default to 0 which is debug. Other values are 1 - info, 2 - warn
         logg.level = ENV["CANIS_LOG_LEVEL"].to_i
         colors = Ncurses.COLORS
@@ -1023,12 +1022,11 @@ module Canis
           return :NO_BLOCK
         end # if
       end
-      ## added on 2009-01-08 00:33 
       # goes with dsl_property
-      # Need to inform listeners - done 2010-02-25 23:09 
       # Can throw a FieldValidationException or PropertyVetoException
       def fire_property_change text, oldvalue, newvalue
-        return if oldvalue.nil? || @_object_created.nil? # added 2010-09-16 so if called by methods it is still effective
+        return if @_object_created.nil? #  2018-05-15 - removed check for nil, as object can be nil on creation
+                                        # but set later.
         #$log.debug " FPC #{self}: #{text} #{oldvalue}, #{newvalue}"
         $log.debug " FPC #{self}: #{text} "
         if @pce.nil?
@@ -1037,7 +1035,7 @@ module Canis
           @pce.set( self, text, oldvalue, newvalue)
         end
         fire_handler :PROPERTY_CHANGE, @pce
-        @repaint_required = true # this was a hack and shoudl go, someone wanted to set this so it would repaint (viewport line 99 fire_prop
+        @repaint_required = true 
         repaint_all(true) # for repainting borders, headers etc 2011-09-28 V1.3.1 
       end
 

data/lib/canis/core/widgets/table.rb

@@ -7,11 +7,11 @@
 #       Author: jkepler http://github.com/mare-imbrium/canis/
 #         Date: 2013-03-29 - 20:07
 #      License: Same as Ruby's License (http://www.ruby-lang.org/LICENSE.txt)
-#  Last update: 2017-03-09 23:14
+#  Last update: 2019-03-12 15:17
 # ----------------------------------------------------------------------------- #
 #   table.rb  Copyright (C) 2012-2014 kepler
 
-# == CHANGES: 
+# == CHANGES:
 #  - changed @content to @list since all multirow wids and utils expect @list
 #  - changed name from tablewidget to table
 #
@@ -29,7 +29,7 @@ require 'logger'
 require 'canis'
 require 'canis/core/widgets/textpad'
 
-## 
+##
 # The motivation to create yet another table widget is because tabular_widget
 # is based on textview etc which have a lot of complex processing and rendering
 # whereas textpad is quite simple. It is easy to just add one's own renderer
@@ -83,8 +83,8 @@ module Canis
     # It's a pretty simple sorter and uses sort, not sort_by.
     # Improvements welcome.
     # Usage: provide model in constructor or using model method
-    # Call toggle_sort_order(column_index) 
-    # Call sort. 
+    # Call toggle_sort_order(column_index)
+    # Call sort.
     # Currently, this sorts the provided model in-place. Future versions
     # may maintain a copy, or use a table that provides a mapping of model to result.
     # # TODO check if column_sortable
@@ -123,11 +123,11 @@ module Canis
         # with textpad, this way we avoid touching it
         header = @model.delete_at 0
         begin
-          # next line often can give error "array within array" - i think on date fields that 
+          # next line often can give error "array within array" - i think on date fields that
           #  contain nils
-        @model.sort!{|x,y| 
+        @model.sort!{|x,y|
           res = 0
-          @sort_keys.each { |ee| 
+          @sort_keys.each { |ee|
             e = ee.abs-1 # since we had offsetted by 1 earlier
             abse = e.abs
             if ee < 0
@@ -173,7 +173,7 @@ module Canis
         # internally, reverse sort is maintained by multiplying number by -1
         @sort_keys ||= []
         if @sort_keys.first && index == @sort_keys.first.abs
-          @sort_keys[0] *= -1 
+          @sort_keys[0] *= -1
         else
           @sort_keys.delete index # in case its already there
           @sort_keys.delete(index*-1) # in case its already there
@@ -241,7 +241,7 @@ module Canis
       # and returns an array which is used for printing
       #
       # return an array so caller can color columns if need be
-      def convert_value_to_text r  
+      def convert_value_to_text r
         str = []
         fmt = nil
         field = nil
@@ -285,12 +285,12 @@ module Canis
       def render pad, lineno, str
         #lineno += 1 # header_adjustment
         # header_adjustment means columns have been set
-        return render_header pad, lineno, 0, str if lineno == 0 && @source.header_adjustment > 0
+        return render_header pad, lineno, 0, str if lineno == 0  && @source.header_adjustment > 0
         #text = str.join " | "
         #text = @fmstr % str
         text = convert_value_to_text str
         if @_check_coloring
-          #$log.debug "XXX:  INSIDE COLORIIN"
+          $log.debug "XXX:  INSIDE COLORIIN"
           text = colorize pad, lineno, text
           return
         end
@@ -314,7 +314,7 @@ module Canis
             att = REVERSE
             # FIXME currentl this overflows into next row
           end
-        
+
         FFI::NCurses.wattron(pad,FFI::NCurses.COLOR_PAIR(cp) | att)
         FFI::NCurses.mvwaddstr(pad, lineno, 0, text)
         FFI::NCurses.wattroff(pad,FFI::NCurses.COLOR_PAIR(cp) | att)
@@ -323,6 +323,7 @@ module Canis
       def render_header pad, lineno, col, columns
         # I could do it once only but if user sets colors midway we can check once whenvever
         # repainting
+        # $log.debug "INSIDE render_header XXXXX #{@source.name}, #{lineno}, #{col}"
         check_colors #if @_check_coloring.nil?
         #text = columns.join " | "
         #text = @fmstr % columns
@@ -349,7 +350,7 @@ module Canis
         }
       end
       def each_column
-        @chash.each_with_index { |c, i| 
+        @chash.each_with_index { |c, i|
           next if c.hidden
           yield c,i if block_given?
         }
@@ -395,10 +396,10 @@ module Canis
     #   focus, and should be used for row operations.
     #
     #   In order to use Textpad easily, the first row of the table model is the column names. Data is maintained
-    #   in an Array. Several operations are delegated to Array, or have the same name. You can get the list 
+    #   in an Array. Several operations are delegated to Array, or have the same name. You can get the list
     #   using `list()` to run other Array operations on it.
     #
-    #   If you modify the Array directly, you may have to use `fire_row_changed(index)` to reflect the update to 
+    #   If you modify the Array directly, you may have to use `fire_row_changed(index)` to reflect the update to
     #   a single row. If you delete or add a row, you will have to use `fire_dimension_changed()`. However,
     #   internal functions do this automatically.
     #
@@ -418,12 +419,12 @@ module Canis
       #  When printing we should loop through chash and get the index in data
       #
       # should be zero here, but then we won't get textpad correct
-      @_header_adjustment = 0 #1
+      @_header_adjustment = 0
       @col_min_width = 3
 
       self.extend DefaultListSelection
       super
-      create_default_renderer unless @renderer # 2014-04-10 - 11:01 
+      create_default_renderer unless @renderer # 2014-04-10 - 11:01
       # NOTE listselection takes + and - for ask_select
       bind_key(?w, "next column") { self.next_column }
       bind_key(?b, "prev column") { self.prev_column }
@@ -441,9 +442,9 @@ module Canis
       @list_selection_model = nil
       @list_selection_model = Canis::DefaultListSelectionModel.new self
     end
-    
+
     # retrieve the column info structure for the given offset. The offset
-    # pertains to the visible offset not actual offset in data model. 
+    # pertains to the visible offset not actual offset in data model.
     # These two differ when we move a column.
     # @return ColumnInfo object containing width align color bgcolor attrib hidden
     def get_column index
@@ -454,7 +455,7 @@ module Canis
       @chash[index] = c
       return c
     end
-    ## 
+    ##
     # returns collection of ColumnInfo objects
     def column_model
       @chash
@@ -463,8 +464,8 @@ module Canis
     # calculate pad width based on widths of columns
     def content_cols
       total = 0
-      #@chash.each_pair { |i, c| 
-      #@chash.each_with_index { |c, i| 
+      #@chash.each_pair { |i, c|
+      #@chash.each_with_index { |c, i|
         #next if c.hidden
       each_column {|c,i|
         w = c.width
@@ -474,7 +475,7 @@ module Canis
       return total
     end
 
-    # 
+    #
     # This calculates and stores the offset at which each column starts.
     # Used when going to next column or doing a find for a string in the table.
     # TODO store this inside the hash so it's not calculated again in renderer
@@ -483,8 +484,8 @@ module Canis
       @coffsets = []
       total = 0
 
-      #@chash.each_pair { |i, c| 
-      #@chash.each_with_index { |c, i| 
+      #@chash.each_pair { |i, c|
+      #@chash.each_with_index { |c, i|
         #next if c.hidden
       each_column {|c,i|
         w = c.width
@@ -501,10 +502,10 @@ module Canis
     def _convert_curpos_to_column  #:nodoc:
       _calculate_column_offsets unless @coffsets
       x = 0
-      @coffsets.each_with_index { |i, ix| 
-        if @curpos < i 
+      @coffsets.each_with_index { |i, ix|
+        if @curpos < i
           break
-        else 
+        else
           x += 1
         end
       }
@@ -526,7 +527,7 @@ module Canis
       # TODO take care of multipliers
       _calculate_column_offsets unless @coffsets
       c = @column_pointer.next
-      cp = @coffsets[c] 
+      cp = @coffsets[c]
       #$log.debug " next_column #{c} , #{cp} "
       @curpos = cp if cp
       down() if c < @column_pointer.last_index
@@ -538,13 +539,13 @@ module Canis
       # TODO take care of multipliers
       _calculate_column_offsets unless @coffsets
       c = @column_pointer.previous
-      cp = @coffsets[c] 
+      cp = @coffsets[c]
       #$log.debug " prev #{c} , #{cp} "
       @curpos = cp if cp
       up() if c > @column_pointer.last_index
       fire_column_event :ENTER_COLUMN
     end
-    # a column traversal has happened. 
+    # a column traversal has happened.
     # FIXME needs to be looked into. is this consistent naming wise and are we using the correct object
     # In old system it was TABLE_TRAVERSAL_EVENT
     def fire_column_event eve
@@ -579,7 +580,7 @@ module Canis
     end
     def contract_column
       x = _convert_curpos_to_column
-      w = get_column(x).width 
+      w = get_column(x).width
       return if w <= @col_min_width
       column_width x, w-1 if w
       @coffsets = nil
@@ -601,7 +602,7 @@ module Canis
 
   ##
   # getter and setter for columns
-  # 2014-04-10 - 13:49 
+  # 2014-04-10 - 13:49
   # @param [Array] columns to set as Array of Strings
   # @return if no args, returns array of column names as Strings
     #  NOTE
@@ -620,10 +621,11 @@ module Canis
       @list << array
       _init_model array
 
-      # update the names in column model 
-      array.each_with_index { |n,i| 
+      # update the names in column model
+      array.each_with_index { |n,i|
         c = get_column(i)
-        c.name = name
+        #c.name = name     ## 2018-05-19 - seems to be a bug
+        c.name = n
       }
       self
     end
@@ -647,7 +649,7 @@ module Canis
     def _init_model array
       # clear the column data -- this line should be called otherwise previous tables stuff will remain.
       @chash.clear
-      array.each_with_index { |e,i| 
+      array.each_with_index { |e,i|
         # if columns added later we could be overwriting the width
         c = get_column(i)
         c.width ||= 10
@@ -658,7 +660,7 @@ module Canis
     # size each column based on widths of this row of data.
     def model_row index
       array = @list[index]
-      array.each_with_index { |c,i| 
+      array.each_with_index { |c,i|
         # if columns added later we could be overwriting the width
         ch = get_column(i)
         ch.width = c.to_s.length + 2
@@ -670,7 +672,7 @@ module Canis
     # estimate columns widths based on data in first 10 or so rows
     # This will override any previous widths, so put custom widths
     # after calling this.
-    def estimate_column_widths  
+    def estimate_column_widths
       each_column {|c,i|
         c.width  = suggest_column_width(i)
       }
@@ -683,7 +685,7 @@ module Canis
       #ret = @cw[col] || 2
       ret = get_column(col).width || 2
       ctr = 0
-      @list.each_with_index { |r, i| 
+      @list.each_with_index { |r, i|
         #next if i < @toprow # this is also a possibility, it checks visible rows
         break if ctr > 10
         ctr += 1
@@ -736,7 +738,7 @@ module Canis
     #
     #     table = Table.new ...
     #     table.filename 'contacts.tsv', :separator => '|', :columns => true
-    #   
+    #
     def filename name, _config = {}
       arr = File.open(name,"r").read.split("\n")
       lines = []
@@ -782,7 +784,7 @@ module Canis
         l = @list
       end
 
-      File.open(outfile, 'w') {|f| 
+      File.open(outfile, 'w') {|f|
         l.each {|r|
           line = r.join "\t"
           f.puts line
@@ -805,7 +807,7 @@ module Canis
     end
     alias :<< :add
 
-    # delete a data row at index 
+    # delete a data row at index
     #
     # NOTE : This does not adjust for header_adjustment. So zero will refer to the header if there is one.
     #   This is to keep consistent with textpad which does not know of header_adjustment and uses the actual
@@ -813,7 +815,7 @@ module Canis
     #
     def delete_at ix
       return unless @list
-      raise ArgumentError, "Argument must be within 0 and #{@list.length}" if ix < 0 or ix >=  @list.length 
+      raise ArgumentError, "Argument must be within 0 and #{@list.length}" if ix < 0 or ix >=  @list.length
       fire_dimension_changed
       #@list.delete_at(ix + @_header_adjustment)
       @list.delete_at(ix)
@@ -843,7 +845,7 @@ module Canis
       fire_row_changed actrow
       self
     end
-    
+
     #------- column related methods ------#
     #
     # convenience method to set width of a column
@@ -871,11 +873,11 @@ module Canis
     def _invalidate_width_cache    #:nodoc:
       @coffsets = nil
     end
-    ## 
+    ##
     # should all this move into table column model or somepn
     # move a column from offset ix to offset newix
     def move_column ix, newix
-      acol = @chash.delete_at ix 
+      acol = @chash.delete_at ix
       @chash.insert newix, acol
       _invalidate_width_cache
       #tmce = TableColumnModelEvent.new(ix, newix, self, :MOVE)
@@ -894,7 +896,7 @@ module Canis
     def calculate_column_width col, maxrows=99
       ret = 3
       ctr = 0
-      @list.each_with_index { |r, i| 
+      @list.each_with_index { |r, i|
         #next if i < @toprow # this is also a possibility, it checks visible rows
         break if ctr > maxrows
         ctr += 1
@@ -914,10 +916,22 @@ module Canis
       sr = @startrow + top
       sc = @startcol + left
       # first do header always in first row
-      retval = FFI::NCurses.prefresh(@pad,0,@pcol, sr , sc , 2 , @cols+ sc );
+      # -- prefresh arguments are:
+      # 1. pad
+      # 2. pminrow
+      # 3. pmincol (pad upper left)
+      # 4, sminrow (screen upper left row)
+      # 5, smincol
+      # 6, smaxrow
+      # 7, smaxcol
+
+      # 2019-03-12 - fixed bug in table, only printed header if table on row 1
+      retval = FFI::NCurses.prefresh(@pad, @prow, @pcol, sr, sc, sr + 1, @cols + sc)
+      $log.warn "XXX:  PADREFRESH HEADER #{retval}, #{@prow}, #{@pcol}, #{sr}, #{sc}, #{1+sr}, #{@cols+sc}." if retval == -1
+      # retval = FFI::NCurses.prefresh(@pad,0,@pcol, sr , sc , 2 , @cols+ sc );
       # now print rest of data
       # h is header_adjustment
-      h = 1 
+      h = 1
       retval = FFI::NCurses.prefresh(@pad,@prow + h,@pcol, sr + h , sc , @rows + sr  , @cols+ sc );
       $log.warn "XXX:  PADREFRESH #{retval}, #{@prow}, #{@pcol}, #{sr}, #{sc}, #{@rows+sr}, #{@cols+sc}." if retval == -1
       # padrefresh can fail if width is greater than NCurses.COLS
@@ -930,7 +944,7 @@ module Canis
     # set a default renderer
     #--
     #  we were not doing this automatically, so repaint was going to TP and failing on mvaddstr
-    #  2014-04-10 - 10:57 
+    #  2014-04-10 - 10:57
     #++
     def create_default_renderer
       r = DefaultTableRenderer.new self
@@ -958,7 +972,7 @@ module Canis
       end
       super
     end
-    ## 
+    ##
     # Find the next row that contains given string
     # Overrides textpad since each line is an array
     # NOTE does not go to next match within row
@@ -982,7 +996,7 @@ module Canis
           # value can be numeric
           col = f.to_s.index str
           if col
-            col += @coffsets[jx] 
+            col += @coffsets[jx]
             first ||= [ ix, col ]
             if ix > @current_index
               return [ix, col]
@@ -996,14 +1010,14 @@ module Canis
     # if yield returns true,  collects index of row into array and returns the array
     # @returns array of indices which can be empty
     # Value yielded can be fixnum or date etc
-    def matching_indices 
+    def matching_indices
       raise "block required for matching_indices" unless block_given?
       @indices = []
       ## content can be string or Chunkline, so we had to write <tt>index</tt> for this.
       @list.each_with_index do |fields, ix|
         flag = yield ix, fields
         if flag
-          @indices << ix 
+          @indices << ix
         end
       end
       #$log.debug "XXX:  INDICES found #{@indices}"
@@ -1023,7 +1037,7 @@ module Canis
       end
       @indices = nil
     end
-    ## 
+    ##
     # Ensure current row is visible, if not make it first row
     #  This overrides textpad due to header_adjustment, otherwise
     #  during next_match, the header overrides the found row.
@@ -1038,7 +1052,7 @@ module Canis
     # yields non-hidden columns (ColumnInfo) and the offset/index
     # This is the order in which columns are to be printed
     def each_column
-      @chash.each_with_index { |c, i| 
+      @chash.each_with_index { |c, i|
         next if c.hidden
         yield c,i if block_given?
       }
@@ -1051,7 +1065,7 @@ module Canis
         end
       else
         @list.each_with_index { |line, ix|
-          #FFI::NCurses.mvwaddstr(@pad,ix, 0, @list[ix])
+          # FFI::NCurses.mvwaddstr(@pad,ix, 0, @list[ix].to_s)
           render @pad, ix, line
         }
       end
@@ -1063,8 +1077,8 @@ module Canis
       ha = @_header_adjustment
       # ha takes into account whether there are headers or not
       footer = "#{@current_index+1-ha} of #{@list.length-ha} "
-      @graphic.printstring( @row + @height -1 , @col+2, footer, @color_pair || $datacolor, @footer_attrib) 
-      @repaint_footer_required = false 
+      @graphic.printstring( @row + @height -1 , @col+2, footer, @color_pair || $datacolor, @footer_attrib)
+      @repaint_footer_required = false
     end
 
   end # class Table

data/lib/canis/core/widgets/textpad.rb

@@ -4,17 +4,17 @@
 # ----------------------------------------------------------------------------- #
 #         File: textpad.rb
 #  Description: A class that displays text using a pad.
-#         The motivation for this is to put formatted text and not care about truncating and 
+#         The motivation for this is to put formatted text and not care about truncating and
 #         stuff. Also, there will be only one write, not each time scrolling happens.
 #         I found textview code for repaint being more complex than required.
 #       Author: jkepler http://github.com/mare-imbrium/mancurses/
 #         Date: 2011-11-09 - 16:59
 #      License: Same as Ruby's License (http://www.ruby-lang.org/LICENSE.txt)
-#  Last update: 2017-03-09 23:14
+#  Last update: 2019-03-12 00:17
 #
 #  == CHANGES
 #   - changed @content to @list since all multirow widgets use that and so do utils etc
-#  == TODO 
+#  == TODO
 #  Take care of 3 cases:
 #     1. complete data change, then recreate pad, and call init_vars resetting row, col and curpos etc
 #        This is done by method text().
@@ -33,7 +33,7 @@ require 'forwardable'
 include Canis
 module Canis
   extend self
-  class TextPad < Widget # 
+  class TextPad < Widget #
     include BorderTitle
     extend Forwardable
 
@@ -116,17 +116,17 @@ module Canis
     end
 
     # calculates the dimensions of the pad which will be used when the pad refreshes, taking into account
-    # whether borders are printed or not. This must be called whenever there is a change in height or width 
+    # whether borders are printed or not. This must be called whenever there is a change in height or width
     # otherwise @rows will not be recalculated.
     # Internal.
     def __calc_dimensions
-      ## NOTE 
+      ## NOTE
       #  ---------------------------------------------------
       #  Since we are using pads, you need to get your height, width and rows correct
       #  Make sure the height factors in the row, else nothing may show
       #  ---------------------------------------------------
-      
-     
+
+
       raise " CALC inside #{@name} h or w is nil #{@height} , #{@width} " if @height.nil? or @width.nil?
       @rows = @height
       @cols = @width
@@ -149,13 +149,13 @@ module Canis
         #@cols -=0
         @cols -=1
         @scrollatrows = @height - 1 # check this out 0 or 1
-        @row_offset = @col_offset = 0 
+        @row_offset = @col_offset = 0
       end
       @top = @row
       @left = @col
       @lastrow = @row + @row_offset
       @lastcol = @col + @col_offset
-      $log.debug "  CALC_DIMENSION #{@rows} , #{@cols}, #{@height} , #{@width} , #{@top} , #{@left} "
+      $log.debug "  CALC_DIMENSION r:#{@rows} , c:#{@cols}, h:#{@height} , w:#{@width} , top:#{@top} , left:#{@left} "
     end
     def scrollatrows
       unless @suppress_borders
@@ -189,12 +189,12 @@ module Canis
     ## creates the pad
     def create_pad
       destroy if @pad
-      #$log.debug "XXXCP: create_pad #{@content_rows} #{@content_cols} , w:#{@width} c #{@cols} , r: #{@rows}" 
+      #$log.debug "XXXCP: create_pad #{@content_rows} #{@content_cols} , w:#{@width} c #{@cols} , r: #{@rows}"
 
       @content_rows = @content_cols = nil
       @content_rows = pad_rows()
       @content_cols = pad_cols()
-      $log.debug "XXXCP: create_pad :#{@content_rows} , #{@content_cols} . w:#{@width} c #{@cols} , r: #{@rows}" 
+      $log.debug "XXXCP: create_pad :#{@content_rows} , #{@content_cols} . w:#{@width} c #{@cols} , r: #{@rows}"
       raise "create_pad content_rows is nil " unless @content_rows
       raise "create_pad content_cols is nil " unless @content_cols
 
@@ -204,7 +204,7 @@ module Canis
     # content_rows can be more than size of pad, but never less. Same for cols.
     # height of pad, or number of row, earlier called @content_rows
     public
-    def pad_rows 
+    def pad_rows
       # content_rows can be more than size of pad, but never less. Same for cols.
       return @content_rows if @content_rows
       content_rows = @list.count
@@ -234,7 +234,7 @@ module Canis
       @clearstring = nil
       $log.debug "  populate pad color = #{@color} , bg = #{@bgcolor} "
       #cp = get_color($datacolor, color(), bgcolor())
-      # commenting off next line meant that textdialog had a black background 2014-05-01 - 23:37 
+      # commenting off next line meant that textdialog had a black background 2014-05-01 - 23:37
       #@cp = FFI::NCurses.COLOR_PAIR(cp)
       # we seem to be clearing always since a pad is often reused. so making the variable whenever pad created.
 
@@ -271,7 +271,7 @@ module Canis
       # REST IS REQUIRED otherwise sometimes last line of window is not cleared
       # Happens in bline.rb. i think the above clears the new pad size in the window
       #  which if it is smaller then does not clear complete window.
-      ## TRYING OUT COMMENTING OFF THE REMAINDER 2014-05-31 - 14:35 
+      ## TRYING OUT COMMENTING OFF THE REMAINDER 2014-05-31 - 14:35
       # next part is messing up messageboxes which have a white background
       # so i use this copied from print_border
       # In messageboxes the border is more inside. but pad cannot clear the entire
@@ -292,7 +292,7 @@ module Canis
       sp = " " * ww
       #if color == $datacolor
       $log.debug "  clear_pad: colors #{@cp}, ( #{_bgcolor} #{_color} ) #{$datacolor} , attrib #{att} . r #{r} w #{ww}, h #{@height} top #{@window.top}  "
-      # 2014-05-15 - 11:01 seems we were clearing an extra row at bottom. 
+      # 2014-05-15 - 11:01 seems we were clearing an extra row at bottom.
       # earlier it was r+1 but that was missing the first row, so now made it r+0 2014-06-20 - 01:15 XXX
         (r+0).upto(r+@height-startcol-1) do |rr|
           @window.printstring( rr, _col ,sp , color, att)
@@ -317,7 +317,7 @@ module Canis
     public
     def padrefresh
       # sometimes padref is called directly from somewhere but dimensions have changed.
-      # 2014-05-27 - 11:42 
+      # 2014-05-27 - 11:42
       unless @__first_time
         __calc_dimensions
         @__first_time = true
@@ -350,7 +350,7 @@ module Canis
         #sr -= _t
         #$log.warn "XXX PADRE after correcting ser #{sr} and #{ser} "
       end
-      # there are some freak cases where prow or pcol comes as -1, but prefresh does not return a -1. However, this 
+      # there are some freak cases where prow or pcol comes as -1, but prefresh does not return a -1. However, this
       # could affect some other calculation somewhere.
 
       retval = FFI::NCurses.prefresh(@pad,@prow,@pcol, sr , sc , ser , sec );
@@ -376,10 +376,11 @@ module Canis
       #$log.debug "XXX:  PADREFRESH #{retval} #{self.class}, #{@prow}, #{@pcol}, #{sr}, #{sc}, #{ser}, #{sec}." if retval == 0
       # padrefresh can fail if width is greater than NCurses.COLS
       # or if height exceeds tput lines. As long as content is less, it will work
-      # the moment content_rows exceeds then this issue happens. 
+      # the moment content_rows exceeds then this issue happens.
       # @rows + sr < tput lines
       #FFI::NCurses.prefresh(@pad,@prow,@pcol, @startrow + top, @startcol + left, @rows + @startrow + top, @cols+@startcol + left);
     end
+
     # length of longest string in array
     # This will give a 'wrong' max length if the array has ansi color escape sequences in it
     # which inc the length but won't be printed. Such lines actually have less length when printed
@@ -390,10 +391,12 @@ module Canis
       return 0 unless longest
       longest.length
     end
+
     public
-    # to be called with program / user has added a row or changed column widths so that 
+
+    # to be called with program / user has added a row or changed column widths so that
     # the pad needs to be recreated. However, cursor positioning is maintained since this
-    # is considered to be a minor change. 
+    # is considered to be a minor change.
     # We do not call `init_vars` since user is continuing to do some work on a row/col.
     # NOTE : if height and width are changed then only render_all is required
     #    not a reparse since content has not changed.
@@ -405,7 +408,7 @@ module Canis
       fire_handler :DIMENSION_CHANGED, _method
       @__first_time = nil
     end
-    # repaint only one row since content of that row has changed. 
+    # repaint only one row since content of that row has changed.
     # No recreate of pad is done.
     def fire_row_changed ix
       return if ix >= @list.length
@@ -414,7 +417,7 @@ module Canis
       fire_handler :ROW_CHANGED, ix
       _arr = _getarray
       render @pad, ix, _arr[ix]
-    
+
     end
 # ---- end pad related ----- }}}
 # ---- Section render related  ----- {{{
@@ -447,13 +450,13 @@ module Canis
     # in Window and Pad stuff and perhaps include it conditionally.
 
 
-    # before updating a single row in a table 
+    # before updating a single row in a table
     # we need to clear the row otherwise previous contents can show through
     def clear_row pad, lineno
       if @renderer and @renderer.respond_to? :clear_row
           @renderer.clear_row pad, lineno
       else
-        # need pad width not window width, the other clearstring uses width of 
+        # need pad width not window width, the other clearstring uses width of
         #  widget to paint on window.
         @_clearstring ||= " " * @content_cols
         # what about bg color ??? XXX, left_margin and internal width
@@ -461,7 +464,7 @@ module Canis
         cp = @cp || FFI::NCurses.COLOR_PAIR($datacolor)
         att = attr() || NORMAL
         FFI::NCurses.wattron(pad,cp | att)
-        FFI::NCurses.mvwaddstr(pad,lineno, 0, @_clearstring) 
+        FFI::NCurses.mvwaddstr(pad,lineno, 0, @_clearstring)
         FFI::NCurses.wattroff(pad,cp | att)
       end
     end
@@ -471,7 +474,7 @@ module Canis
       return unless @print_footer
       return unless @suppress_borders
       footer = "R: #{@current_index+1}, C: #{@curpos+@pcol}, #{@list.length} lines  "
-      @graphic.printstring( @row + @height -1 , @col+2, footer, @color_pair || $datacolor, @footer_attrib) 
+      @graphic.printstring( @row + @height -1 , @col+2, footer, @color_pair || $datacolor, @footer_attrib)
 =begin
       if @list_footer
         if false
@@ -480,18 +483,18 @@ module Canis
           footer_attrib = @list_footer.config[:attrib] ||  Ncurses::A_REVERSE
           #footer = "R: #{@current_index+1}, C: #{@curpos+@pcol}, #{@list.length} lines  "
           $log.debug " print_foot calling printstring with #{@row} + #{@height} -1, #{@col}+2"
-          @graphic.printstring( @row + @height -1 , @col+2, footer, @color_pair || $datacolor, footer_attrib) 
+          @graphic.printstring( @row + @height -1 , @col+2, footer, @color_pair || $datacolor, footer_attrib)
         end
         # use default print method which only prints on left
         @list_footer.print self
       end
 =end
-      @repaint_footer_required = false # 2010-01-23 22:55 
+      @repaint_footer_required = false # 2010-01-23 22:55
     end
 
     # ---- Section render related  end ----- }}}
 # ---- Section data related start {{{
-    
+
     # supply a filename as source for textpad
     # Reads up file into @list
     # One can optionally send in a method which takes a filename and returns an array of data
@@ -553,7 +556,7 @@ module Canis
 
           @document = val[0]
           @document.source ||= self
-          @document.parse_required # added 2014-09-03 - 17:54 
+          @document.parse_required # added 2014-09-03 - 17:54
           @list = @document.text
         when Array
           # This is the complex case which i would like to phase out.
@@ -597,7 +600,7 @@ module Canis
 
         @document = val
         @document.source ||= self
-        @document.parse_required # added 2014-09-03 - 17:54 
+        @document.parse_required # added 2014-09-03 - 17:54
         @list = @document.text
       end
       @_populate_needed = true
@@ -618,8 +621,8 @@ module Canis
       fmt = val.size == 2 ? val[1] : nil
       case fmt
       when Hash
-        #raise "textpad.text expected content_type in Hash : #{fmt}" 
-        c = fmt[:content_type] 
+        #raise "textpad.text expected content_type in Hash : #{fmt}"
+        c = fmt[:content_type]
         t = fmt[:title]
         @title = t if t
         @content_type = c if c
@@ -631,11 +634,11 @@ module Canis
         @content_type = fmt
       when NilClass
       else
-        raise "textpad.text expected symbol or content_type in Hash, got #{fmt.class} " 
+        raise "textpad.text expected symbol or content_type in Hash, got #{fmt.class} "
       end
 
       ## some programs like testlistbox which uses multibuffers calls this with a config
-      # in arg2 containing :content_type and :title 
+      # in arg2 containing :content_type and :title
 
 
       # added so callers can have one interface and avoid an if condition
@@ -708,7 +711,7 @@ module Canis
       #@native_text[@current_index]
       _getarray[@current_index]
     end
-    ## NOTE : 2014-04-09 - 14:05 i think this does not have line wise operations since we deal with 
+    ## NOTE : 2014-04-09 - 14:05 i think this does not have line wise operations since we deal with
     #    formatting of data
     #    But what if data is not formatted. This imposes a severe limitation. listbox does have linewise
     #    operations, so lets try them
@@ -723,10 +726,10 @@ module Canis
       self
     end
     # @deprecated : row_count used just for compat, use length or size
-    def row_count ; @list.length ; end  
+    def row_count ; @list.length ; end
 
     ## ------ LIST / ARRAY OPERATIONS ----
-    # All multirow widgets must use Array semantics 2014-04-10 - 17:29 
+    # All multirow widgets must use Array semantics 2014-04-10 - 17:29
     # NOTE some operations will make selected indices in selection modules invalid
     # clear will need to clear indices, delete_at and insert may need to also adjust
     # selection or focus index/es.
@@ -739,7 +742,7 @@ module Canis
     # delegate some modify operations to Array: insert, clear, delete_at, []= <<
     # However, we should check if content array is nil ?
     # fire_dim is called, although it is not required in []=
-    %w[ insert delete_at << push].each { |e| 
+    %w[ insert delete_at << push].each { |e|
       eval %{
       def #{e}(*args)
         @list ||= []
@@ -755,7 +758,7 @@ module Canis
     def clear
       return unless @list
       @list.clear
-      @native_text.clear if @native_text # check this line, should it be removed 2014-08-27 - 20:54 
+      @native_text.clear if @native_text # check this line, should it be removed 2014-08-27 - 20:54
       fire_dimension_changed :clear
       init_vars
     end
@@ -805,7 +808,7 @@ module Canis
       $multiplier = 0
     end
     def top_of_window
-      @current_index = @prow 
+      @current_index = @prow
       $multiplier ||= 0
       if $multiplier > 0
         @current_index += $multiplier
@@ -884,7 +887,7 @@ module Canis
       @prow = @current_index - @scrollatrows
     end
 
-    # scrolls lines backward a window full at a time, on pressing pageup 
+    # scrolls lines backward a window full at a time, on pressing pageup
     # C-u may not work since it is trapped by form earlier. Need to fix
     def scroll_backward
       #@oldindex = @current_index
@@ -985,7 +988,7 @@ module Canis
       # if curpos is at zero , we should be checking previous line !
       $multiplier.times {
         # if at start of line, go to previous line
-        if pos == 0 
+        if pos == 0
           if @current_index > 0
             line -= 1
             pos = _arr[line].to_s.size
@@ -1056,7 +1059,7 @@ module Canis
       @curpos = _arr[@current_index].size
       @repaint_required = true
     end
-    # 
+    #
     # moves cursor to start of line, panning if required
     def cursor_bol
       # copy of C-a - start of line
@@ -1064,7 +1067,7 @@ module Canis
       @pcol = 0
       @curpos = 0
     end
-    # 
+    #
     # return true if the given row is visible
     def is_visible? index
       j = index - @prow #@toprow
@@ -1117,7 +1120,7 @@ module Canis
     # event when user hits ENTER on a row, user would bind :PRESS
     # callers may use +text()+ to get the value of the row, +source+ to get parent object.
     #
-    #     obj.bind :PRESS { |eve| eve.text } 
+    #     obj.bind :PRESS { |eve| eve.text }
     #
     def fire_action_event
       return if @list.nil? || @list.size == 0
@@ -1129,7 +1132,7 @@ module Canis
     def text_action_event
       aev = TextActionEvent.new self, :PRESS, current_value().to_s, @current_index, @curpos
     end
-    # 
+    #
     # execute binding when a row is entered, used more in lists to display some text
     # in a header or footer as one traverses
     #
@@ -1137,18 +1140,15 @@ module Canis
       return nil if @list.nil? || @list.size == 0
 
       @repaint_footer_required = true
-      #alert "on_enter rr #{@repaint_required}, #{@repaint_all} oi #{@oldindex}, ci #{@current_index}, or #{@oldrow}  "
 
       ## can this be done once and stored, and one instance used since a lot of traversal will be done
       require 'canis/core/include/ractionevent'
       aev = TextActionEvent.new self, :ENTER_ROW, current_value().to_s, @current_index, @curpos
       fire_handler :ENTER_ROW, aev
-      #@repaint_required = true
     end
 
     #
     # called when this widget is entered, by form
-    # 2014-05-27 - 17:02 we were not calling super, so :ENTER was not triggered !!!
     def on_enter
       super
       set_form_row
@@ -1161,11 +1161,6 @@ module Canis
     def set_form_col
     end
 
-    private
-    
-    # check that current_index and prow are within correct ranges
-    # sets row (and someday col too)
-    # sets repaint_required
 
     public
     def bounds_check
@@ -1198,7 +1193,7 @@ module Canis
         @repaint_required = true
       end
     end
-    # 
+    #
     # save last cursor position so when reentering, cursor can be repositioned
     def lastcurpos r,c
       @lastrow = r
@@ -1228,7 +1223,7 @@ module Canis
       #$log.debug "  check_prow after prow #{@prow} , list count #{cc} "
       # we still need to check the max that prow can go otherwise
       # the pad shows earlier stuff.
-      # 
+      #
       return
     end
     public
@@ -1237,7 +1232,7 @@ module Canis
         __calc_dimensions
         @__first_time = true
       end
-      return unless @list # trying out since it goes into padrefresh even when no data 2014-04-10 - 00:32 
+      return unless @list # trying out since it goes into padrefresh even when no data 2014-04-10 - 00:32
       @graphic = @form.window unless @graphic
       @window ||= @graphic
       raise "Window not set in textpad" unless @window
@@ -1253,16 +1248,16 @@ module Canis
       #$log.debug "  repaint textpad RR #{@repaint_required} #{@window.top} "
       unless @repaint_required
         print_foot if @repaint_footer_required  # set in on_enter_row
-        # trying out removing this, since too many refreshes 2014-05-01 - 12:45 
-        #padrefresh 
-        return 
+        # trying out removing this, since too many refreshes 2014-05-01 - 12:45
+        #padrefresh
+        return
       end
-      # if repaint is required, print_foot not called. unless repaint_all is set, and that 
+      # if repaint is required, print_foot not called. unless repaint_all is set, and that
       # is rarely set.
-      
+
       preprocess_text
 
-      # in textdialog, @window was nil going into create_pad 2014-04-15 - 01:28 
+      # in textdialog, @window was nil going into create_pad 2014-04-15 - 01:28
 
       # creates pad and calls render_all
       populate_pad if !@pad or @_populate_needed
@@ -1295,8 +1290,8 @@ module Canis
           @window.print_border_only @top, @left, @height-1, @width, clr
           print_title
 
-          # oldrow changed to oldindex 2014-04-13 - 16:55 
-          @repaint_footer_required = true if @oldindex != @current_index 
+          # oldrow changed to oldindex 2014-04-13 - 16:55
+          @repaint_footer_required = true if @oldindex != @current_index
           print_foot if @print_footer && !@suppress_borders && @repaint_footer_required
 
           @window.wrefresh
@@ -1314,15 +1309,15 @@ module Canis
       bindings
 =begin
       bind_key([?g,?g], 'goto_start'){ goto_start } # mapping double keys like vim
-      bind_key(279, 'goto_start'){ goto_start } 
-      bind_keys([?G,277], 'goto end'){ goto_end } 
-      bind_keys([?k,KEY_UP], "Up"){ up } 
-      bind_keys([?j,KEY_DOWN], "Down"){ down } 
-      bind_key(?\C-e, "Scroll Window Down"){ scroll_window_down } 
-      bind_key(?\C-y, "Scroll Window Up"){ scroll_window_up } 
-      bind_keys([32,338, ?\C-d], "Scroll Forward"){ scroll_forward } 
-      # adding CTRL_SPACE as back scroll 2014-04-14 
-      bind_keys([0,?\C-b,339], "Scroll Backward"){ scroll_backward } 
+      bind_key(279, 'goto_start'){ goto_start }
+      bind_keys([?G,277], 'goto end'){ goto_end }
+      bind_keys([?k,KEY_UP], "Up"){ up }
+      bind_keys([?j,KEY_DOWN], "Down"){ down }
+      bind_key(?\C-e, "Scroll Window Down"){ scroll_window_down }
+      bind_key(?\C-y, "Scroll Window Up"){ scroll_window_up }
+      bind_keys([32,338, ?\C-d], "Scroll Forward"){ scroll_forward }
+      # adding CTRL_SPACE as back scroll 2014-04-14
+      bind_keys([0,?\C-b,339], "Scroll Backward"){ scroll_backward }
       # the next one invalidates the single-quote binding for bookmarks
       #bind_key([?',?']){ goto_last_position } # vim , goto last row position (not column)
       bind_key(?/, :ask_search)
@@ -1351,7 +1346,7 @@ module Canis
 # ----------- end internal stuff --------------- }}}
     public
 # ---- Section search related start ----- {{{
-    ## 
+    ##
     # Ask user for string to search for
     # This uses the dialog, but what if user wants the old style.
     # Isn't there a cleaner way to let user override style, or allow user
@@ -1363,7 +1358,7 @@ module Canis
       # the following is a change that enables callers to prompt for the string
       # using some other style, basically the classical style and send the string in
       str = get_string("Enter pattern: ", :title => "Find pattern") unless str
-      return if str.nil? 
+      return if str.nil?
       str = @last_regex if str == ""
       return if !str or str == ""
       str = Regexp.new str if str.is_a? String
@@ -1376,7 +1371,7 @@ module Canis
       @curpos = ix[1]
       ensure_visible
     end
-    ## 
+    ##
     # Find next matching row for string accepted in ask_search
     #
     def find_more
@@ -1389,7 +1384,7 @@ module Canis
       ensure_visible
     end
 
-    ## 
+    ##
     # Find the next row that contains given string
     # @return row and col offset of match, or nil
     # FIXME: 2014-05-26 - 01:26 currently if there are two matches in a row skips the second
@@ -1429,7 +1424,7 @@ module Canis
     # NOTE:
     #    startline and endline are more for internal purposes, externally we would call this only with
     #    the pattern.
-    # @example 
+    # @example
     #     pos = next_match /abc/
     #     # pos is nil or [line, col]
     #
@@ -1448,7 +1443,7 @@ module Canis
         _line = to_searchable(startline)
         _col = _line.index(str, pos) if _line
         if _col
-          return [startline, _col] 
+          return [startline, _col]
         end
         startline += 1 # FIXME check this end of file
       end
@@ -1485,10 +1480,10 @@ module Canis
     def to_searchable index
       _getarray[index].to_s
     end
-    ## 
+    ##
     # Ensure current row is visible, if not make it first row
     # NOTE - need to check if its at end and then reduce scroll at rows, check_prow does that
-    # 
+    #
     # @param current_index (default if not given)
     #
     def ensure_visible row = @current_index
@@ -1502,7 +1497,7 @@ module Canis
     # where the current index is actually being displayed (example if it wishes to display
     # a popup at that row)
     # An argument is not being taken since the index should be visible.
-    def visual_index 
+    def visual_index
       row = @current_index
       row - @prow
     end
@@ -1517,7 +1512,7 @@ module Canis
     #
     # This was just indicative, and is not used anywhere
     def DEADhighlight_row index = @current_index, cfg={}
-      return unless index 
+      return unless index
       c = 0 # we are using pads so no col except for left_margin if present
       # in a pad we don't need to convert index to printable
       r = index
@@ -1530,7 +1525,7 @@ module Canis
     end
 ##---- dead unused }}}
 
-  end  # class textpad 
+  end  # class textpad
 
 # renderer {{{
 # Very basic renderer that only prints based on color pair of the textpad
@@ -1540,7 +1535,7 @@ module Canis
     # content cols is the width in columns of pad
     # list is the data array
     attr_accessor :content_cols, :list
-    # the widget this is associated with. 
+    # the widget this is associated with.
     attr_accessor :source
 
     def initialize source=nil
@@ -1558,7 +1553,7 @@ module Canis
     alias :update_colors :pre_render
 
     # derived classes may choose to override this.
-    # However, they should set size and color and attrib at the start since these 
+    # However, they should set size and color and attrib at the start since these
     # can change after the object has been created depending on the application.
     def render_all pad, arr
       pre_render
@@ -1609,7 +1604,7 @@ module Canis
   end
 # renderer }}}
 # This is the default key handler.
-  # It takes care of catching numbers so that vim's movement can use numeric args. 
+  # It takes care of catching numbers so that vim's movement can use numeric args.
   # That is taken care of by multiplier. Other than that it has the key_map process the key.
   #
   class DefaultKeyHandler # ---- {{{
@@ -1649,10 +1644,10 @@ module Canis
           end
           # --- NOTE ABOUT BLACK RECT LEFT on SCREEN {{{
           ## NOTE if textpad does not handle the event and it goes to form which pops
-          # up a messagebox, then padrefresh does not happen, since control does not 
+          # up a messagebox, then padrefresh does not happen, since control does not
           # come back here, so a black rect is left on screen
-          # please note that a bounds check will not happen for stuff that 
-          # is triggered by form, so you'll have to to it yourself or 
+          # please note that a bounds check will not happen for stuff that
+          # is triggered by form, so you'll have to to it yourself or
           # call setrowcol explicity if the cursor is not updated
           # --- }}}
 

data/lib/canis/version.rb

@@ -1,3 +1,3 @@
 module Canis
-  VERSION = "0.0.11"
+  VERSION = "0.0.13"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: canis
 version: !ruby/object:Gem::Version
-  version: 0.0.11
+  version: 0.0.13
 platform: ruby
 authors:
 - kepler
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-05-05 00:00:00.000000000 Z
+date: 2019-03-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -220,8 +220,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 3.0.2
 signing_key: 
 specification_version: 4
 summary: ruby ncurses library for easy application development