pure-x11 0.0.5 → 0.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0ecd289a7ec4a0a7f44bba1d05a437774ca6ed3d4c403b572b28a8691ef2a950
-  data.tar.gz: ac9460f0c0db0635e33358550176802119db88c9bb4dee097807f21c970f68ba
+  metadata.gz: 927c52f35913e7d8bc466563bb5d238919c4a43d6a9f9fb1be8a6df4735cc5fb
+  data.tar.gz: 3b92ed556881c32486ed189c9de27fdd373988379118fefdbd6f40ff6268c0e7
 SHA512:
-  metadata.gz: db23062260daabe75bfe5a9f419772be329023777c405ae7bb3a08125b9942f63ec8d635c1c11869648272d2c3ecad16a432fbca2d4d584d80ad2dbf15250d5a
-  data.tar.gz: f303d01e2c4b43fa2c5d44af4e469288684887c3efe71ee644275f233d584b9e342f9ebd80374632afe1199ad0d4182cebb515a95a7408b79aa36d239c9c9ffb
+  metadata.gz: 580ab719f597acaecfa467389f26526dc50c1d25cacb14dbda9d63f61e3cb63f3665fd010408fc093501893a229a4b6a6f314734f6cf35786c58d965d0ec51cd
+  data.tar.gz: 211b56d642b773c29769e9620c746a657596b6314e1d5a89487ace79bf29c6f01f4675ec2456a8668b8723912b2c81f2c95309dcdaf9eac459682150b884f726

data/.gitignore

@@ -1,3 +1,5 @@
+vendor/
+
 # rcov generated
 coverage
 

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    pure-x11 (0.0.4)
+    pure-x11 (0.0.6)
 
 GEM
   remote: https://rubygems.org/

data/example/test.rb

@@ -143,7 +143,7 @@ def render_glyph(display, wid, x,y, ch)
 #p img
 #  p ch
  display.put_image(
-    X11::Form::ZPixmap, wid, $gc2,
+    :ZPixmap, wid, $gc2,
     mtx.min_width,mtx.min_height,
     x, y - mtx.y_offset, 0, depth, data
   )
@@ -158,12 +158,12 @@ def render_str(display, wid, x,y, str)
 end
 
 def redraw(dpy, wid, gc)
-  dpy.poly_fill_rectangle(wid, gc, [X11::Form::Rectangle.new(20,20, 60, 80)])
+  dpy.poly_fill_rectangle(wid, gc, [20,20, 60, 80])
   dpy.clear_area(false, wid, 30, 30, 5, 5)
   dpy.image_text16(wid, $gc2, 30, 70, "ΑΒΓΔΕΖΗΘΙΚΛΜΝΞΟΠΡΣΤΥΦΧΨΩ")
   #"\u25f0\u25ef Hello World")
   dpy.put_image(
-    X11::Form::ZPixmap, wid, $gc2,
+    :ZPixmap, wid, $gc2,
     $png.width, $png.height, 80, 120, 0, 24, $data
   )
   render_str(dpy, wid, 30,90, 'HelloWorld')

data/lib/X11/display.rb

@@ -116,7 +116,7 @@ module X11
       # FIXME: 30: SelectionRequest
       # FIXME: 31: SelectionNotify
       # FIXME: 32: ColormapNotify
-      when 33 then return Form::ClientMessage.from_packet(StringIO.new(data))
+      when 33 then return Form::ClientMessage.from_packet(io)
       # FIXME: 34: MappingNotify
       else
         STDERR.puts "FIXME: Event: #{type}"
@@ -165,11 +165,7 @@ module X11
     end
     
     def write_request ob
-      #p data
-      #p [:write_request, @requestseq, ob.class]
       data = ob.to_packet if ob.respond_to?(:to_packet)
-      #p [:AddGlyph,data] if ob.is_a?(X11::Form::XRenderAddGlyphs)
-      #p [ob.request_length.to_i*4, data.size]
       raise "BAD LENGTH for #{ob.inspect} (#{ob.request_length.to_i*4} ! #{data.size} " if ob.request_length && ob.request_length.to_i*4 != data.size
       write_raw_packet(data)
     end
@@ -252,24 +248,21 @@ module X11
       values = values.sort_by{_1[0]}
       mask =   values.inject(0) {|acc,v| (acc | v[0]) }
       values = values.map{_1[1]}
-      write_request(
-        X11::Form::ChangeWindowAttributes.new(wid, mask, values)
-      )
+      write_request(Form::ChangeWindowAttributes.new(wid, mask, values))
     end
 
     def select_input(w, events) = change_window_attributes(w, values: {Form::CWEventMask => events})
 
     def atom(name)
+      return name if name.is_a?(Integer) # Allow atom(atom_integer_or_symbol)
       name = name.to_sym
       intern_atom(false, name) if !@atoms[name]
       @atoms[name]
     end
 
     def query_extension(name)
