Skip to content

Commit

Permalink
Merge pull request #6669 from ObsidianMinor/csharp/fix-comments
Browse files Browse the repository at this point in the history
Fix incorrect Proto2 C# doc comments
  • Loading branch information
jtattermusch committed Sep 19, 2019
2 parents 2a46579 + c54ff8d commit 3782317
Show file tree
Hide file tree
Showing 7 changed files with 14 additions and 14 deletions.
4 changes: 2 additions & 2 deletions csharp/src/Google.Protobuf/Reflection/EnumValueDescriptor.cs
Expand Up @@ -77,7 +77,7 @@ public sealed class EnumValueDescriptor : DescriptorBase
public CustomOptions CustomOptions => new CustomOptions(Proto.Options._extensions?.ValuesByNumber);

/// <summary>
/// Gets a single value enum option for this descriptor
/// Gets a single value enum value option for this descriptor
/// </summary>
public T GetOption<T>(Extension<EnumValueOptions, T> extension)
{
Expand All @@ -86,7 +86,7 @@ public T GetOption<T>(Extension<EnumValueOptions, T> extension)
}

/// <summary>
/// Gets a repeated value enum option for this descriptor
/// Gets a repeated value enum value option for this descriptor
/// </summary>
public RepeatedField<T> GetOption<T>(RepeatedExtension<EnumValueOptions, T> extension)
{
Expand Down
4 changes: 2 additions & 2 deletions csharp/src/Google.Protobuf/Reflection/FieldDescriptor.cs
Expand Up @@ -302,7 +302,7 @@ public MessageDescriptor ExtendeeType
public CustomOptions CustomOptions => new CustomOptions(Proto.Options._extensions?.ValuesByNumber);

/// <summary>
/// Gets a single value enum option for this descriptor
/// Gets a single value field option for this descriptor
/// </summary>
public T GetOption<T>(Extension<FieldOptions, T> extension)
{
Expand All @@ -311,7 +311,7 @@ public T GetOption<T>(Extension<FieldOptions, T> extension)
}

/// <summary>
/// Gets a repeated value enum option for this descriptor
/// Gets a repeated value field option for this descriptor
/// </summary>
public RepeatedField<T> GetOption<T>(RepeatedExtension<FieldOptions, T> extension)
{
Expand Down
4 changes: 2 additions & 2 deletions csharp/src/Google.Protobuf/Reflection/FileDescriptor.cs
Expand Up @@ -545,7 +545,7 @@ public override string ToString()
public CustomOptions CustomOptions => new CustomOptions(Proto.Options._extensions?.ValuesByNumber);

/// <summary>
/// Gets a single value enum option for this descriptor
/// Gets a single value file option for this descriptor
/// </summary>
public T GetOption<T>(Extension<FileOptions, T> extension)
{
Expand All @@ -554,7 +554,7 @@ public T GetOption<T>(Extension<FileOptions, T> extension)
}

/// <summary>
/// Gets a repeated value enum option for this descriptor
/// Gets a repeated value file option for this descriptor
/// </summary>
public RepeatedField<T> GetOption<T>(RepeatedExtension<FileOptions, T> extension)
{
Expand Down
4 changes: 2 additions & 2 deletions csharp/src/Google.Protobuf/Reflection/MessageDescriptor.cs
Expand Up @@ -264,7 +264,7 @@ internal bool IsExtensionsInitialized(IMessage message)
public CustomOptions CustomOptions => new CustomOptions(Proto.Options._extensions?.ValuesByNumber);

/// <summary>
/// Gets a single value enum option for this descriptor
/// Gets a single value message option for this descriptor
/// </summary>
public T GetOption<T>(Extension<MessageOptions, T> extension)
{
Expand All @@ -273,7 +273,7 @@ public T GetOption<T>(Extension<MessageOptions, T> extension)
}

/// <summary>
/// Gets a repeated value enum option for this descriptor
/// Gets a repeated value message option for this descriptor
/// </summary>
public Collections.RepeatedField<T> GetOption<T>(RepeatedExtension<MessageOptions, T> extension)
{
Expand Down
4 changes: 2 additions & 2 deletions csharp/src/Google.Protobuf/Reflection/MethodDescriptor.cs
Expand Up @@ -77,7 +77,7 @@ public sealed class MethodDescriptor : DescriptorBase
public CustomOptions CustomOptions => new CustomOptions(Proto.Options._extensions?.ValuesByNumber);

/// <summary>
/// Gets a single value enum option for this descriptor
/// Gets a single value method option for this descriptor
/// </summary>
public T GetOption<T>(Extension<MethodOptions, T> extension)
{
Expand All @@ -86,7 +86,7 @@ public T GetOption<T>(Extension<MethodOptions, T> extension)
}

/// <summary>
/// Gets a repeated value enum option for this descriptor
/// Gets a repeated value method option for this descriptor
/// </summary>
public RepeatedField<T> GetOption<T>(RepeatedExtension<MethodOptions, T> extension)
{
Expand Down
4 changes: 2 additions & 2 deletions csharp/src/Google.Protobuf/Reflection/OneofDescriptor.cs
Expand Up @@ -109,7 +109,7 @@ public MessageDescriptor ContainingType
public CustomOptions CustomOptions => new CustomOptions(proto.Options._extensions?.ValuesByNumber);

/// <summary>
/// Gets a single value enum option for this descriptor
/// Gets a single value oneof option for this descriptor
/// </summary>
public T GetOption<T>(Extension<OneofOptions, T> extension)
{
Expand All @@ -118,7 +118,7 @@ public T GetOption<T>(Extension<OneofOptions, T> extension)
}

/// <summary>
/// Gets a repeated value enum option for this descriptor
/// Gets a repeated value oneof option for this descriptor
/// </summary>
public RepeatedField<T> GetOption<T>(RepeatedExtension<OneofOptions, T> extension)
{
Expand Down
4 changes: 2 additions & 2 deletions csharp/src/Google.Protobuf/Reflection/ServiceDescriptor.cs
Expand Up @@ -98,7 +98,7 @@ public MethodDescriptor FindMethodByName(String name)
public CustomOptions CustomOptions => new CustomOptions(Proto.Options._extensions?.ValuesByNumber);

/// <summary>
/// Gets a single value enum option for this descriptor
/// Gets a single value service option for this descriptor
/// </summary>
public T GetOption<T>(Extension<ServiceOptions, T> extension)
{
Expand All @@ -107,7 +107,7 @@ public T GetOption<T>(Extension<ServiceOptions, T> extension)
}

/// <summary>
/// Gets a repeated value enum option for this descriptor
/// Gets a repeated value service option for this descriptor
/// </summary>
public RepeatedField<T> GetOption<T>(RepeatedExtension<ServiceOptions, T> extension)
{
Expand Down

0 comments on commit 3782317

Please sign in to comment.