ffi.rb in libnotify-0.8.4

- old
+ new

@@ -2,18 +2,21 @@
   # Raw FFI bindings.
   module FFI
     require 'ffi'
     extend ::FFI::Library
 
+    enum :urgency, [ :low, :normal, :critical ]
+
+    # Load libnotify library and attach functions from C to Ruby via FFI.
     def self.included(base)
       load_libs
       attach_functions!
     rescue LoadError => e
       warn e.message
     end
 
-    def self.load_libs
+    def self.load_libs # :nodoc:
       libnotify_libs = %w[libnotify libnotify.so.4 libnotify.so.3 libnotify.so.2 libnotify.so.1 libnotify.so]
 
       # Workaround for "half-linked" libnotify.so. Does not work on rubinius (no ffi_lib_flags there)!
       # See: https://bugzilla.redhat.com/show_bug.cgi?id=626852
       ffi_lib_flags :lazy, :local, :global if respond_to?(:ffi_lib_flags)
@@ -23,30 +26,24 @@
       ffi_lib %w[libgtk-x11-2.0 libgtk-x11-2.0.so.0 libgtk-x11-2.0.so libgtk-3 libgtk-3.so.0 libgtk-3.so],
               %w[libgtkmm-2.4 libgtkmm-2.4.so.1 libgtkmm-2.4.so libgtkmm-3.0 libgtkmm-3.0.so.1 libgtkmm-3.0.so],
               libnotify_libs
     end
 
-    URGENCY = [ :low, :normal, :critical ]
-
-    def self.attach_functions!
+    def self.attach_functions! # :nodoc:
       attach_function :notify_init,                         [:string],                              :bool
       attach_function :notify_uninit,                       [],                                     :void
       attach_function :notify_notification_new,             [:string, :string, :string, :pointer],  :pointer
       attach_function :notify_notification_update,          [:pointer, :string, :string, :string, :pointer],  :pointer
-      attach_function :notify_notification_set_urgency,     [:pointer, :int],                       :void
+      attach_function :notify_notification_set_urgency,     [:pointer, :urgency],                   :void
       attach_function :notify_notification_set_timeout,     [:pointer, :long],                      :void
       attach_function :notify_notification_set_hint_string, [:pointer, :string, :string],           :void
       attach_function :notify_notification_set_hint_uint32, [:pointer, :string, :int],              :void
       attach_function :notify_notification_clear_hints,     [:pointer],                             :void
       attach_function :notify_notification_show,            [:pointer, :pointer],                   :bool
       attach_function :notify_notification_close,           [:pointer, :pointer],                   :bool
     end
 
-    def lookup_urgency(urgency)
-      URGENCY.index(urgency)
-    end
-
-    def method_missing(method, *args, &block)
+    def method_missing(method, *args, &block) # :nodoc:
       if method.to_s =~ /^notify_/
         warn "libnotify.so not found!"
       end
 
       super