diff options
| author | Pietro Gagliardi <[email protected]> | 2014-08-11 17:18:02 -0400 |
|---|---|---|
| committer | Pietro Gagliardi <[email protected]> | 2014-08-11 17:18:02 -0400 |
| commit | 67adbd43a1bbf35fdc3963b34ae5631758006fee (patch) | |
| tree | a1f32908e14e818b427599924d08c81db1174305 /redo/table_darwin.go | |
| parent | 2c7c8b3a6b522cf64c66d606dab6c7a6fb75e2e3 (diff) | |
Revert "Settled race condition TODOs across all platforms... as an unfortunate case :("
Things broke when introducing this after the new GC... let's see if this broke under the new GC or not.
This reverts commit b198a4dfec53a15eac5cdca9fe9d91c145a0119c.
Diffstat (limited to 'redo/table_darwin.go')
| -rw-r--r-- | redo/table_darwin.go | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/redo/table_darwin.go b/redo/table_darwin.go index 95344c1..c80dbdc 100644 --- a/redo/table_darwin.go +++ b/redo/table_darwin.go @@ -36,15 +36,11 @@ func finishNewTable(b *tablebase, ty reflect.Type) Table { func (t *table) Unlock() { t.unlock() - // there's a possibility that user actions can happen at this point, before the view is updated - // alas, this is something we have to deal with, because Unlock() can be called from any thread - go func() { - Do(func() { - t.RLock() - defer t.RUnlock() - C.tableUpdate(t._id) - }) - }() + // TODO RACE CONDITION HERE + // not sure about this one... + t.RLock() + defer t.RUnlock() + C.tableUpdate(t._id) } //export goTableDataSource_getValue |
