Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix regression introduced by commit 49aec20d #926

Merged
merged 1 commit into from
Jul 13, 2014

Conversation

ceefour
Copy link
Contributor

@ceefour ceefour commented Jul 13, 2014

Fix regression introduced by commit 49aec20#diff-0141897e92401b447418eef495135e7eL142

Warning file /home/ceefour/git/opencog/opencog/nlp/scm/relex-to-logic.scm ended with unterminated input begun at line 140
scm_unprotect_object called on unprotected object
[2014-07-13 06:18:09:507] [ERROR] Caught signal 6 (Aborted) on thread 140628137334784
        Stack Trace:
        2: opencog/server/cogserver (opencog::Logger::Base::~Base() +0x7c) [0x41743e]

cc @sebastianruder

…1b447418eef495135e7eL142

```
Warning file /home/ceefour/git/opencog/opencog/nlp/scm/relex-to-logic.scm ended with unterminated input begun at line 140
scm_unprotect_object called on unprotected object
[2014-07-13 06:18:09:507] [ERROR] Caught signal 6 (Aborted) on thread 140628137334784
        Stack Trace:
        2: opencog/server/cogserver (opencog::Logger::Base::~Base() +0x7c) [0x41743e]
```
cosmoharrigan pushed a commit that referenced this pull request Jul 13, 2014
Fix regression introduced by commit 49aec20
@cosmoharrigan cosmoharrigan merged commit 1a7f6a6 into opencog:master Jul 13, 2014
@cosmoharrigan
Copy link
Member

Thanks @ceefour

@ceefour
Copy link
Contributor Author

ceefour commented Jul 13, 2014

:)

@ceefour ceefour deleted the fix-49ae branch July 13, 2014 08:09
@sebastianruder
Copy link
Contributor

Thanks @ceefour. I had missed that.

linas added a commit that referenced this pull request Jul 13, 2014
This fixes an icky memory leak in the guile code.
@linas
Copy link
Member

linas commented Jul 13, 2014

It shouldn't have crashed in the first place; I just fixed the crash.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants