aboutsummaryrefslogtreecommitdiffhomepage
path: root/csharp
Commit message (Collapse)AuthorAge
* Remove duplicate test cases.Gravatar Jon Skeet2016-04-01
| | | | (NCrunch noticed these.)
* Merge pull request #1369 from jskeet/tools-nuspecGravatar Jan Tattermusch2016-03-31
|\ | | | | Introduce a new nuget package, Google.Protobuf.Tools, basically to contain protoc on multiple platforms.
| * Remove duplicate lineGravatar Jon Skeet2016-03-31
| |
| * Introduce a new nuget package, Google.Protobuf.Tools, basically to contain ↵Gravatar Jon Skeet2016-03-31
| | | | | | | | | | | | | | | | protoc on multiple platforms. I've moved both protoc.exe and the proto files out of Google.Protobuf. The .proto files aren't a slam-dunk, but it feels like they belong with protoc as you'd *use* them with protoc. It's not clear to me whether we really need both an x86 and x64 version of protoc.exe, as x86 would work on 64-bit Windows anyway. Discuss :)
* | Refactoring of FieldDescriptorGravatar Jon Skeet2016-03-30
|/ | | | | | | | This makes no externally visible behavioral changes. Internally and non-behaviorally: - We use a field (compiler-generated) to store the JsonName to avoid recomputing it repeatedly - The documentation for JsonName is updated to reflect the meaning better - Readonly autoprops and expression-bodied properties used where possible
* Code review fixesGravatar alien2016-03-29
|
* csharp: add support for the json_name optionGravatar alien2016-03-18
| | | | | Conflicts: csharp/src/Google.Protobuf/JsonFormatter.cs
* Replace StringBuilder with TextWriter in JsonFormatterGravatar avgweb2016-03-06
|
* Fix copy/paste typo in CodedInputStreamTestGravatar Jon Skeet2016-03-01
|
* Implement IDisposable for CodedInputStream and CodedOutputStreamGravatar Jon Skeet2016-02-29
| | | | | This fixes issue #679 and issue #1282. (The .gitignore change is just around ncrunch; I can put it in a separate PR if you really want.)
* Require VS2015 in the solution fileGravatar Jon Skeet2016-02-22
|
* Merge pull request #1240 from jskeet/validate_groupGravatar Jan Tattermusch2016-02-18
|\ | | | | Validate that end-group tags match their corresponding start-group tags
* \ Merge pull request #1239 from jskeet/call_generate_protosGravatar Jon Skeet2016-02-18
|\ \ | | | | | | Generate C# code whenever descriptor.proto changes
* | | Add more tests around merging wrappersGravatar Jon Skeet2016-02-15
| | | | | | | | | | | | This was in an attempt to fix the wrapper handling corner case, but it's really fiddly.
| | * Validate that end-group tags match their corresponding start-group tagsGravatar Jon Skeet2016-02-15
| |/ |/| | | | | | | | | | | | | This detects: - An end-group tag with the wrong field number (doesn't match the start-group field) - An end-group tag with no preceding start-group tag Fixes issue #688.
| * Generate C# code whenever descriptor.proto changesGravatar Jon Skeet2016-02-15
|/ | | | | | | This is a start to fixing issue #1212. It won't help for test protos, conformance etc, but it will definitely be better than nothing, and would have highlighted a change in descriptor.proto which broken C# earlier.
* Merge pull request #1218 from jskeet/nuspec-dependenciesGravatar Jan Tattermusch2016-02-11
|\ | | | | nuspec dependency fixes
* | Add a linkGravatar Jan Tattermusch2016-02-10
| |
* | Use checked-in key to build ReleaseSignedGravatar Jan Tattermusch2016-02-09
| |
* | add private key for strong name assembliesGravatar Jan Tattermusch2016-02-09
| |
| * nuspec dependency fixesGravatar Jon Skeet2016-02-08
|/ | | | | | | Explicitly don't add any dependencies for "old" platforms, to avoid unnecessary dependencies in those cases. Fixes issue #1203.
* Generated code changes from previous commitGravatar Jon Skeet2016-02-04
|
* Rename Preconditions to ProtoPreconditionsGravatar Jon Skeet2016-02-04
| | | | (Generated code changes in next commit.)
* Generated code for GeneratedCodeInfo changeGravatar Jon Skeet2016-02-04
|
* Rename GeneratedCodeInfo to GeneratedClrTypeInfoGravatar Jon Skeet2016-02-04
| | | | | | | Recently, descriptor.proto gained a GeneratedCodeInfo message, which means the generated code conflicts with our type. Unfortunately this affects codegen as well, although this is a part of the public API which is very unlikely to affect hand-written code. Generated code changes in next commit.
* Clarify readmeGravatar Jon Skeet2016-02-03
|
* Merge pull request #1158 from jskeet/nonnormalized-tostringGravatar Jan Tattermusch2016-01-26
|\ | | | | Ensure that FieldMask, Timestamp and Duration ToString() calls don't throw
* | Fix mistakes in highest-representable float64 integersGravatar Jon Skeet2016-01-21
| |
| * Ensure that FieldMask, Timestamp and Duration ToString() calls don't throwGravatar Jon Skeet2016-01-20
|/ | | | | | | | | | | | | The usage of ICustomDiagnosticMessage here is non-essential - ToDiagnosticString doesn't actually get called by ToString() in this case, due to JsonFormatter code. It was intended to make it clearer that it *did* have a custom format... but then arguably I should do the same for Value, Struct, Any etc. Moving some of the code out of JsonFormatter and into Duration/Timestamp/FieldMask likewise feels somewhat nice, somewhat nasty... basically there are JSON-specific bits of formatting, but also domain-specific bits of computation. <sigh> Thoughts welcome.
* Merge pull request #1096 from jskeet/custom-to-stringGravatar Jan Tattermusch2016-01-19
|\ | | | | Introduce ICustomDiagnosticMessage to allow for custom string formatting
* | Fix broken testGravatar Jon Skeet2016-01-15
| |
* | Make sure thatGravatar Jon Skeet2016-01-15
| | | | | | | | | | | | "valueField": null is parsed appropriately, i.e. that it remembers that the field is set.
* | Reject JSON containing the same oneof field twiceGravatar Jon Skeet2016-01-15
| |
* | Change handling of unknown enums: we now write out the value as a number.Gravatar Jon Skeet2016-01-15
| |
* | Extra strictness for FieldMask conversionGravatar Jon Skeet2016-01-15
| |
* | Allow the original field name (rather than camel-cased) when parsing JSONGravatar Jon Skeet2016-01-15
| |
* | Fixes to JSON timestamp/duration representationsGravatar Jon Skeet2016-01-15
| |
* | Report serialization errors in conformance testsGravatar Jon Skeet2016-01-15
| |
* | Prohibit null values in repeated and map fields in JSONGravatar Jon Skeet2016-01-15
| |
* | Throw a better exception when invalid base64 is detected in JSONGravatar Jon Skeet2016-01-15
| |
* | Support (and test) numeric enum parsing in JSONGravatar Jon Skeet2016-01-15
| |
* | Fix handling of repeated wrappersGravatar Jon Skeet2016-01-15
| | | | | | | | | | Previously we were incorrectly packing wrapper types. This also refactors FieldCodec a bit as well, using more C# 6-ness.
* | JSON conformance test fixesGravatar Jon Skeet2016-01-13
| | | | | | | | | | - Spot an Any without a type URL - In the conformance test runner, catch exceptions due to generally-invalid JSON
| * Introduce ICustomDiagnosticMessage to allow for custom string formattingGravatar Jon Skeet2016-01-13
|/ | | | This fixes issue #933, effectively.
* Merge pull request #1089 from jskeet/map-nullGravatar Jan Tattermusch2016-01-12
|\ | | | | Prohibit null values in maps
* | Fix typos in README.md/CHANGES.txtGravatar Dongjoon Hyun2016-01-11
| |
| * Prohibit null values in map fieldsGravatar Jon Skeet2016-01-11
|/ | | | | On deserialization, missing values for message types are replaced with a "default" message.
* Merge pull request #1098 from jskeet/wkt-jsonGravatar Jan Tattermusch2016-01-08
|\ | | | | Ensure all formatted well-known-type values are valid JSON
* \ Merge pull request #1088 from jskeet/codec-pruningGravatar Jan Tattermusch2016-01-08
|\ \ | | | | | | Remove unused method in FieldCodec.
| | * Ensure all formatted well-known-type values are valid JSONGravatar Jon Skeet2016-01-06
| | | | | | | | | | | | | | | | | | | | | This involves quoting timestamp/duration/field-mask values, even when they're not in fields. It's better for consistency. Fixes issue #1097.