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

Jump to bottom

C# protocol buffers (initial pull request) #288

Merged
merged 560 commits into from Apr 18, 2015
Merged

C# protocol buffers (initial pull request) #288

merged 560 commits into from Apr 18, 2015

Conversation

jtattermusch
Copy link
Contributor

-- Adding legacy code from protobuf csharp port project under csharp/ subtree. Some cleanup and history rewriting was performed.
-- A protoc C# plugin rewritten to C++.

csharptest and others added 30 commits June 10, 2011 14:41
@csharptest
Added the XmlFormatWriter/Reader
2b86884
@csharptest
Added the JsonFormatWriter/Reader
afe844b
@csharptest
Using List<Char> instead of StringBuilder for building strings from c…
ddb74eb 
…hars
@csharptest
Added initial DictionaryReader/Writer implementations
4dc0dfb
@csharptest
Added benchmark for Json via XML using JsonReaderWriterFactory
ea5fd37
@csharptest
Reader/Writer implementations changed to use static factories
7fc785c
@csharptest
Added convenience methods for to/from xml and json
f292523
@csharptest
Added option to enable format benchmarks
57fa7fd
@csharptest
Added recursion limits to AbstractReader.cs
3b70dd7
@csharptest
bla, changed name to Depth for CLSComplaint :)
6c69373
@csharptest
missing copyright notice
ff628f6
@csharptest
Fix for leaving Builder a public ctor on internal classes
579bb3d
@csharptest
Fixed A few last methods taking CodedInputStream
afff2c6
@csharptest
Reformatted to include braces
74c5e0c
@csharptest
Reformatted to include braces
8dc7b83
@csharptest
Fix for unqualified ByteString references in LITE generation
b73bd16
@csharptest
Removed unnessary parital declaration
398a7f5
@csharptest
renamed ByteString.WriteTo(CodedOutputStream) to WriteRawBytesTo
ed701ad
@csharptest
Fixed comment
84f7e09
@csharptest
Integrated feedback from revision c97eeb8d933f
b5ba93b
@csharptest
Removed left-over commented code
541686c
@csharptest
Integrated feedback from revision 1525875aec27
d95293e
@csharptest
Regeneration with qualified references to ByteString
02760b2
@csharptest
Refactoring removed using statement for System.Reflection, added qual…
c11bf71 
…ified

references to avoid future issues.
@csharptest
Removal of #condition commented source
3aa0635
@ngbrown
Added tests for service generation and corrected internal access leve…
d4b3859 
…l error in

service definition.
@csharptest
Changes from review f60f2676f7fa
9b3de69
@csharptest
Changes from review e805fdfd3a6b
304ff3a
@csharptest
Split off the Serialization namespace into a new assembly.
0f3540e
@jskeet
Overload of ContinueArray for situations where we know in advance tha…
cdb4518 
…t the array isn't packed.
jtattermusch and others added 22 commits April 13, 2015 12:40
@jtattermusch
updated C# codegen to use restricted set of csharp options from descr…
44664bb 
…iptor.proto
@jtattermusch
enable overriding of property name using csharp_property_name
16a3e2b
@jtattermusch
added support for deprecated fields, reworded some todos
8cec65e
@jtattermusch
output file naming logic, configurable file extension, comment rewording
f61e179
@jtattermusch
creating private ctor for builders is now the default (as it used to …
d92de17 
…be for old codegen)
@jtattermusch
addressed TODO for lite runtime
db9060d
@jtattermusch
implemented HasRequiredFields logic
5ac8de5
@jtattermusch
fixing implementation of GetFileUmbrellaNamespace
7cc05ec
@jtattermusch
integrate CalcSerializedSize() fix from protobuf-csharp-port issue 84
333d200
@jtattermusch
Imported protobuf-csharp-port into csharp/ tree
84fa3e2
@jtattermusch
Updated readme.md and changes.txt, removed old license.txt
e77878e
@jtattermusch
cleanup of csharp/lib/ directory
6b7f017
@jtattermusch
Added instructions to manually download the dependencies
4e80fe1
@jtattermusch
removed .hgignore file
1eab56a
@jtattermusch
fixing field number of csharp_umbrella_namespace option
2c2f76c
@jtattermusch
Fixed comment for csharp_umbrella_namespace (nest_classes has been re…
abcde24 
…moved)
@jtattermusch
fix: do not mention umbrella_classname in comment
b6555b1
@jtattermusch
one more fix to csharp_umbrella_namespace comment
b36420a
@jtattermusch
removing csharp_property_name field option
83a5991
@jtattermusch
temporarily removing csharp_umbrella_namespace file option
8e190d4
@jtattermusch
Merge pull request #2 from jtattermusch/csharp_protobuf_imported
01c8c35 
Pulling protobuf-csharp-port into csharp/ directory
@jtattermusch
Merge pull request #1 from jtattermusch/csharp_protogen_rewrite
8394a8d 
Rewrite of C# protogen to C++ (initial version)
@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@jtattermusch
Copy link
Contributor Author

FTR:
Details about CLA signers and consent to contribute to this repo can be found in go/csharp-protobuf-migration document.

anandolee added a commit that referenced this pull request Apr 18, 2015
@anandolee
Merge pull request #288 from jtattermusch/csharp_protobuf_reviewed
7646eec 
C# protocol buffers (initial pull request)
@anandolee anandolee merged commit 7646eec into protocolbuffers:csharp Apr 18, 2015
@jtattermusch jtattermusch deleted the csharp_protobuf_reviewed branch May 13, 2015 18:04
bithium pushed a commit to bithium/protobuf that referenced this pull request Sep 4, 2023
@anandolee
Merge pull request protocolbuffers#288 from jtattermusch/csharp_proto…
ae01797 
…buf_reviewed

C# protocol buffers (initial pull request)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
cla: no
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
7 participants
@jtattermusch @googlebot @anandolee @csharptest @ngbrown @jskeet @SergeyGeyn