-      r = write_sync(X11::Form::QueryExtension.new(name), X11::Form::QueryExtensionReply)
-      @extensions[name] = {
-        major: r.major_opcode
-      }
+      r = write_sync(Form::QueryExtension.new(name), Form::QueryExtensionReply)
+      @extensions[name] = { major: r.major_opcode }
       r
     end
 
@@ -282,38 +275,33 @@ module X11
     end
     
     def intern_atom(flag, name)
-      reply = write_sync(X11::Form::InternAtom.new(flag, name.to_s),
-      X11::Form::InternAtomReply)
+      reply = write_sync(Form::InternAtom.new(flag, name.to_s),Form::InternAtomReply)
       if reply
         @atoms[name.to_sym] = reply.atom
       end
     end
 
-    def get_atom_name(atom)
-      reply = write_sync(X11::Form::GetAtomName.new(atom), X11::Form::AtomName)
-      reply&.name
-    end
-
-    def destroy_window(window) = write_request(X11::Form::DestroyWindow.new(window))
-    def get_geometry(drawable) = write_sync(X11::Form::GetGeometry.new(drawable), X11::Form::Geometry)
+    def get_atom_name(atom)    = write_sync(Form::GetAtomName.new(atom), Form::AtomName)&.name
+    def destroy_window(window) = write_request(Form::DestroyWindow.new(window))
+    def get_geometry(drawable) = write_sync(Form::GetGeometry.new(drawable), Form::Geometry)
     
     def get_keyboard_mapping(min_keycode=display_info.min_keycode, count= display_info.max_keycode - min_keycode)
-      write_sync(X11::Form::GetKeyboardMapping.new(min_keycode, count), X11::Form::GetKeyboardMappingReply)
+      write_sync(Form::GetKeyboardMapping.new(min_keycode, count), Form::GetKeyboardMappingReply)
     end
 
     def create_colormap(alloc, window, visual)
       mid = new_id
-      write_request(X11::Form::CreateColormap.new(alloc, mid, window, visual))
+      write_request(Form::CreateColormap.new(alloc, mid, window, visual))
       mid
     end
 
     def get_property(window, property, type, offset: 0, length: 4, delete: false)
-      property = atom(property) if !property.is_a?(Integer)
+      property = atom(property)
       type     = atom_enum(type)
       
-      result = write_sync(X11::Form::GetProperty.new(
+      result = write_sync(Form::GetProperty.new(
         delete, window, property, type, offset, length
-      ), X11::Form::Property)
+      ), Form::Property)
 
       if result && result.format != 0
         case result.format
@@ -335,25 +323,22 @@ module X11
 
       mode = open_enum(mode, {replace: 0, prepend: 1, append: 2})
       type = atom_enum(type)
-      write_request(X11::Form::ChangeProperty.new(mode, window, property, type, format, data))
+      write_request(Form::ChangeProperty.new(mode, window, property, type, format, data))
     end
 
-    def list_fonts(*args)
-      write_sync(X11::Form::ListFonts.new(*args),
-        X11::Form::ListFontsReply)
-    end
-
-    def open_font(*args)    = write_request(X11::Form::OpenFont.new(*args))
-    def change_gc(*args)    = write_request(X11::Form::ChangeGC.new(*args))
-    def change_save_set(...)= write_request(X11::Form::ChangeSaveSet.new(...))
+    def list_fonts(...)     = write_sync(Form::ListFonts.new(...), Form::ListFontsReply)
+    def open_font(...)      = write_request(Form::OpenFont.new(...))
+    def change_gc(...)      = write_request(Form::ChangeGC.new(...))
+    def change_save_set(...)= write_request(Form::ChangeSaveSet.new(...))
+      
     def reparent_window(window, parent, x, y, save: true)
       # You so almost always want this that it should've been a single request
       change_save_set(0, window) if save
-      write_request(X11::Form::ReparentWindow.new(window, parent, x,y))
+      write_request(Form::ReparentWindow.new(window, parent, x,y))
     end
     
-    def map_window(*args)   = write_request(X11::Form::MapWindow.new(*args))
-    def unmap_window(*args) = write_request(X11::Form::UnmapWindow.new(*args))
+    def map_window(...)   = write_request(Form::MapWindow.new(...))
+    def unmap_window(...) = write_request(Form::UnmapWindow.new(...))
 
     def u8(*args)  = args.pack("c*")
     def u16(*args) = args.pack("v*")
@@ -375,9 +360,9 @@ module X11
       time      = open_enum(time,      {current_time: 0, now: 0})
       write_packet(u8(42,revert_to), u16(3), window(focus), u32(time))
     end
-      
+
     def grab_key(owner_events, grab_window, modifiers, keycode, pointer_mode, keyboard_mode)
