NYI messages from LuaJIT

[kleisauke]

Hi John,

Running the soak.lua or the units tests with -jv outputs a few NYI messages. NYI messages indicate that this has not yet been implemented in the in LuaJIT's JIT compiler. See: http://wiki.luajit.org/NYI.

I think it will be good to solve these problems (where possible), especially in performance-critical code (such as voperation and gvalue and vobject).

NYI messages:

1. [NYI: bytecode 50 at voperation.lua:102] Fixed with: kleisauke@f898808.
2. [NYI: bytecode 51 at voperation.lua:118] Fixed with: Remove vips_argument_map callback #28
3. [NYI: bytecode 51 at voperation.lua:188] Fixed with: kleisauke@8c46fac.
4. [NYI: bytecode 72 at voperation.lua:214] Not fixable, we must iterate with pairs here.
5. [NYI: bytecode 50 at voperation.lua:265] Fixed with: kleisauke@c4d87b5.
6. [NYI: unsupported C type conversion at vobject.lua:89] Fixed with: kleisauke@f898808
7. [NYI: bytecode 51 at Image_methods.lua:771] Not fixable, we can't circumvent the vargargs closure.
8. [NYI: unsupported C type conversion at Image_methods.lua:448] Fixed with: kleisauke@f898808.
9. [NYI: unsupported C type conversion at gvalue.lua:134], see footnote 1.
10. [NYI: bytecode 51 at voperation.lua:91] Fixed with: kleisauke@6dd0e10

Footnotes:

1. I think there might be ints in the array (because I don't see these messages for the array_int_type gtype).

I'll make a PR in the course of next week.

[kleisauke]

Update:
Number 3 and 5 are resolved with kleisauke@8c46fac and kleisauke@c4d87b5. Number 4 and 7 are not fixable. I'm not sure if we can fix number 1 and 6, I will investigate that further.

Also, I've removed vips.gvalue.new() and switched to the default constructor (vips.gvalue()) with kleisauke@0d30c81.

[jcupitt]

Huh I didn't know about -jv, that's useful.

[kleisauke]

Update:
Number 10 is resolved with kleisauke@6dd0e10. It seems that number 1, 6 and 8 can be solved with:

diff --git a/vips/Image_methods.lua b/vips/Image_methods.lua
index 1111111..2222222 100644
--- a/vips/Image_methods.lua
+++ b/vips/Image_methods.lua
@@ -445,10 +445,11 @@ function Image_method:get(name)
 end
 
 function Image_method:set_type(gtype, name, value)
-    local gv = gvalue.new()
-    gv:init(gtype)
-    gv:set(value)
-    vips_lib.vips_image_set(self.vimage, name, gv)
+    local pgv = gvalue.newp()
+    pgv[0]:init(gtype)
+    pgv[0]:set(value)
+    vips_lib.vips_image_set(self.vimage, name, pgv)
+    gobject_lib.g_value_unset(pgv[0])
 end
 
 function Image_method:set(name, value)

diff --git a/vips/vobject.lua b/vips/vobject.lua
index 1111111..2222222 100644
--- a/vips/vobject.lua
+++ b/vips/vobject.lua
@@ -86,10 +86,11 @@ local vobject_mt = {
                 return false
             end
 
-            local gv = gvalue.new()
-            gv:init(gtype)
-            gv:set(value)
-            gobject_lib.g_object_set_property(self, name, gv)
+            local pgv = gvalue.newp()
+            pgv[0]:init(gtype)
+            pgv[0]:set(value)
+            gobject_lib.g_object_set_property(self, name, pgv)
+            gobject_lib.g_value_unset(pgv[0])
 
             return true
         end

but this is awful (the __gc function from the GValue class is then no longer used).