You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucy.apache.org by ma...@apache.org on 2014/11/17 18:33:36 UTC
[19/19] lucy-clownfish git commit: Use `finalize` for finalizers.
Use `finalize` for finalizers.
* Be consistent about defaulting to non-exported name.
* Finalizers may do more than decref, so RunDecRef is misleading.
Project: http://git-wip-us.apache.org/repos/asf/lucy-clownfish/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucy-clownfish/commit/af86afbe
Tree: http://git-wip-us.apache.org/repos/asf/lucy-clownfish/tree/af86afbe
Diff: http://git-wip-us.apache.org/repos/asf/lucy-clownfish/diff/af86afbe
Branch: refs/heads/go_bindings_1
Commit: af86afbef684c231f002ff6e2316ea14e6e520cc
Parents: 4c8fa47
Author: Marvin Humphrey <ma...@rectangular.com>
Authored: Sun Nov 16 21:18:36 2014 -0800
Committer: Marvin Humphrey <ma...@rectangular.com>
Committed: Sun Nov 16 21:18:36 2014 -0800
----------------------------------------------------------------------
compiler/go/cfc/cfc.go | 12 ++++++------
runtime/go/clownfish/clownfish.go | 10 +++++-----
2 files changed, 11 insertions(+), 11 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/lucy-clownfish/blob/af86afbe/compiler/go/cfc/cfc.go
----------------------------------------------------------------------
diff --git a/compiler/go/cfc/cfc.go b/compiler/go/cfc/cfc.go
index 92f0094..7c12a6c 100644
--- a/compiler/go/cfc/cfc.go
+++ b/compiler/go/cfc/cfc.go
@@ -84,11 +84,11 @@ func NewHierarchy(dest string) *Hierarchy {
defer C.free(unsafe.Pointer(destCString))
obj := &Hierarchy{C.CFCHierarchy_new(destCString)}
obj.AddIncludeDir(mainIncDir)
- runtime.SetFinalizer(obj, (*Hierarchy).RunDecRef)
+ runtime.SetFinalizer(obj, (*Hierarchy).finalize)
return obj
}
-func (obj *Hierarchy) RunDecRef() {
+func (obj *Hierarchy) finalize() {
C.CFCBase_decref((*C.CFCBase)(unsafe.Pointer(obj.ref)))
}
@@ -120,11 +120,11 @@ func NewBindCore(hierarchy *Hierarchy, header string, footer string) *BindCore {
obj := &BindCore{
C.CFCBindCore_new(hierarchy.ref, headerCString, footerCString),
}
- runtime.SetFinalizer(obj, (*BindCore).RunDecRef)
+ runtime.SetFinalizer(obj, (*BindCore).finalize)
return obj
}
-func (obj *BindCore) RunDecRef() {
+func (obj *BindCore) finalize() {
C.CFCBase_decref((*C.CFCBase)(unsafe.Pointer(obj.ref)))
}
@@ -144,11 +144,11 @@ func NewBindC(hierarchy *Hierarchy, header string, footer string) *BindC {
obj := &BindC{
C.CFCC_new(hierarchy.ref, headerCString, footerCString),
}
- runtime.SetFinalizer(obj, (*BindC).RunDecRef)
+ runtime.SetFinalizer(obj, (*BindC).finalize)
return obj
}
-func (obj *BindC) RunDecRef() {
+func (obj *BindC) finalize() {
C.CFCBase_decref((*C.CFCBase)(unsafe.Pointer(obj.ref)))
}
http://git-wip-us.apache.org/repos/asf/lucy-clownfish/blob/af86afbe/runtime/go/clownfish/clownfish.go
----------------------------------------------------------------------
diff --git a/runtime/go/clownfish/clownfish.go b/runtime/go/clownfish/clownfish.go
index 1ad7195..c8860c1 100644
--- a/runtime/go/clownfish/clownfish.go
+++ b/runtime/go/clownfish/clownfish.go
@@ -110,11 +110,11 @@ func NewString(goString string) *String {
obj := &String{
C.cfish_Str_new_steal_utf8(str, len),
}
- runtime.SetFinalizer(obj, (*String).callDecRef)
+ runtime.SetFinalizer(obj, (*String).finalize)
return obj
}
-func (obj *String) callDecRef() {
+func (obj *String) finalize() {
C.CFISH_Str_Dec_RefCount(obj.ref)
obj.ref = nil
}
@@ -143,11 +143,11 @@ func NewError(mess string) error {
len := C.size_t(len(mess))
messC := C.cfish_Str_new_steal_utf8(str, len)
obj := &Err{C.cfish_Err_new(messC)}
- runtime.SetFinalizer(obj, (*Err).callDecRef)
+ runtime.SetFinalizer(obj, (*Err).finalize)
return obj
}
-func (obj *Err) callDecRef() {
+func (obj *Err) finalize() {
C.CFISH_Err_Dec_RefCount(obj.ref)
obj.ref = nil
}
@@ -160,7 +160,7 @@ func (obj *Err) Error() string {
func GoCfish_PanicErr_internal(cfErr *C.cfish_Err) {
goErr := &Err{cfErr}
C.CFISH_Err_Inc_RefCount(cfErr)
- runtime.SetFinalizer(goErr, (*Err).callDecRef)
+ runtime.SetFinalizer(goErr, (*Err).finalize)
panic(goErr)
}