cosmos 3.3.3 → 3.4.0

data/ext/cosmos/ext/telemetry/telemetry.c

@@ -171,8 +171,9 @@ static VALUE value(int argc, VALUE* argv, VALUE self)
  *   can be passed which will convert all items the same way. Or
  *   an array of symbols can be passed to control how each item is
  *   converted.
- * @return [Array, Array] The first array contains the item values and the
- *   second their limits state
+ * @return [Array, Array, Array] The first array contains the item values, the
+ *   second their limits state, and the third the limits settings which includes
+ *   red, yellow, and green (if given) limits values.
  */
 static VALUE values_and_limits_states(int argc, VALUE* argv, VALUE self) {
   VALUE item_array = Qnil;

data/lib/cosmos/gui/line_graph/line_graph_drawing.rb

@@ -33,7 +33,7 @@ module Cosmos
       calculate_scaling_factors()
 
       # Draw overall graph and origin lines
-      clear_canvas_and_draw_graph_rectangle(@painter)
+      draw_graph_background(@painter)
       draw_origin_lines(@painter)
 
       # Draw gridlines and titles
@@ -45,7 +45,6 @@ module Cosmos
       draw_y_axis_title(@painter, :LEFT)
       draw_y_axis_title(@painter, :RIGHT)
 
-      # Draw legend and lines
       draw_legend(@painter)
 
       draw_lines(@painter, :LEFT)
@@ -54,7 +53,6 @@ module Cosmos
 
     # Draws the graph to the screen
     def draw_graph_to_screen
-      # Draw frame around graph
       draw_frame(@painter)
 
       # Draw cursor line and popups if present
@@ -64,16 +62,10 @@ module Cosmos
       draw_error_icon(@painter)
     end # def draw_graph_to_screen
 
-    # Clears the entire canvas and then draws the colored rectangle for the graph
-    def clear_canvas_and_draw_graph_rectangle(dc)
-      # Draw graph background
-      color = Cosmos::getColor(@graph_back_color)
-      dc.setPen(color)
-      dc.setBrush(Cosmos.getBrush(@@gradient))
-      dc.drawRect(@graph_left_x,@graph_top_y,@graph_right_x - @graph_left_x,@graph_bottom_y - @graph_top_y)
-      # Draw the graph border
-      dc.addRectColor(@graph_left_x, @graph_top_y, @graph_right_x - @graph_left_x, @graph_bottom_y - @graph_top_y, @label_and_border_color)
-    end # def clear_canvas_and_draw_graph_rectangle
+    # Draws the colored rectangle for the graph
+    def draw_graph_background(dc)
+      dc.addRectColorFill(@graph_left_x, @graph_top_y, @graph_right_x - @graph_left_x, @graph_bottom_y - @graph_top_y, @label_and_border_color, @@gradient)
+    end
 
     # Draws origin lines if they fall on the graph
     def draw_origin_lines(dc)
@@ -93,14 +85,19 @@ module Cosmos
 
     # Draws the gridlines for a y-axis
     def draw_y_axis_grid_lines(dc)
+      grid_lines = []
       @y_grid_lines.each_with_index do |value, index|
         # Don't draw gridlines that are too close to 0
         if ((value > (@y_grid_line_scale / 2.0)) || (value < (@y_grid_line_scale / -2.0)))
-          draw_y_label_and_grid_line(dc, value, index, true)
+          grid_lines << draw_y_label(dc, value, index)
         else
-          draw_y_label_and_grid_line(dc, 0, index, true)
+          grid_lines << draw_y_label(dc, 0, index)
         end
       end
+      # Now draw all the grid lines so we can use a single Pen for all
+      grid_lines.each do |y|
+        dc.addLineColor(@graph_left_x, y, @graph_right_x, y, Cosmos::DASHLINE_PEN)
+      end
     end # def draw_y_axis_grid_lines
 
     # Calcuate the Y axis labels as well as their width and adjust the size of
@@ -144,10 +141,8 @@ module Cosmos
       @graph_right_x -= (right_widths.max + GRAPH_SPACER)
     end
 
-    # This function is used to draw a horizontal line on the graph with a label.
-    # This line is not clipped so the value must fall on the graph. Uses the
-    # label values calculated in calculate_y_labels.
-    def draw_y_label_and_grid_line(dc, value, index, show_line)
+    # This function is used to draw the y labels.
+    def draw_y_label(dc, value, index)
       left_value = value
       right_value = value
       left_text = @left_text[index]
@@ -193,28 +188,29 @@ module Cosmos
                              y + (@font_size / 2))
         end
       end
