Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Address review comments. | Jon Skeet | 2015-08-08 | 1 | -2/+0 |
| | |||||
* | Remove reference to generic types within XML comment. | Jon Skeet | 2015-08-08 | 1 | -2/+2 |
| | | | | Visual Studio is happy with it, but Mono isn't :( | ||||
* | Tidying up - fix a bunch of TODOs and remove outdated ones. | Jon Skeet | 2015-08-08 | 17 | -102/+84 |
| | |||||
* | Address review comments. | Jon Skeet | 2015-08-08 | 2 | -2/+24 |
| | |||||
* | Generated code for previous commit. | Jon Skeet | 2015-08-06 | 21 | -452/+142 |
| | |||||
* | Skip groups properly. | Jon Skeet | 2015-08-06 | 6 | -32/+138 |
| | | | | | | | | | | | Now the generated code doesn't need to check for end group tags, as it will skip whole groups at a time. Currently it will ignore extraneous end group tags, which may or may not be a good thing. Renamed ConsumeLastField to SkipLastField as it felt more natural. Removed WireFormat.IsEndGroupTag as it's no longer useful. This mostly fixes issue 688. (Generated code changes coming in next commit.) | ||||
* | Make InvalidProtocolBufferException.InvalidTag internal. | Jon Skeet | 2015-08-05 | 1 | -1/+1 |
| | | | | We don't need to expose the InvalidProtocolBufferException factory method now that the generated code doesn't throw the exception. | ||||
* | Generated code changes for previous commit. | Jon Skeet | 2015-08-05 | 19 | -339/+113 |
| | |||||
* | Change ReadTag and PeekTag to just use 0 as a return value for "end of ↵ | Jon Skeet | 2015-08-05 | 7 | -62/+52 |
| | | | | | | | | stream", rather than using an awkward out parameter. This simplifies quite a lot of code. Generated code in next commit. | ||||
* | Update AssemblyInfo for conformance - mostly to fix version numbers. | Jon Skeet | 2015-08-05 | 1 | -23/+35 |
| | |||||
* | Validate that after reading a message, we've consumed as many bytes as we ↵ | Jon Skeet | 2015-08-05 | 2 | -8/+47 |
| | | | | | | expected to. We should now have no conformance failures. | ||||
* | Generated code changes for previous commit. | Jon Skeet | 2015-08-05 | 19 | -0/+113 |
| | |||||
* | Consume unknown fields when parsing. | Jon Skeet | 2015-08-05 | 2 | -0/+62 |
| | | | | | This is expected to be the cause of the conformance test failures. Generated code in next commit. | ||||
* | First pass at C# conformance tests. | Jon Skeet | 2015-08-05 | 6 | -0/+2637 |
| | | | | Completely untested so far - easier to get started in VS and then transfer to Linux for tweaking... | ||||
* | Merge pull request #691 from jskeet/xml-documentation | Jon Skeet | 2015-08-05 | 27 | -108/+742 |
|\ | | | | | Document everything, and turn on errors if we fail to document anything in the future | ||||
| * | Fix one cref that Mono was unhappy with | Jon Skeet | 2015-08-05 | 1 | -1/+1 |
| | | |||||
| * | Document everything, and turn on errors if we fail to document anything in ↵ | Jon Skeet | 2015-08-04 | 26 | -107/+741 |
| | | | | | | | | the future. | ||||
* | | Fix build warnings around unused variables | Jon Skeet | 2015-08-04 | 3 | -3/+3 |
|/ | |||||
* | Merge pull request #681 from jskeet/json-fieldmask | Jon Skeet | 2015-08-03 | 2 | -1/+39 |
|\ | | | | | JSON formatting for FieldMask | ||||
| * | JSON formatting for FieldMask | Jon Skeet | 2015-08-03 | 2 | -1/+39 |
| | | |||||
* | | Merge pull request #680 from jskeet/aptca | Jon Skeet | 2015-08-03 | 1 | -0/+2 |
|\ \ | |/ |/| | Allow partially-trusted callers again. | ||||
| * | Allow partially-trusted callers again. | Jon Skeet | 2015-08-03 | 1 | -0/+2 |
| | | | | | | | | Fixes issue #552. (And yay, it looks like our build profile supports this...) | ||||
* | | Expose Coded*Stream constructors directly. | Jon Skeet | 2015-08-03 | 11 | -127/+100 |
|/ | |||||
* | Merge pull request #675 from jtattermusch/signing | Jon Skeet | 2015-08-03 | 8 | -76/+80 |
|\ | | | | | Add ReleaseSigned configuration for C# | ||||
| * | make changes and fixes in signing | Jan Tattermusch | 2015-08-02 | 10 | -96/+16 |
| | | |||||
| * | added protoc.exe and well known protobuf files to the nuspec file | Jan Tattermusch | 2015-08-02 | 1 | -1/+15 |
| | | |||||
| * | build signed assemblies | Jan Tattermusch | 2015-08-02 | 10 | -22/+92 |
| | | |||||
* | | Initial pass at formatting Struct as JSON. | Jon Skeet | 2015-08-03 | 2 | -3/+98 |
| | | | | | | | | This seems remarkably little code, but it appears to work. I can add tests for invalid structs at some point, once the general approach is approved. | ||||
* | | Addressed issues raised in code review. Will merge when green. | Jon Skeet | 2015-08-03 | 1 | -25/+14 |
| | | |||||
* | | Format JSON for Duration and Timestamp. | Jon Skeet | 2015-08-03 | 3 | -11/+182 |
|/ | | | | This is taking an approach of putting all the logic in JsonFormatter. That's helpful in terms of concealing the details of whether or not to wrap the value in quotes, but it does lack flexibility. I don't *think* we want to allow user-defined formatting of messages, so that much shouldn't be a problem. | ||||
* | Simplify the JSON tests for readability | Jon Skeet | 2015-07-31 | 1 | -60/+71 |
| | | | | Use ' instead of " in the expected JSON, then replace it before asserting. | ||||
* | Fix JSON formatting to always emit fields in field order, including oneofs | Jon Skeet | 2015-07-31 | 3 | -36/+328 |
| | |||||
* | Fix trivial bug in field orderings. | Jon Skeet | 2015-07-31 | 2 | -1/+14 |
| | | | | (Shows the benefit of unit testing even code "too simple to fail"...) | ||||
* | Well-known type operations for Timestamp and Duration (but not JSON formatting). | Jon Skeet | 2015-07-31 | 7 | -0/+563 |
| | | | | | While I've provided operators, I haven't yet provided the method equivalents. It's not clear to me that they're actually a good idea, while we're really targeting C# developers who definitely *can* use the user-defined operators. | ||||
* | Handle field names of "descriptor" and "types". | Jon Skeet | 2015-07-31 | 2 | -6/+235 |
| | |||||
* | Generated code for previous commit. | Jon Skeet | 2015-07-30 | 9 | -57/+57 |
| | |||||
* | Prohibit null values for string/bytes fields in generated code. | Jon Skeet | 2015-07-30 | 1 | -0/+10 |
| | |||||
* | Rename ThrowHelper to Preconditions and make it public - we'll want to use ↵ | Jon Skeet | 2015-07-30 | 6 | -78/+99 |
| | | | | | | | it from the generated code soon. Additionally, change it to return the value passed, and make it generic with a class constraint. A separate method doesn't have the class constraint, for more unusual scenarios. | ||||
* | Generated code from previous commit. | Jon Skeet | 2015-07-30 | 18 | -315/+0 |
| | |||||
* | More freezing tidy-up; generated code in next commit. | Jon Skeet | 2015-07-30 | 1 | -2/+1 |
| | |||||
* | Minor bits of left-over frozenness. | Jon Skeet | 2015-07-30 | 2 | -7/+1 |
| | |||||
* | Merge pull request #654 from jtattermusch/csharp_hide_freeze | Jon Skeet | 2015-07-30 | 26 | -1451/+6 |
|\ | | | | | Remove the C# Freeze API | ||||
| * | regenerate code | Jan Tattermusch | 2015-07-29 | 18 | -1192/+0 |
| | | |||||
| * | remove the freeze API | Jan Tattermusch | 2015-07-29 | 8 | -259/+6 |
| | | |||||
* | | add IsClientStreaming and IsServerStreaming to MethodDescriptor | Jan Tattermusch | 2015-07-29 | 1 | -0/+10 |
|/ | |||||
* | Humbug - previous commit didn't include project file changes :( | Jon Skeet | 2015-07-28 | 1 | -0/+2 |
| | |||||
* | Tweaks to Profile259 support | Jon Skeet | 2015-07-28 | 6 | -11/+303 |
| | | | | | | - Fix nupec paths - Remove an obsolete part of the JSON build - Add documentation and tests to reflection extension methods, and improve implementations | ||||
* | First attempt at using profile 259 for Google.Protobuf. | Jon Skeet | 2015-07-27 | 14 | -15/+146 |
| | | | | | | | | This requires .NET 4.5, and there are a few compatibility changes required around reflection. Creating a PR from this to see how our CI systems handle it. Will want to add more documentation, validation and probably tests before merging. This is in aid of issue #590. | ||||
* | expose original binary data for filedescriptor | Jan Tattermusch | 2015-07-24 | 2 | -4/+21 |
| | |||||
* | Implemented Jan's suggestion of FieldCollection, replacing ↵ | Jon Skeet | 2015-07-22 | 6 | -119/+128 |
| | | | | | | | | | | FieldAccessorCollection. I think Jan was actually suggesting keeping both, but that feels redundant to me. The test diff is misleading here IMO, because I wouldn't expect real code using reflection to use several accessors one after another like this, unless it was within a loop. Evidence to the contrary would be welcome :) This change also incidentally goes part way to fixing the issue of the JSON formatter not writing out the fields in field number order - with this change, it does except for oneofs, which we can fix in a follow-up change. I haven't actually added a test with a message with fields deliberately out of order - I'm happy to do so though. It feels like it would make sense to be in google/src/protobuf, but it's not entirely clear what the rules of engagement are for adding new messages there. (unittest_proto3.proto?) |