Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fix mistakes in highest-representable float64 integers | Jon Skeet | 2016-01-21 | 1 | -2/+2 |
| | |||||
* | Merge pull request #1096 from jskeet/custom-to-string | Jan Tattermusch | 2016-01-19 | 7 | -1/+142 |
|\ | | | | | Introduce ICustomDiagnosticMessage to allow for custom string formatting | ||||
| * | Introduce ICustomDiagnosticMessage to allow for custom string formatting | Jon Skeet | 2016-01-13 | 7 | -1/+142 |
| | | | | | | | | This fixes issue #933, effectively. | ||||
* | | Fix broken test | Jon Skeet | 2016-01-15 | 1 | -1/+1 |
| | | |||||
* | | Make sure that | Jon Skeet | 2016-01-15 | 3 | -8/+68 |
| | | | | | | | | | | | | "valueField": null is parsed appropriately, i.e. that it remembers that the field is set. | ||||
* | | Reject JSON containing the same oneof field twice | Jon Skeet | 2016-01-15 | 2 | -0/+22 |
| | | |||||
* | | Change handling of unknown enums: we now write out the value as a number. | Jon Skeet | 2016-01-15 | 4 | -36/+23 |
| | | |||||
* | | Extra strictness for FieldMask conversion | Jon Skeet | 2016-01-15 | 4 | -2/+51 |
| | | |||||
* | | Allow the original field name (rather than camel-cased) when parsing JSON | Jon Skeet | 2016-01-15 | 2 | -3/+23 |
| | | |||||
* | | Fixes to JSON timestamp/duration representations | Jon Skeet | 2016-01-15 | 8 | -35/+139 |
| | | |||||
* | | Report serialization errors in conformance tests | Jon Skeet | 2016-01-15 | 1 | -8/+15 |
| | | |||||
* | | Prohibit null values in repeated and map fields in JSON | Jon Skeet | 2016-01-15 | 2 | -1/+38 |
| | | |||||
* | | Throw a better exception when invalid base64 is detected in JSON | Jon Skeet | 2016-01-15 | 3 | -1/+28 |
| | | |||||
* | | Support (and test) numeric enum parsing in JSON | Jon Skeet | 2016-01-15 | 2 | -0/+29 |
| | | |||||
* | | Fix handling of repeated wrappers | Jon Skeet | 2016-01-15 | 3 | -77/+94 |
| | | | | | | | | | | Previously we were incorrectly packing wrapper types. This also refactors FieldCodec a bit as well, using more C# 6-ness. | ||||
* | | JSON conformance test fixes | Jon Skeet | 2016-01-13 | 3 | -0/+16 |
|/ | | | | | - Spot an Any without a type URL - In the conformance test runner, catch exceptions due to generally-invalid JSON | ||||
* | Merge pull request #1089 from jskeet/map-null | Jan Tattermusch | 2016-01-12 | 13 | -174/+167 |
|\ | | | | | Prohibit null values in maps | ||||
| * | Prohibit null values in map fields | Jon Skeet | 2016-01-11 | 13 | -174/+167 |
| | | | | | | | | | | On deserialization, missing values for message types are replaced with a "default" message. | ||||
* | | Fix typos in README.md/CHANGES.txt | Dongjoon Hyun | 2016-01-11 | 1 | -1/+1 |
|/ | |||||
* | Merge pull request #1098 from jskeet/wkt-json | Jan Tattermusch | 2016-01-08 | 3 | -63/+55 |
|\ | | | | | Ensure all formatted well-known-type values are valid JSON | ||||
| * | Ensure all formatted well-known-type values are valid JSON | Jon Skeet | 2016-01-06 | 3 | -63/+55 |
| | | | | | | | | | | | | | | This involves quoting timestamp/duration/field-mask values, even when they're not in fields. It's better for consistency. Fixes issue #1097. | ||||
* | | Merge pull request #1088 from jskeet/codec-pruning | Jan Tattermusch | 2016-01-08 | 1 | -5/+0 |
|\ \ | | | | | | | Remove unused method in FieldCodec. | ||||
| * | | Remove unused method in FieldCodec. | Jon Skeet | 2016-01-05 | 1 | -5/+0 |
| |/ | | | | | | | (The method was last used a very long time ago, if ever.) | ||||
* / | Improve number handling in JSON parsing | Jon Skeet | 2016-01-06 | 2 | -51/+91 |
|/ | | | | | | - Tighten up on Infinity/NaN handling in terms of whitespace handling (and test casing) - Validate that values are genuinely integers when they've been parsed from a JSON number (ignoring the fact that 1.0000000000000000001 == 1 as a double...) - Allow exponents and decimal points in string representations | ||||
* | Fix C# source generation shell script and remove out of date notes | Jon Skeet | 2016-01-04 | 2 | -125/+1301 |
| | | | | | The conformance tests now use types which are part of src/google/protobuf, so we need to include src in the proto path. The notes around "fix-ups" have been out of date for some time now. | ||||
* | add reference generator dependency | Jan Tattermusch | 2015-12-17 | 2 | -0/+14 |
| | |||||
* | Merge pull request #1049 from jskeet/any-format | Jon Skeet | 2015-12-17 | 21 | -117/+191 |
|\ | | | | | Handle Any formatting for diagnostic purposes | ||||
| * | Generated code changes from previous commit. | Jon Skeet | 2015-12-15 | 19 | -114/+117 |
| | | |||||
| * | Make ToString() valid without a type registry | Jon Skeet | 2015-12-15 | 2 | -3/+74 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This addresses issue #1008, by creating a JsonFormatter which is private and only different to JsonFormatter.Default in terms of reference equality. Other plausible designs: - The same, but expose the diagnostic-only formatter - Add something to settings to say "I don't have a type registry at all" - Change the behaviour of JsonFormatter.Default (bad idea IMO, as we really *don't* want the result of this used as regular JSON to be parsed) Note that just trying to find a separate fix to issue #933 and using that to override Any.ToString() differently wouldn't work for messages that *contain* an Any. Generated code changes follow in the next commit. | ||||
* | | Merge pull request #1051 from jtattermusch/support_coreclr | Jan Tattermusch | 2015-12-16 | 1 | -19/+38 |
|\ \ | | | | | | | Make nuget package support coreCLR | ||||
| * | | make nuget package support CoreCLR | Jan Tattermusch | 2015-12-16 | 1 | -19/+38 |
| |/ | |||||
* / | Add JSON parsing to conformance tests. | Jon Skeet | 2015-12-16 | 1 | -19/+24 |
|/ | |||||
* | Handle JSON parsing for Any. | Jon Skeet | 2015-12-02 | 5 | -509/+761 |
| | | | | This required a rework of the tokenizer to allow for a "replaying" tokenizer, basically in case the @type value comes after the data itself. This rework is nice in some ways (all the pushback and object depth logic in one place) but is a little fragile in terms of token push-back when using the replay tokenizer. It'll be fine for the scenario we need it for, but we should be careful... | ||||
* | JSON formatting for Any. | Jon Skeet | 2015-12-02 | 2 | -18/+135 |
| | |||||
* | Generated code changes for previous commit (basically ↵ | Jon Skeet | 2015-11-22 | 21 | -22/+22 |
| | | | | InternalBuildGeneratedFileFrom => FromGeneratedCode) | ||||
* | Tidy up reflection in advance of attempting to implement DynamicMessage. | Jon Skeet | 2015-11-22 | 9 | -224/+187 |
| | | | | | | | | | There are corner cases where MessageDescriptor.{ClrType,Parser} will return null, and these are now documented. However, normally they *should* be implemented, even for descriptors of for dynamic messages. Ditto FieldDescriptor.Accessor. We'll still need a fair amount of work to implement dynamic messages, but this change means that the public API will be remain intact. Additionally, this change starts making use of C# 6 features in the files that it touches. This is far from exhaustive, and later PRs will have more. Generated code changes coming in the next commit. | ||||
* | Added the type registry in advance of implementing Any support. | Jon Skeet | 2015-11-21 | 4 | -0/+279 |
| | | | | Biting off just this bit first as I don't need the changes from a previous PR for this part. | ||||
* | Merge pull request #988 from gabikliot/CSharp-SampleUsage-Big-Fix | Jan Tattermusch | 2015-11-19 | 1 | -2/+5 |
|\ | | | | | Fixed a bug in CSharp SampleUsage. | ||||
| * | Removed redundant comment. | Gabriel Kliot | 2015-11-19 | 1 | -1/+0 |
| | | |||||
| * | Fixed a bug in CSharp SampleUsage. | Gabriel Kliot | 2015-11-18 | 1 | -1/+5 |
| | | |||||
* | | Finished an implementation comment | Jon Skeet | 2015-11-19 | 1 | -1/+3 |
| | | |||||
* | | Generated code from previous commit. | Jon Skeet | 2015-11-19 | 20 | -105/+107 |
| | | |||||
* | | Introduce a Parser property into MessageDescriptor, and populate it from ↵ | Jon Skeet | 2015-11-19 | 4 | -16/+134 |
| | | | | | | | | | | | | generated types. Generated code coming in next commit - in a subsequent PR I want to do a bit of renaming and redocumenting around this, in anticipation of DynamicMessage. | ||||
* | | Generated code changes and manual changes for previous commit. | Jon Skeet | 2015-11-09 | 27 | -467/+441 |
|/ | |||||
* | Generated code for previous commit. | Jon Skeet | 2015-11-06 | 10 | -0/+115 |
| | |||||
* | Merge pull request #941 from jskeet/recursion-limit | Jan Tattermusch | 2015-11-05 | 6 | -15/+135 |
|\ | | | | | Add recursion limit handling to JSON parsing. | ||||
| * | Reimplement JSON recursion by detecting the depth in the tokenizer. | Jon Skeet | 2015-11-05 | 4 | -34/+124 |
| | | | | | | | | Added a TODO around a possible change to the tokenizer API, changing PushBack(token) into just Rewind() or something similar. | ||||
| * | Add recursion limit handling to JSON parsing. | Jon Skeet | 2015-11-04 | 3 | -44/+74 |
| | | | | | | | | Fixes issue #932. | ||||
* | | Merge pull request #940 from jskeet/json-names | Jon Skeet | 2015-11-05 | 2 | -6/+16 |
|\ \ | | | | | | | Move the creation of the "fields by JSON name" dictionary to the descriptor | ||||
| * | | Move the creation of the "fields by JSON name" dictionary to the descriptor. | Jon Skeet | 2015-11-04 | 2 | -6/+16 |
| |/ |