From: John L. <jla...@gm...> - 2013-08-16 02:16:14
|
On Thu, Aug 15, 2013 at 8:59 AM, Andreas Falkenhahn <an...@fa...>wrote: > Hi, > > I noticed that the second method of wxImage::GetAlpha() is missing. AFAICS > wxLua > only has the first one. So I've quickly adapted this for wxLua. Here's the > code in > case you want to merge this with the official branch: > Thanks! I really appreciate these patches. > I'm not sure if the wxluatype_NULL return is consistent with wxLua's > standard. > What is the function expected to return if wxImage::GetAlpha() returns > NULL? > Currently it returns wx.NULL but of course it could also be made to return > NIL, > an empty string, or even nothing here.... not sure how wxLua is supposed to > behave in this case. > > I made it return nil since that's probably more natural. Regards, John |