Skip to content

Refactor classes #250

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.

Sign up for GitHub

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

Merged
tbranyen merged 8 commits into from
Oct 23, 2014
Merged

Refactor classes #250

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
ed966eb
add test for segfaults in revwalk
maxkorp Oct 23, 2014
f7a1903
change the way classes free themselves
maxkorp Oct 23, 2014
cde854b
change "createX" methods to "create"
maxkorp Oct 23, 2014
c72360e
fix linting
maxkorp Oct 23, 2014
266376e
expose gc for travis
maxkorp Oct 23, 2014
57b0623
move to nannew for boolean passing of selfFreeing
maxkorp Oct 23, 2014
2c92bc9
fix issue with building in v11 using nanreturnvalue
maxkorp Oct 23, 2014
86a41a0
add custom macro for dealing with returning while escaping a scope in…
maxkorp Oct 23, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix linting
  • Loading branch information
@maxkorp
maxkorp committed Oct 23, 2014
commit c72360e19568bf5b5837f49950ac9f36f269f9ab
36 changes: 18 additions & 18 deletions test/tests/revwalk.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -62,27 +62,27 @@ describe("Revwalk", function() {
// node rather than npm, with a la `node --expose-gc [pathtohmoca] [testglob]`
var testGC = (global.gc ? it : it.skip);

testGC("doesnt segfault when accessing commit.author() twice", function(done) {
this.timeout(10000);
return Repository.open(reposPath).then(function(repository) {
var walker = repository.createRevWalk();
repository.getMaster().then(function(master) {
var did = false;
walker.walk(master, function(error, commit) {
for (var i = 0; i < 1000; i++) {
if (true) {
commit.author().name();
commit.author().email();
testGC("doesnt segfault when accessing .author() twice", function(done) {
this.timeout(10000);
return Repository.open(reposPath).then(function(repository) {
var walker = repository.createRevWalk();
repository.getMaster().then(function(master) {
var did = false;
walker.walk(master, function(error, commit) {
for (var i = 0; i < 1000; i++) {
if (true) {
commit.author().name();
commit.author().email();
}
global.gc();
}
if (!did) {
done();
did = true;
}
global.gc();
}
if (!did) {
done();
did = true;
}
});
});
});
});
});

});