Skip to content

Attempt to fix Windows file locking bug #425

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 12 commits into from
Feb 26, 2015
Merged

Attempt to fix Windows file locking bug #425

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
749826c
Attempt to fix Windows file locking bug
tbranyen Feb 25, 2015
aea1ae7
Change to a single clone path
tbranyen Feb 25, 2015
0e80c58
Ensure gc exists
tbranyen Feb 25, 2015
be57dcb
Trying to fix issue
tbranyen Feb 25, 2015
7eb2108
Add ClearValue method to structs and classes
Feb 25, 2015
98119a6
Cleanup repo after each clone test
Feb 25, 2015
9141d59
Add unique directories to each clone test
Feb 25, 2015
762fdf3
Fix null ref error in beforeEach
Feb 25, 2015
90d580a
Fix test var null ref
Feb 25, 2015
796c09f
Add repo from git clone to test instance so it can be cleaned up
Feb 25, 2015
e19e8ea
Added global `afterEach`, standardized repo attach
tbranyen Feb 25, 2015
342b6e6
Converted to `beforeEach` for all tests
tbranyen Feb 25, 2015
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
Add ClearValue method to structs and classes
  • Loading branch information
John Haley committed Feb 26, 2015
commit 7eb2108c06f7d4bfefccbbacd7dfd82b168a36ea
3 changes: 1 addition & 2 deletions generate/templates/partials/sync_function.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,8 +74,7 @@ from_{{ arg.name }}
{%endif%}

{% if cppFunctionName == "Free" %}
// FIXME Stuck here unable to NULL out this->repo
// ObjectWrap::Unwrap<{{ cppClassName }}>(args.This())->SetValue(NULL);
ObjectWrap::Unwrap<{{ cppClassName }}>(args.This())->ClearValue();
}
{% endif %}

Expand Down
6 changes: 5 additions & 1 deletion generate/templates/templates/class_content.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,11 @@ using namespace node;
}

{{ cType }} **{{ cppClassName }}::GetRefValue() {
return &this->raw;
return this->raw == NULL ? NULL : &this->raw;
}

void {{ cppClassName }}::ClearValue() {
this->raw = NULL;
}

{% else %}
Expand Down
1 change: 1 addition & 0 deletions generate/templates/templates/class_header.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ class {{ cppClassName }} : public ObjectWrap {
{%if cType%}
{{ cType }} *GetValue();
{{ cType }} **GetRefValue();
void ClearValue();

static Handle<v8::Value> New(void *raw, bool selfFreeing);
{%endif%}
Expand Down
6 changes: 5 additions & 1 deletion generate/templates/templates/struct_content.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,11 @@ Handle<v8::Value> {{ cppClassName }}::New(void* raw, bool selfFreeing) {
}

{{ cType }} **{{ cppClassName }}::GetRefValue() {
return &this->raw;
return this->raw == NULL ? NULL : &this->raw;
}

void {{ cppClassName }}::ClearValue() {
this->raw = NULL;
}

{% partial fieldAccessors . %}
Expand Down
1 change: 1 addition & 0 deletions generate/templates/templates/struct_header.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ class {{ cppClassName }} : public ObjectWrap {

{{ cType }} *GetValue();
{{ cType }} **GetRefValue();
void ClearValue();

static Handle<v8::Value> New(void *raw, bool selfFreeing);

Expand Down