-
-      if show_line and y
-        dc.addLineColor(@graph_left_x, y, @graph_right_x, y, Cosmos::DASHLINE_PEN)
-      end
-
-    end # def draw_y_label_and_grid_line
+      return y
+    end
 
     # Draws the gridlines for the x-axis
     def draw_x_axis_grid_lines(dc)
+      grid_lines = []
       @x_grid_lines.each do |value, label|
         # If the line has states or is far enough away from the origin
         if @lines.x_states || ((value > (@x_grid_line_scale / 2.0)) || (value < (@x_grid_line_scale / -2.0)))
-          draw_x_label_and_grid_line(dc, value, label, true)
+          grid_lines << draw_x_label(dc, value, label)
         else
-          draw_x_label_and_grid_line(dc, 0, nil, true)
+          grid_lines << draw_x_label(dc, 0, nil)
         end
       end
+      # Now draw all the grid lines so we can use a single Pen for all
+      grid_lines.each do |x1, y1, x2, y2|
+        dc.addLineColor(x1, y1, x2, y2, Cosmos::DASHLINE_PEN)
+      end
     end # def draw_x_axis_grid_lines
 
-    # This function is used to draw a vertical line on the graph with a label.
-    # This line is not clipped so the value must fall on the graph
-    def draw_x_label_and_grid_line(dc, value, label, show_line)
+    # This function is used to draw the x labels and returns the line
+    # positions.
+    def draw_x_label(dc, value, label)
       if label
         text = label.to_s
       else
@@ -231,13 +227,8 @@ module Cosmos
       if (x1 > @graph_right_x)
         x1 = @graph_right_x
       end
-      if show_line
-        y2 = @graph_top_y
-      else
-        y2 = @graph_bottom_y
-      end
+      y2 = @graph_top_y
       x2 = x1
-      dc.addLineColor(x1, y1, x2, y2, Cosmos::DASHLINE_PEN)
 
       # Only display the label if we have room. This really only affects the
       # right side of the graph since that's where new grid lines appear. The
@@ -245,10 +236,11 @@ module Cosmos
       # white space on the right side of the graph.
       if (x1 == @graph_right_x) || (x1 < (@graph_right_x - (1.25 * text_width)))
         # Shift the far right label left a bit to eliminate white space
-        x1 -= text_width / 4 if x1 == @graph_right_x
-        x1 -= text_width / 2 # center the text
-        dc.addSimpleTextAt(text, x1, @graph_bottom_y + text_height + GRAPH_SPACER)
+        text_x = x1 - text_width / 4 if x1 == @graph_right_x
+        text_x = x1 - text_width / 2 # center the text
+        dc.addSimpleTextAt(text, text_x, @graph_bottom_y + text_height + GRAPH_SPACER)
       end
+      [x1, y1, x2, y2]
     end # def draw_x_label_and_grid_line
 
     # Converts a x value into text with a max number of characters
@@ -286,7 +278,7 @@ module Cosmos
       end
     end
 
-    # Draw the overall graph title
+    # Draw the overall graph title above the graph
     def draw_title(dc)
       if @title
         metrics = Cosmos.getFontMetrics(@title_font)
@@ -294,9 +286,9 @@ module Cosmos
         dc.addSimpleTextAt(@title, (self.width / 2) - (metrics.width(@title) / 2), metrics.height)
         dc.setFont(@font)
       end
-    end # def draw_title
+    end
 
-    # Draws the title for the X-axis
+    # Draws the x axis title below the graph
     def draw_x_axis_title(dc)
       if @x_axis_title
         metrics = Cosmos.getFontMetrics(@font)
@@ -306,9 +298,9 @@ module Cosmos
           (((@graph_right_x - @graph_left_x) / 2) + @graph_left_x) - (text_width / 2),
           @graph_bottom_y + (2 * text_height) + GRAPH_SPACER)
       end
-    end # def draw_x_axis_title
+    end
 
-    # Draws a title for a Y-axis
+    # Draws titles to the left and right of the Y axis
     def draw_y_axis_title(dc, axis)
       metrics = Cosmos.getFontMetrics(@font)
       if axis == :LEFT