-      write_request(X11::Form::GrabKey.new(
+      write_request(Form::GrabKey.new(
         owner_events,
         grab_window,
         modifiers,
@@ -389,7 +374,7 @@ module X11
 
     def grab_button(owner_events, grab_window, event_mask, pointer_mode,
       keyboard_mode, confine_to, cursor, button, modifiers)
-      write_request(X11::Form::GrabButton.new(
+      write_request(Form::GrabButton.new(
         owner_events, grab_window, event_mask,
         pointer_mode == :async ? 1 : 0,
         keyboard_mode == :async ? 1 : 0,
@@ -397,41 +382,27 @@ module X11
       )
     end
 
+    def set_value(values, mask, x)
+      if x
+        values << x
+        mask
+      else
+        0
+      end
+    end
+      
     def configure_window(window, x: nil, y: nil, width: nil, height: nil,
       border_width: nil, sibling: nil, stack_mode: nil)
 
       mask = 0
       values = []
 
-      if x
-        mask |= 0x001
-        values << x
-      end
-
-      if y
-        mask |= 0x002
-        values << y
-      end
-
-      if width
-        mask |= 0x004
-        values << width
-      end
-
-      if height
-        mask |= 0x008
-        values << height
-      end
-
-      if border_width
-        mask |= 0x010
-        values << border_width
-      end
-
-      if sibling
-        mask |= 0x020
-        values << sibling
-      end
+      mask |= set_value(values, 0x001, x)
+      mask |= set_value(values, 0x002, y)
+      mask |= set_value(values, 0x004, width)
+      mask |= set_value(values, 0x008, height)
+      mask |= set_value(values, 0x010, border_width)
+      mask |= set_value(values, 0x020, sibling)
 
       if stack_mode
         mask |= 0x040
@@ -455,15 +426,8 @@ module X11
       # FIXME:
       # The rest can be found here:
       # https://tronche.com/gui/x/xlib/GC/manipulating.html#XGCValues
-      if foreground
-        mask |= 0x04
-        args << foreground
-      end
-      if background
-        mask |= 0x08
-        args << background
-      end
-
+      mask |= set_value(args, 0x04, foreground)
+      mask |= set_value(args, 0x08, background)
       
       gc = new_id
       write_request(X11::Form::CreateGC.new(gc, window, mask, args))
@@ -475,12 +439,13 @@ module X11
     def copy_area(*args)   = write_request(X11::Form::CopyArea.new(*args))
     def image_text8(*args) = write_request(X11::Form::ImageText8.new(*args))
     def image_text16(*args)= write_request(X11::Form::ImageText16.new(*args))
-    def poly_fill_rectangle(*args) = write_request(X11::Form::PolyFillRectangle.new(*args))
+    def poly_fill_rectangle(wid, gc, *rects)
+      rects = rects.map{|r| r.is_a?(Array) ? Form::Rectangle.new(*r) : r}
+      write_request(X11::Form::PolyFillRectangle.new(wid, gc, rects))
+    end
 
     def create_pixmap(depth, drawable, w,h)
-      pid = new_id
-      write_request(X11::Form::CreatePixmap.new(depth, pid, drawable, w,h))
-      pid
+      new_id.tap{|pid| write_request(Form::CreatePixmap.new(depth, pid, drawable, w,h)) }
     end
 
     # XRender
@@ -506,8 +471,8 @@ module X11
 
     def render_query_pict_formats
       @render_formats ||= write_sync(
-        X11::Form::XRenderQueryPictFormats.new(render_opcode),
-        X11::Form::XRenderQueryPictFormatsReply
+        Form::XRenderQueryPictFormats.new(render_opcode),
+        Form::XRenderQueryPictFormatsReply
       )
     end
 
@@ -528,26 +493,26 @@ module X11
       case sym
       when :a8
         @a8 ||= formats.formats.find do |f|
-          f.type == 1 &&
+          f.type  == 1 &&
           f.depth == 8 &&
           f.direct.alpha_mask == 255
         end
       when :rgb24
         @rgb24 ||= formats.formats.find do |f|
-          f.type == 1 &&
-          f.depth == 24 &&
-          f.direct.red == 16 &&
+          f.type         == 1 &&
+          f.depth        == 24 &&
+          f.direct.red   == 16 &&
           f.direct.green == 8 &&
-          f.direct.blue == 0
+          f.direct.blue  == 0
         end
       when :argb24
         @argb24 ||= formats.formats.find do |f|
-          f.type == 1 &&
-          f.depth == 32 &&
+          f.type         == 1 &&
+          f.depth        == 32 &&
           f.direct.alpha == 24 &&
-          f.direct.red == 16 &&
+          f.direct.red   == 16 &&
           f.direct.green == 8 &&
-          f.direct.blue == 0
+          f.direct.blue  == 0
         end
       else
         raise "Unsupported format (a4/a1 by omission)"

data/lib/X11/form.rb

@@ -65,13 +65,6 @@ module X11
 
             if value.is_a?(BaseForm)
               v = value.to_packet
-            elsif value.is_a?(Symbol)
-              #if !@atoms[value]
-              #  reply = write_sync(X11::Forms::InternAtom.new(false, value.to_s), X11::Forms::InternAtomReply)
-              #  @
-              #end
-              #value = @atoms[value]
-              raise "FIXME"
             else
               #p [s,value]
               v = s.type_klass.pack(value)

data/lib/X11/type.rb

@@ -1,54 +1,50 @@
 # This module is used for encoding Ruby Objects to binary
 # data. The types Int8, Int16, etc. are data-types defined
-# in the X11 protocol. We wrap each data-type in a lambda expression
-# which gets evaluated when a packet is created.
+# in the X11 protocol.
 
 module X11
   module Type
 
-    def self.define(type, directive, bytesize)
-      eval %{
-        class X11::Type::#{type}
-          def self.pack(x)
-            [x].pack(\"#{directive}\")
-          end
-
-          def self.unpack(x)
-            x.unpack(\"#{directive}\").first
-          end
+    class BaseType
+      @directive = nil
+      @bytesize = nil
 
-          def self.size
-            #{bytesize}
-          end
-
-          def self.from_packet(sock)
-            r = sock.read(size)
-            r ? unpack(r) : nil
+      def self.config(d,b) = (@directive, @bytesize = d,b)
+        
+      def self.pack(x)
+        if x.is_a?(Symbol)
+          if (t = X11::Form.const_get(x)) && t.is_a?(Numeric)
+            x = t
           end
         end
-      }
-    end
+        [x].pack(@directive)
+      rescue TypeError => e
+        raise "Expected #{self.name}, got #{x.class} (value: #{x})"
+      end
 
+      def self.unpack(x) = x.nil? ? nil  : x.unpack1(@directive)
+      def self.size = @bytesize
+      def self.from_packet(sock) = unpack(sock.read(size))
+    end
+    
     # Primitive Types
-    define "Int8", "c", 1
-    define "Int16", "s", 2
-    define "Int32", "l", 4
-    define "Uint8", "C", 1
-    define "Uint16", "S", 2
-    define "Uint32", "L", 4
+    class Int8   < BaseType; config("c",1); end
+    class Int16  < BaseType; config("s",2); end
+    class Int32  < BaseType; config("l",4); end
+    class Uint8  < BaseType; config("C",1); end
+    class Uint16 < BaseType; config("S",2); end
+    class Uint32 < BaseType; config("L",4); end
     
-    define "Message", "c*", 20
-
     class Message
-      def self.pack(x) = x.b
+      def self.pack(x)   = x.b
       def self.unpack(x) = x.b
-      def self.size = 20
+      def self.size      = 20
       def self.from_packet(sock) = sock.read(2).b
     end
 
     class String8
       def self.pack(x)
-        x.force_encoding("ASCII-8BIT") + "\x00"*(-x.length & 3)
+        x.b + "\x00"*(-x.length & 3)
       end
 
       def self.unpack(socket, size)
@@ -62,7 +58,7 @@ module X11
 
     class String16
       def self.pack(x)
-        x.encode("UTF-16BE").force_encoding("ASCII-8BIT") + "\x00\x00"*(-x.length & 1)
+        x.encode("UTF-16BE").b + "\x00\x00"*(-x.length & 1)
       end
 
       def self.unpack(socket, size)
@@ -75,28 +71,14 @@ module X11
 
 
     class String8Unpadded
-      def self.pack(x)
-        x
-      end
-
-
-      def self.unpack(socket, size)
-        val = socket.read(size)
-      end
+      def self.pack(x) = x
+      def self.unpack(socket, size) = socket.read(size)
     end
-    
+      
     class Bool
-      def self.pack(x)
-        x ? "\x01" : "\x00"
-      end
-
-      def self.unpack(str)
-        str[0] == "\x01"
-      end
-
-      def self.size
-        1
-      end
+      def self.pack(x)     = (x ? "\x01" : "\x00")
+      def self.unpack(str) = (str[0] == "\x01")
+      def self.size = 1
     end
     
     KeyCode      = Uint8

data/lib/X11/version.rb

@@ -1,3 +1,3 @@
 module X11
-  VERSION = "0.0.5"
+  VERSION = "0.0.6"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pure-x11
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.6
 platform: ruby
 authors:
 - Vidar Hokstad
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-12-16 00:00:00.000000000 Z
+date: 2023-12-28 00:00:00.000000000 Z
 dependencies: []
 description: Pure Ruby X11 bindings
 email: