Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge pull request #738 from jtattermusch/include_descriptor_proto | Jon Skeet | 2015-08-26 | 1 | -0/+1 |
|\ | | | | | Include descriptor.proto in nuget package | ||||
| * | include descriptor.proto in nuget package | Jan Tattermusch | 2015-08-25 | 1 | -0/+1 |
| | | |||||
* | | Change where we rename Descriptor.cs to DescriptorProtoFile.cs. | Jon Skeet | 2015-08-25 | 2 | -114/+114 |
|/ | | | | | | | | | | | We now do this in protoc instead of the generation simpler. Benefits: - Generation script is simpler - Detection is simpler as we now only need to care about one filename - The embedded descriptor knows itself as "google/protobuf/descriptor.proto" avoiding dependency issues This PR also makes the "invalid dependency" exception clearer in terms of expected and actual dependencies. | ||||
* | fix type initialization problem with FileDescriptor | Jan Tattermusch | 2015-08-14 | 1 | -148/+175 |
| | |||||
* | Allow public access to descriptor.proto as a dependency. | Jon Skeet | 2015-08-13 | 1 | -0/+15 |
| | | | | | With this in place, generating APIs on github.com/google/googleapis works - previously annotations.proto failed. Currently there's no access to the annotations (stored as extensions) but we could potentially expose those at a later date. | ||||
* | More TODOs done. | Jon Skeet | 2015-08-10 | 5 | -100/+144 |
| | | | | | | - Removed a TODO without change in DescriptorPool.LookupSymbol - the TODOs were around performance, and this is only used during descriptor initialization - Make the CodedInputStream limits read-only, adding a static factory method for the rare cases when this is useful - Extracted IDeepCloneable into its own file. | ||||
* | Merge pull request #701 from jskeet/map-views | Jon Skeet | 2015-08-10 | 1 | -4/+95 |
|\ | | | | | Implement Keys and Values as views in MapField | ||||
| * | Fix parameter name in exception. | Jon Skeet | 2015-08-10 | 1 | -1/+1 |
| | | |||||
| * | Implement ICollection.CopyTo (using Array) for MapField views. | Jon Skeet | 2015-08-10 | 1 | -1/+12 |
| | | |||||
| * | Implement Keys and Values as views | Jon Skeet | 2015-08-08 | 1 | -4/+84 |
| | | |||||
* | | Make FieldDescriptor.IsPacked work appropriately. | Jon Skeet | 2015-08-08 | 2 | -3/+17 |
|/ | | | | | This is a bit of a grotty hack, as we need to sort of fake proto2 field presence, but with only a proto3 version of the descriptor messages (a bit like oneof detection). Should be okay, but will need to be careful of this if we ever implement proto2. | ||||
* | 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 | 16 | -101/+84 |
| | |||||
* | Address review comments. | Jon Skeet | 2015-08-08 | 1 | -2/+2 |
| | |||||
* | Generated code for previous commit. | Jon Skeet | 2015-08-06 | 12 | -188/+53 |
| | |||||
* | Skip groups properly. | Jon Skeet | 2015-08-06 | 5 | -32/+51 |
| | | | | | | | | | | | 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 | 11 | -141/+47 |
| | |||||
* | Change ReadTag and PeekTag to just use 0 as a return value for "end of ↵ | Jon Skeet | 2015-08-05 | 3 | -42/+26 |
| | | | | | | | | stream", rather than using an awkward out parameter. This simplifies quite a lot of code. Generated code in next commit. | ||||
* | Validate that after reading a message, we've consumed as many bytes as we ↵ | Jon Skeet | 2015-08-05 | 1 | -0/+29 |
| | | | | | | expected to. We should now have no conformance failures. | ||||
* | Generated code changes for previous commit. | Jon Skeet | 2015-08-05 | 11 | -0/+47 |
| | |||||
* | Consume unknown fields when parsing. | Jon Skeet | 2015-08-05 | 1 | -0/+33 |
| | | | | | This is expected to be the cause of the conformance test failures. Generated code in next commit. | ||||
* | 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 | 1 | -1/+0 |
|/ | |||||
* | Merge pull request #681 from jskeet/json-fieldmask | Jon Skeet | 2015-08-03 | 1 | -1/+21 |
|\ | | | | | JSON formatting for FieldMask | ||||
| * | JSON formatting for FieldMask | Jon Skeet | 2015-08-03 | 1 | -1/+21 |
| | | |||||
* | | 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 | 4 | -60/+33 |
|/ | |||||
* | Merge pull request #675 from jtattermusch/signing | Jon Skeet | 2015-08-03 | 3 | -31/+46 |
|\ | | | | | Add ReleaseSigned configuration for C# | ||||
| * | make changes and fixes in signing | Jan Tattermusch | 2015-08-02 | 3 | -24/+6 |
| | | |||||
| * | 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 | 3 | -12/+31 |
| | | |||||
* | | Initial pass at formatting Struct as JSON. | Jon Skeet | 2015-08-03 | 1 | -3/+81 |
| | | | | | | | | 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 | 2 | -11/+121 |
|/ | | | | 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. | ||||
* | Fix JSON formatting to always emit fields in field order, including oneofs | Jon Skeet | 2015-07-31 | 1 | -30/+5 |
| | |||||
* | Fix trivial bug in field orderings. | Jon Skeet | 2015-07-31 | 1 | -1/+1 |
| | | | | (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 | 4 | -0/+373 |
| | | | | | 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. | ||||
* | Generated code for previous commit. | Jon Skeet | 2015-07-30 | 7 | -43/+43 |
| | |||||
* | 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 | 11 | -141/+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 | 16 | -654/+6 |
|\ | | | | | Remove the C# Freeze API | ||||
| * | regenerate code | Jan Tattermusch | 2015-07-29 | 11 | -504/+0 |
| | | |||||
| * | remove the freeze API | Jan Tattermusch | 2015-07-29 | 5 | -150/+6 |
| | |