@@ -330,58 +322,46 @@ module Cosmos
           dc.addSimpleTextAt(character, graph_x + ((max_width - cur_width) / 2), start_height + text_height * index)
         end
       end
-    end # def draw_y_axis_title
+    end
 
     # Draws the legend on the bottom of the graph
     def draw_legend(dc)
+      return if !@show_legend || @lines.empty?
+
+      text_x, text_y, legend_width, legend_height = get_legend_position()
+      if @lines.axes == :BOTH
+        draw_legend_text(dc, :LEFT, text_x, text_y, legend_height)
+        draw_legend_text(dc, :RIGHT, text_x + (legend_width / 2), text_y, legend_height)
+      else
+        draw_legend_text(dc, false, text_x, text_y, legend_height)
+      end
+    end
+
+    # Calculate the legend x, y, width and height
+    def get_legend_position
       metrics = Cosmos.getFontMetrics(@font)
-      # Draw Legend
-      if @show_legend && !@lines.empty?
-        if @lines.axes == :BOTH
-          both_axis = true
-        else
-          both_axis = false
-        end
-        legend_width = 0
-        @lines.legend.each do |legend_text, color, axis|
-          text_width   = metrics.width(legend_text)
-          legend_width = text_width if text_width > legend_width
-        end
-        legend_width += (GRAPH_SPACER * 2)
-        legend_width *= 2 if both_axis
-        legend_graph_x = (self.width - legend_width) / 2
-
-        text_x = legend_graph_x + GRAPH_SPACER
-        text_y = self.height - metrics.height
-
-        # Draw legend text for each line
-        if both_axis
-          left_text_x  = text_x
-          right_text_x = text_x + (legend_width / 2)
-          left_text_y  = text_y
-          right_text_y = text_y
-          @lines.legend.reverse_each do |legend_text, color, axis|
-            if axis == :LEFT
-              dc.addSimpleTextAt(legend_text, left_text_x, left_text_y, color)
-              left_text_y -= metrics.height
-            else
-              dc.addSimpleTextAt(legend_text, right_text_x, right_text_y, color)
-              right_text_y -= metrics.height
-            end
-          end
-          if left_text_y < right_text_y
-            text_y = left_text_y
-          else
-            text_y = right_text_y
-          end
-        else
-          @lines.legend.reverse_each do |legend_text, color, axis|
-            dc.addSimpleTextAt(legend_text, text_x, text_y, color)
-            text_y -= metrics.height
-          end
+      legend_width = 0
+      @lines.legend.each do |legend_text, color, axis|
+        text_width   = metrics.width(legend_text)
+        legend_width = text_width if text_width > legend_width
+      end
+      legend_width += (GRAPH_SPACER * 2)
+      legend_width *= 2 if @lines.axes == :BOTH
+      legend_graph_x = (self.width - legend_width) / 2
+
+      text_x = legend_graph_x + GRAPH_SPACER
+      text_y = self.height - metrics.height
+      return [text_x, text_y, legend_width, metrics.height]
+    end
+
+    def draw_legend_text(dc, specific_axis, text_x, text_y, line_height)
+      @lines.legend.reverse_each do |legend_text, color, axis|
+        if !specific_axis || specific_axis == axis
+          dc.addSimpleTextAt(legend_text, text_x, text_y, color)
+          text_y -= line_height
         end
       end
-    end # def draw_legend
+    end
 
     # Draws all lines for the given axis
     # def draw_lines(dc, axis)
@@ -438,9 +418,8 @@ module Cosmos
       end # unless popups.empty?
     end
 
-    # Draws the overall frame around the canvas
+    # Draws the overall frame around the graph, legend, labels, etc
     def draw_frame(dc)
-      # Draw Frame Rectangle around canvas
       dc.addRectColor(0,0,self.width-1,self.height-1, @frame_color)
     end
 

data/lib/cosmos/gui/line_graph/overview_graph.rb

@@ -400,7 +400,7 @@ module Cosmos
 
     def draw_graph_into_back_buffer
       # Draw overall graph and origin lines and graph lines
-      clear_canvas_and_draw_graph_rectangle(@painter)
+      draw_graph_background(@painter)
       draw_origin_lines(@painter)
       draw_lines(@painter, :LEFT)
 

data/lib/cosmos/gui/qt.rb

@@ -94,6 +94,7 @@ module Cosmos
     color = nil
     key = color_r
     key = key.to_i if key.is_a? Qt::Enum
