Skip to content
This repository has been archived by the owner on Jun 16, 2021. It is now read-only.

Add Close() method to Project to release resources #451

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add Close() method to Project to release resources #451

wants to merge 1 commit into from

Commits on Apr 25, 2017

  1. Add Close() method to Project to release resources 

    Problem Observed

Goroutine leaks because there is no way to "close" projects

Problem Detail

NewProject() creates a defaultListener in it.
The listener starts a new goroutine in NewDefaultListener().
We currently don't have a method to stop this goroutine.

Suggested Resolution

This commit adds a new method Close() to release resources
tied to a Project.

Signed-off-by: Iwasaki Yudai <[email protected]>
    @yudai
    yudai committed Apr 25, 2017
    Configuration menu
    Copy the full SHA
    6cce1d7 View commit details
    Browse the repository at this point in the history