Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Fix for Bug #20, uses Protobuf FieldDescriptor->name() #21

Merged
merged 1 commit into from

2 participants

@AdamMagaluk

Getter and setter functions changed from using FieldDescriptor->camelcase_name() to FieldDescriptor->name()

Allows for protobuf field names with underscores to be used.

Adam Magaluk Getter and setter functions changed from using FieldDescriptor->camel…
…case_name() to FieldDescriptor->name()
d45c173
@chrisdew chrisdew merged commit 771b73d into from
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Nov 18, 2012
  1. Getter and setter functions changed from using FieldDescriptor->camel…

    Adam Magaluk authored
    …case_name() to FieldDescriptor->name()
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 2 deletions.
  1. +2 −2 protobuf_for_node.cc
View
4 protobuf_for_node.cc
@@ -143,11 +143,11 @@ namespace protobuf_for_node {
from <<
"var x = arr[" << i << "]; "
"if(x !== undefined) this['" <<
- descriptor->field(i)->camelcase_name() <<
+ descriptor->field(i)->name() <<
"'] = x; ";
if (i > 0) to << ", ";
- to << "this['" << descriptor->field(i)->camelcase_name() << "']";
+ to << "this['" << descriptor->field(i)->name() << "']";
}
from << " }})";
Something went wrong with that request. Please try again.