+
     if color_r && color_g && color_b
       key = (color_r.to_i << 24) + (color_g.to_i << 16) + (color_b.to_i << 8)
     end
@@ -112,6 +113,8 @@ module Cosmos
   end
 
   def self.getBrush(color)
+    return nil unless color
+    return color if color.is_a? Qt::Brush
     brush = nil
     color = Cosmos.getColor(color)
     brush = BRUSHES[color]
@@ -280,10 +283,16 @@ class Qt::TableWidget
     resizeRowsToContents()
     setHorizontalScrollBarPolicy(Qt::ScrollBarAlwaysOff)
     setVerticalScrollBarPolicy(Qt::ScrollBarAlwaysOff)
-    setMinimumSize(2*frameWidth() + horizontalHeader.length + verticalHeader.width,
-                   2*frameWidth() + verticalHeader.length + horizontalHeader.height)
-    setMaximumSize(2*frameWidth() + horizontalHeader.length + verticalHeader.width,
-                   2*frameWidth() + verticalHeader.length + horizontalHeader.height)
+    setMinimumSize(fullWidth, fullHeight)
+    setMaximumSize(fullWidth, fullHeight)
+  end
+
+  def fullWidth
+    2*frameWidth() + horizontalHeader.length + verticalHeader.width
+  end
+
+  def fullHeight
+    2*frameWidth() + verticalHeader.length + horizontalHeader.height
   end
 end
 
@@ -669,47 +678,52 @@ class Qt::ColorListWidget < Qt::ListWidget
 end
 
 class Qt::Painter
+  def setPen(pen_color)
+    super(Cosmos::getColor(pen_color))
+    @pen_color = pen_color
+  end
+  def setBrush(brush)
+    super(Cosmos::getBrush(brush))
+    @brush = brush
+  end
+
   def addLineColor(x, y, w, h, color = Cosmos::BLACK)
-    setPenColor(color)
+    setPen(color) if color != @pen_color
     drawLine(x,y,w,h)
   end
 
   def addRectColor(x, y, w, h, color = Cosmos::BLACK)
-    setPenColor(color)
-    setBrush(nil)
+    setPen(color) if color != @pen_color
+    setBrush(nil) if @brush
     drawRect(x,y,w,h)
   end
 
-  def addRectColorFill(x, y, w, h, color = Cosmos::BLACK)
-    setPenColor(color)
-    setBrush(Cosmos.getBrush(color))
+  # Note if brush_color is not specified it will be the same as pen_color
+  def addRectColorFill(x, y, w, h, pen_color = Cosmos::BLACK, brush_color = nil)
+    setPen(pen_color) if pen_color != @pen_color
+    brush_color = pen_color unless brush_color
+    setBrush(brush_color) if brush_color != @brush
     drawRect(x,y,w,h)
   end
 
   def addSimpleTextAt(text, x, y, color = Cosmos::BLACK)
-    setPenColor(color)
+    setPen(color) if color != @pen_color
     drawText(x,y,text)
   end
 
   def addEllipseColor(x, y, w, h, color = Cosmos::BLACK)
-    setPenColor(color)
-    setBrush(nil)
+    setPen(color) if color != @pen_color
+    setBrush(nil) if @brush
     drawEllipse(x,y,w,h)
   end
 
-  def addEllipseColorFill(x, y, w, h, color = Cosmos::BLACK)
-    setPenColor(color)
-    setBrush(Cosmos.getBrush(color))
+  # Note if brush_color is not specified it will be the same as pen_color
+  def addEllipseColorFill(x, y, w, h, pen_color = Cosmos::BLACK, brush_color = nil)
+    setPen(pen_color) if pen_color != @pen_color
+    brush_color = pen_color unless brush_color
+    setBrush(brush_color) if brush_color != @brush
     drawEllipse(x,y,w,h)
   end
-
-  private
-
-  def setPenColor(color)
-    color = Cosmos::getColor(color)
-    setPen(color)
-  end
-
 end
 
 class Qt::MatrixLayout < Qt::GridLayout

data/lib/cosmos/gui/text/ruby_editor.rb

@@ -363,7 +363,7 @@ module Cosmos
                              number.to_s)     # text
 
             if @enable_breakpoints and @breakpoints.include?(number)
