ext/win32ole/win32ole.c: some refactoring.

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@24898 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
suke 2009-09-13 10:42:24 +00:00
parent bc832b8e5b
commit 1c75c0e28c
2 changed files with 10 additions and 48 deletions

View File

@ -1,3 +1,7 @@
Sun Sep 13 19:39:59 2009 Masaki Suketa <masaki.suketa@nifty.ne.jp>
* ext/win32ole/win32ole.c: some refactoring.
Sun Sep 13 19:38:34 2009 Nobuyoshi Nakada <nobu@ruby-lang.org> Sun Sep 13 19:38:34 2009 Nobuyoshi Nakada <nobu@ruby-lang.org>
* ruby.c (load_file_internal): no need to define DATA when error. * ruby.c (load_file_internal): no need to define DATA when error.

View File

@ -130,7 +130,7 @@ const IID IID_IMultiLanguage2 = {0xDCCFC164, 0x2B38, 0x11d2, {0xB7, 0xEC, 0x00,
#define WC2VSTR(x) ole_wc2vstr((x), TRUE) #define WC2VSTR(x) ole_wc2vstr((x), TRUE)
#define WIN32OLE_VERSION "1.4.3" #define WIN32OLE_VERSION "1.4.4"
typedef HRESULT (STDAPICALLTYPE FNCOCREATEINSTANCEEX) typedef HRESULT (STDAPICALLTYPE FNCOCREATEINSTANCEEX)
(REFCLSID, IUnknown*, DWORD, COSERVERINFO*, DWORD, MULTI_QI*); (REFCLSID, IUnknown*, DWORD, COSERVERINFO*, DWORD, MULTI_QI*);
@ -387,8 +387,6 @@ static VALUE fole_put_methods(VALUE self);
static VALUE fole_func_methods(VALUE self); static VALUE fole_func_methods(VALUE self);
static VALUE ole_type_from_itypeinfo(ITypeInfo *pTypeInfo); static VALUE ole_type_from_itypeinfo(ITypeInfo *pTypeInfo);
static VALUE fole_type(VALUE self); static VALUE fole_type(VALUE self);
static VALUE make_oletypelib_obj(VALUE guid, VALUE major_version, VALUE minor_version);
static VALUE ole_typelib_from_itypelib(ITypeLib *pTypeLib);
static VALUE ole_typelib_from_itypeinfo(ITypeInfo *pTypeInfo); static VALUE ole_typelib_from_itypeinfo(ITypeInfo *pTypeInfo);
static VALUE fole_typelib(VALUE self); static VALUE fole_typelib(VALUE self);
static VALUE fole_query_interface(VALUE self, VALUE str_iid); static VALUE fole_query_interface(VALUE self, VALUE str_iid);
@ -4329,44 +4327,6 @@ fole_type(VALUE self)
return type; return type;
} }
static VALUE
make_oletypelib_obj(VALUE guid, VALUE major_version, VALUE minor_version)
{
VALUE args = rb_ary_new();
rb_ary_push(args, guid);
rb_ary_push(args, major_version);
rb_ary_push(args, minor_version);
return rb_apply(cWIN32OLE_TYPELIB, rb_intern("new"), args);
}
static VALUE
ole_typelib_from_itypelib(ITypeLib *pTypeLib)
{
TLIBATTR *pTLibAttr;
OLECHAR bstr[80];
VALUE guid = Qnil;
VALUE major;
VALUE minor;
int len = 0;
HRESULT hr = S_OK;
hr = pTypeLib->lpVtbl->GetLibAttr(pTypeLib, &pTLibAttr);
if (FAILED(hr)) {
return Qnil;
}
len = StringFromGUID2(&pTLibAttr->guid, bstr, sizeof(bstr)/sizeof(OLECHAR));
if (len > 3) {
guid = ole_wc2vstr(bstr, FALSE);
}
major = INT2NUM(pTLibAttr->wMajorVerNum);
minor = INT2NUM(pTLibAttr->wMinorVerNum);
pTypeLib->lpVtbl->ReleaseTLibAttr(pTypeLib, pTLibAttr);
if (guid == Qnil) {
return Qnil;
}
return make_oletypelib_obj(guid, major, minor);
}
static VALUE static VALUE
ole_typelib_from_itypeinfo(ITypeInfo *pTypeInfo) ole_typelib_from_itypeinfo(ITypeInfo *pTypeInfo)
{ {
@ -4379,8 +4339,8 @@ ole_typelib_from_itypeinfo(ITypeInfo *pTypeInfo)
if(FAILED(hr)) { if(FAILED(hr)) {
return Qnil; return Qnil;
} }
retval = ole_typelib_from_itypelib(pTypeLib); retval = rb_funcall(cWIN32OLE_TYPELIB, rb_intern("allocate"), 0);
OLE_RELEASE(pTypeLib); oletypelib_set_member(retval, pTypeLib);
return retval; return retval;
} }
@ -5225,12 +5185,11 @@ foletypelib_name(VALUE self)
hr = pTypeLib->lpVtbl->GetDocumentation(pTypeLib, -1, hr = pTypeLib->lpVtbl->GetDocumentation(pTypeLib, -1,
&bstr, &bstr2, NULL, NULL); &bstr, &bstr2, NULL, NULL);
if (SUCCEEDED(hr)) { if (FAILED(hr)) {
name = WC2VSTR(bstr2);
return rb_enc_str_new(StringValuePtr(name), strlen(StringValuePtr(name)), cWIN32OLE_enc);
} else {
ole_raise(hr, eWIN32OLERuntimeError, "failed to get name from ITypeLib"); ole_raise(hr, eWIN32OLERuntimeError, "failed to get name from ITypeLib");
} }
name = WC2VSTR(bstr2);
return rb_enc_str_new(StringValuePtr(name), strlen(StringValuePtr(name)), cWIN32OLE_enc);
} }
/* /*
@ -5384,7 +5343,6 @@ static VALUE
foletypelib_path(VALUE self) foletypelib_path(VALUE self)
{ {
TLIBATTR *pTLibAttr; TLIBATTR *pTLibAttr;
int len = 0;
HRESULT hr = S_OK; HRESULT hr = S_OK;
BSTR bstr; BSTR bstr;
LCID lcid = cWIN32OLE_lcid; LCID lcid = cWIN32OLE_lcid;