-              painter.setBrush(Cosmos.getBrush(Cosmos::RED))
+              painter.setBrush(Cosmos::RED)
               painter.drawEllipse(2,
                                   top+2,
                                   ellipse_width,

data/lib/cosmos/packets/binary_accessor.rb

@@ -111,294 +111,7 @@ module Cosmos
     # @param endianness [Symbol] {ENDIANNESS}
     # @param overflow [Symbol] {OVERFLOW_TYPES}
     # @return [Integer] value passed in as a parameter
-    def self.write(value, bit_offset, bit_size, data_type, buffer, endianness, overflow)
-      # Save given values of bit offset and bit size
-      given_bit_offset = bit_offset
-      given_bit_size = bit_size
-
-      # Handle negative and zero bit sizes
-      if bit_size <= 0
-        if data_type == :STRING or data_type == :BLOCK
-          if given_bit_offset < 0
-            raise ArgumentError, "negative or zero bit_sizes (#{given_bit_size}) cannot be given with negative bit_offsets (#{given_bit_offset})"
-          else
-            bit_size = value.to_s.length * 8
-          end
-        else
-          raise ArgumentError, "bit_size #{given_bit_size} must be positive for data types other than :STRING and :BLOCK"
-        end
-      end
-
-      # Handle negative bit offsets
-      if bit_offset < 0
-        bit_offset = ((buffer.length * 8) + bit_offset)
-        raise_buffer_error(:write, buffer, data_type, given_bit_offset, given_bit_size) if bit_offset < 0
-      end
-
-      # Define bounds of string to access this item
-      lower_bound = bit_offset / 8
-      upper_bound = (bit_offset + bit_size - 1) / 8
-
-      # Check for byte alignment
-      byte_aligned = ((bit_offset % 8) == 0)
-
-      # Sanity check buffer size
-      if upper_bound >= buffer.length and given_bit_size > 0
-        # Check special case of little endian bit field
-        if endianness == :LITTLE_ENDIAN and (data_type == :INT or data_type == :UINT) and !(byte_aligned and (bit_size == 8 or bit_size == 16 or bit_size == 32 or bit_size == 64)) and lower_bound < buffer.length
-          # Ok little endian bit field
-        else
-          raise_buffer_error(:write, buffer, data_type, given_bit_offset, given_bit_size)
-        end
-      end
-
-      # Check overflow type
-      raise "unknown overflow type #{overflow}" unless OVERFLOW_TYPES.include?(overflow)
-
-      case data_type
-      when :STRING, :BLOCK
-        #######################################
-        # Handle :STRING and :BLOCK data types
-        #######################################
-
-        # Ensure value is the correct type
-        value = value.to_s
-
-        if byte_aligned
-          temp = value
-          if given_bit_size <= 0
-            end_bytes = -(given_bit_size / 8)
-            old_upper_bound = buffer.length - 1 - end_bytes
-            if old_upper_bound < lower_bound
-              # String was completely empty
-              if end_bytes > 0
-                # Preserve bytes at end of buffer
-                buffer_length = buffer.length
-                buffer << ZERO_STRING * value.length
-                buffer[(lower_bound + value.length)..(buffer.length - 1)] = buffer[lower_bound..(buffer_length - 1)]
-              end
-            elsif bit_size == 0
-              # Remove entire string
-              buffer[lower_bound..old_upper_bound] = ''
-            elsif upper_bound < old_upper_bound
-              # Remove extra bytes from old string
-              buffer[(upper_bound + 1)..old_upper_bound] = ''
-            elsif upper_bound > old_upper_bound and end_bytes > 0
-              # Preserve bytes at end of buffer
-              buffer_length = buffer.length
-              diff = upper_bound - old_upper_bound
-              buffer << ZERO_STRING * diff
-              buffer[(upper_bound + 1)..(buffer.length - 1)] = buffer[(old_upper_bound + 1)..(buffer_length - 1)]
-            end
-          else
-            byte_size = bit_size / 8
-            if value.length < byte_size
-              temp = value.ljust(byte_size, ZERO_STRING)
-            elsif value.length > byte_size
-              if overflow == :TRUNCATE
-                temp = value[0..(byte_size - 1)]
-              else
-                raise ArgumentError, "value of #{value.length} bytes does not fit into #{byte_size} bytes for data_type #{data_type}"
-              end
-            else
-              temp = value
-            end
-          end
-          buffer[lower_bound..upper_bound] = temp if bit_size != 0
-        else
-          raise ArgumentError, "bit_offset #{given_bit_offset} is not byte aligned for data_type #{data_type}"
-        end
-
-      when :INT, :UINT
-        ###################################
-        # Handle :INT and :UINT data types
-        ###################################
-
-        # Ensure value is the correct type
-        value = Integer(value)
-
-        if byte_aligned and (bit_size == 8 or bit_size == 16 or bit_size == 32 or bit_size == 64)
-          ###########################################################
-          # Handle byte-aligned 8, 16, 32, and 64 bit :INT and :UINT
-          ###########################################################
-
-          case bit_size
-          when 8
-            if data_type == :INT
-              value = self.check_overflow(value, -128, 127, 255, bit_size, data_type, overflow)
-            else
-              value = self.check_overflow(value, 0, 255, 255, bit_size, data_type, overflow)
-            end
-            buffer.setbyte(lower_bound, value % 256)
-
-          when 16
-            if data_type == :INT
-              value = self.check_overflow(value, -32768, 32767, 65535, bit_size, data_type, overflow)
-              if endianness == HOST_ENDIANNESS
-                buffer[lower_bound..upper_bound] = [value].pack(PACK_NATIVE_16_BIT_INT)
-              else # endianness != HOST_ENDIANNESS
-                buffer[lower_bound..upper_bound] = [value].pack(PACK_NATIVE_16_BIT_INT).reverse
-              end
-            else # data_type == :UINT
-              value = self.check_overflow(value, 0, 65535, 65535, bit_size, data_type, overflow)
-              if endianness == :BIG_ENDIAN
-                buffer[lower_bound..upper_bound] = [value].pack(PACK_BIG_ENDIAN_16_BIT_UINT)
-              else # endianness == :LITTLE_ENDIAN
-                buffer[lower_bound..upper_bound] = [value].pack(PACK_LITTLE_ENDIAN_16_BIT_UINT)
-              end
-            end
-
-          when 32
-            if data_type == :INT
-              # Note signed integers must allow up to the maximum unsigned value to support values given in hex
-              value = self.check_overflow(value, -2147483648, 2147483647, 4294967295, bit_size, data_type, overflow)
-              if endianness == HOST_ENDIANNESS
-                buffer[lower_bound..upper_bound] = [value].pack(PACK_NATIVE_32_BIT_INT)
-              else # endianness != HOST_ENDIANNESS
-                buffer[lower_bound..upper_bound] = [value].pack(PACK_NATIVE_32_BIT_INT).reverse
-              end
-            elsif data_type == :UINT
-              value = self.check_overflow(value, 0, 4294967295, 4294967295, bit_size, data_type, overflow)
-              if endianness == :BIG_ENDIAN
-                buffer[lower_bound..upper_bound] = [value].pack(PACK_BIG_ENDIAN_32_BIT_UINT)
-              else # endianness == :LITTLE_ENDIAN
-                buffer[lower_bound..upper_bound] = [value].pack(PACK_LITTLE_ENDIAN_32_BIT_UINT)
-              end
-            end
-
-          when 64
-            if data_type == :INT
-              # Note signed integers must allow up to the maximum unsigned value to support values given in hex
-              value = self.check_overflow(value, -9223372036854775808, 9223372036854775807, 18446744073709551615, bit_size, data_type, overflow)
-              if endianness == HOST_ENDIANNESS
-                buffer[lower_bound..upper_bound] = [value].pack(PACK_NATIVE_64_BIT_INT)
-              else # endianness != HOST_ENDIANNESS
-                buffer[lower_bound..upper_bound] = [value].pack(PACK_NATIVE_64_BIT_INT).reverse
-              end
-            elsif data_type == :UINT
-              value = self.check_overflow(value, 0, 18446744073709551615, 18446744073709551615, bit_size, data_type, overflow)
-              if endianness == HOST_ENDIANNESS
-                buffer[lower_bound..upper_bound] = [value].pack(PACK_NATIVE_64_BIT_UINT)
-              else # endianness != HOST_ENDIANNESS
-                buffer[lower_bound..upper_bound] = [value].pack(PACK_NATIVE_64_BIT_UINT).reverse
-              end
-            end
-          end
-
-        else
-          ##################################
-          # Handle :INT and :UINT Bitfields
-          ##################################
-
-          if data_type == :INT
-            # Note signed integers must allow up to the maximum unsigned value to support values given in hex
-            if bit_size > 1
-              min_value = -(2 ** (bit_size - 1))
-              max_value = -min_value - 1
-              hex_max_value = (2 ** bit_size) - 1
-            else
-              min_value = -1
-              max_value = 1
-              hex_max_value = 1
-            end
-            value = self.check_overflow(value, min_value, max_value, hex_max_value, bit_size, data_type, overflow)
-          else
-            max_value = (2 ** bit_size) - 1
-            value = self.check_overflow(value, 0, max_value, max_value, bit_size, data_type, overflow)
-          end
-
-          # Extract Existing Data
-          if endianness == :LITTLE_ENDIAN
-            # Bitoffset always refers to the most significant bit of a bitfield
-            num_bytes = (((bit_offset % 8) + bit_size - 1) / 8) + 1
-            upper_bound = bit_offset / 8
-            lower_bound = upper_bound - num_bytes + 1
-
-            if lower_bound < 0
-              raise ArgumentError, "LITTLE_ENDIAN bitfield with bit_offset #{given_bit_offset} and bit_size #{given_bit_size} is invalid"
-            end
-
-            temp_data = buffer[lower_bound..upper_bound].reverse
-          else
-            temp_data = buffer[lower_bound..upper_bound]
-          end
-
-          # Determine temp upper bound
-          temp_upper = upper_bound - lower_bound
-
-          # Determine Values needed to Handle Bitfield
-          start_bits = bit_offset % 8
-          start_mask = (0xFF << (8 - start_bits))
-          total_bits = (temp_upper + 1) * 8
-          end_bits = total_bits - start_bits - bit_size
-          end_mask = ~(0xFF << end_bits)
-
-          # Add in Start Bits
-          temp = temp_data.getbyte(0) & start_mask
-
-          # Adjust value to correct number of bits
-          temp_mask = (2 ** bit_size) - 1
-          temp_value = value.to_i & temp_mask
-
-          # Add in New Data
-          temp = (temp << (bit_size - (8 - start_bits))) + temp_value
-
-          # Add in Remainder of Existing Data
-          temp = (temp << end_bits) + (temp_data.getbyte(temp_upper) & end_mask)
-
-          # Extract into an array of bytes
-          temp_array = []
-          (0..temp_upper).each { temp_array.insert(0, (temp & 0xFF)); temp = temp >> 8 }
-
-          # Store into buffer
-          if endianness == :LITTLE_ENDIAN
-            buffer[lower_bound..upper_bound] = temp_array.pack(PACK_8_BIT_UINT_ARRAY).reverse
-          else
-            buffer[lower_bound..upper_bound] = temp_array.pack(PACK_8_BIT_UINT_ARRAY)
-          end
-        end
-
-      when :FLOAT
-        ##########################
-        # Handle :FLOAT data type
-        ##########################
-
-        # Ensure value is the correct type
-        value = Float(value)
-
-        if byte_aligned
-          case bit_size
-          when 32
-            if endianness == :BIG_ENDIAN
-              buffer[lower_bound..upper_bound] = [value].pack(PACK_BIG_ENDIAN_32_BIT_FLOAT)
-            else # endianness == :LITTLE_ENDIAN
-              buffer[lower_bound..upper_bound] = [value].pack(PACK_LITTLE_ENDIAN_32_BIT_FLOAT)
-            end
-
-          when 64
-            if endianness == :BIG_ENDIAN
-              buffer[lower_bound..upper_bound] = [value].pack(PACK_BIG_ENDIAN_64_BIT_FLOAT)
-            else # endianness == :LITTLE_ENDIAN
-              buffer[lower_bound..upper_bound] = [value].pack(PACK_LITTLE_ENDIAN_64_BIT_FLOAT)
-            end
-
-          else
-            raise ArgumentError, "bit_size is #{given_bit_size} but must be 32 or 64 for data_type #{data_type}"
-          end
-        else
-          raise ArgumentError, "bit_offset #{given_bit_offset} is not byte aligned for data_type #{data_type}"
-        end
-
-      else
-        ############################
-        # Handle Unknown data types
-        ############################
-
-        raise ArgumentError, "data_type #{data_type} is not recognized"
-      end
-
-      value
-    end # def self.write
+    # def self.write(value, bit_offset, bit_size, data_type, buffer, endianness, overflow)
 
     # Reads an array of binary data of any data type from a buffer
     #