Fixed code style.

Author: gfraiteur

.editorconfig

@@ -141,6 +141,6 @@ csharp_space_between_parentheses = false
 csharp_space_between_square_brackets = false
 
 # Wrapping preferences
-csharp_preserve_single_line_blocks = true
-csharp_preserve_single_line_statements = true
+csharp_preserve_single_line_blocks = false
+csharp_preserve_single_line_statements = false
 

Diagnostics/PostSharp.Samples.Audit.Extended/DbAuditRecord.cs

@@ -29,7 +29,9 @@ public void AppendToDatabase()
       Console.WriteLine(
         $"Writing to the database: {{PrimaryBusinessObjectId={PrimaryBusinessObjectId}, Operation={Method}, Description=\"{Description}\", User={User}}}.");
       foreach (var id in RelatedBusinessObjectIds)
+      {
         Console.WriteLine($"Writing to database: correlation with BusinessObjectId={id}.");
+      }
     }
   }
 }

Diagnostics/PostSharp.Samples.Audit.Extended/ExtendedAuditRecordBuilder.cs

@@ -28,7 +28,9 @@ public override void SetParameter<T>(
       var businessObject = value as BusinessObject;
 
       if (businessObject != null)
+      {
         ((ExtendedAuditRecord) CurrentRecord).RelatedBusinessObjects.Add(businessObject);
+      }
     }
   }
 }

Diagnostics/PostSharp.Samples.Logging.BusinessLogic/QueueProcessor.cs

@@ -33,10 +33,13 @@ private static void ProcessItem(QueueItem item)
         }
 
         if (item.Id == 145)
+        {
           RequestStorage.GetUser(0);
+        }
         else
+        {
           RequestStorage.GetUser(14);
-
+        }
 
         Thread.Sleep(125);
 

Diagnostics/PostSharp.Samples.Logging.BusinessLogic/RequestStorage.cs

@@ -13,9 +13,15 @@ public static Request GetRequest(int id)
 
     public static User GetUser(int id)
     {
-      if (id <= 0) throw new ArgumentOutOfRangeException("id", "The user id must be greater than zero.");
+      if (id <= 0)
+      {
+        throw new ArgumentOutOfRangeException("id", "The user id must be greater than zero.");
+      }
 
-      if (id == 14) Thread.Sleep(56);
+      if (id == 14)
+      {
+        Thread.Sleep(56);
+      }
 
       return new User();
     }

Diagnostics/PostSharp.Samples.Logging.CustomBackend.ServiceStack/ServiceStackLogRecordBuilder.cs

@@ -24,37 +24,62 @@ protected override void Write(UnsafeString message)
         case LogLevel.Trace:
         case LogLevel.Debug:
           if (Exception == null)
+          {
             log.Debug(messageString);
+          }
           else
+          {
             log.Debug(messageString, Exception);
+          }
+
           break;
 
         case LogLevel.Info:
           if (Exception == null)
+          {
             log.Info(messageString);
+          }
           else
+          {
             log.Info(messageString, Exception);
+          }
+
           break;
 
         case LogLevel.Warning:
           if (Exception == null)
+          {
             log.Warn(messageString);
+          }
           else
+          {
             log.Warn(messageString, Exception);
+          }
+
           break;
 
         case LogLevel.Error:
           if (Exception == null)
+          {
             log.Error(messageString);
+          }
           else
+          {
             log.Error(messageString, Exception);
+          }
+
           break;
 
         case LogLevel.Critical:
           if (Exception == null)
+          {
             log.Fatal(messageString);
+          }
           else
+          {
             log.Fatal(messageString, Exception);
+          }
+
           break;
       }
     }

Diagnostics/PostSharp.Samples.Logging.ElasticStack/ClientExample/InstrumentOutgoingRequestsAspect.cs

@@ -50,7 +50,10 @@ public override async Task OnInvokeAsync(MethodInterceptionArgs args)
           {
             activity.Context.ForEachProperty((LoggingProperty property, object value, ref object _) =>
             {
-              if (!property.IsBaggage || !propertyNames.Add(property.Name)) return;
+              if (!property.IsBaggage || !propertyNames.Add(property.Name))
+              {
+                return;
+              }
 
               if (correlationContextBuilder == null)
               {

Diagnostics/PostSharp.Samples.Logging.ElasticStack/MicroserviceExample/LoggingActionFilter.cs

@@ -39,7 +39,11 @@ public async Task OnActionExecutionAsync(ActionExecutingContext context, ActionE
         foreach (var pair in correlationContext.Split(',', StringSplitOptions.RemoveEmptyEntries))
         {
           var posOfEqual = pair.IndexOf('=');
-          if (posOfEqual <= 0) continue;
+          if (posOfEqual <= 0)
+          {
+            continue;
+          }
+
           var propertyName = pair.Substring(0, posOfEqual);
           var propertyValue = pair.Substring(posOfEqual + 1);
           properties.Add(new LoggingProperty(propertyName, propertyValue) { IsBaggage = true });

Diagnostics/PostSharp.Samples.Logging.Etw.CustomSource/Program.cs

@@ -14,7 +14,9 @@ private static void Main(string[] args)
     {
       var eventSourceBackend = new EventSourceLoggingBackend(new MyEventSource());
       if (eventSourceBackend.EventSource.ConstructionException != null)
+      {
         throw eventSourceBackend.EventSource.ConstructionException;
+      }
 
       LoggingServices.DefaultBackend = eventSourceBackend;
 

Diagnostics/PostSharp.Samples.Logging.Etw/Program.cs

@@ -14,7 +14,9 @@ private static void Main(string[] args)
     {
       var eventSourceBackend = new EventSourceLoggingBackend(new PostSharpEventSource());
       if (eventSourceBackend.EventSource.ConstructionException != null)
+      {
         throw eventSourceBackend.EventSource.ConstructionException;
+      }
 
       LoggingServices.DefaultBackend = eventSourceBackend;
 

Framework/PostSharp.Samples.Authorization/BusinessObjects/Entity.cs

@@ -23,11 +23,20 @@ protected Entity()
     public virtual bool HasRole(ISubject subject, IRole role)
     {
       if (role.Equals(Role.Owner))
+      {
         return subject.Equals(Owner);
+      }
+
       if (UserRoleAssignments.Any(a => a.Role.Equals(role) && a.Subject.Equals(subject)))
+      {
         return true;
+      }
+
       if (SecurityParent != null)
+      {
         return SecurityParent.HasRole(subject, role);
+      }
+
       return false;
     }
   }

Framework/PostSharp.Samples.Authorization/BusinessObjects/Role.cs

@@ -37,9 +37,21 @@ public bool Equals(Role other)
 
     public override bool Equals(object obj)
     {
-      if (ReferenceEquals(null, obj)) return false;
-      if (ReferenceEquals(this, obj)) return true;
-      if (obj.GetType() != GetType()) return false;
+      if (ReferenceEquals(null, obj))
+      {
+        return false;
+      }
+
+      if (ReferenceEquals(this, obj))
+      {
+        return true;
+      }
+
+      if (obj.GetType() != GetType())
+      {
+        return false;
+      }
+
       return Equals((Role) obj);
     }
 

Framework/PostSharp.Samples.Authorization/BusinessObjects/User.cs

@@ -22,8 +22,16 @@ bool IEquatable<ISubject>.Equals(ISubject other)
 
     public override bool Equals(object obj)
     {
-      if (ReferenceEquals(null, obj)) return false;
-      if (ReferenceEquals(this, obj)) return true;
+      if (ReferenceEquals(null, obj))
+      {
+        return false;
+      }
+
+      if (ReferenceEquals(this, obj))
+      {
+        return true;
+      }
+
       return Id.Equals(((User) obj).Id);
     }
 

Framework/PostSharp.Samples.Authorization/Framework/ApplyDefaultPermissionsAttribute.cs

@@ -28,7 +28,9 @@ IEnumerable<AspectInstance> IAspectProvider.ProvideAspects(object targetElement)
         targetType.GetProperties(BindingFlags.Instance | BindingFlags.Public).Cast<MemberInfo>()))
       {
         if (location.IsDefined(typeof(RequiresPermissionBaseAttribute), true))
+        {
           continue;
+        }
 
         var aspect = new LocationAuthorizationAspect();
         aspect.AddPermission(0, permissionFactory);

Framework/PostSharp.Samples.Authorization/Framework/AuthorizationAspect.cs

@@ -53,6 +53,7 @@ internal void InitializePermissions(int parameterCount, OperationSemantic[] sema
 
       var i = 0;
       foreach (var permissionFactory in _permissionFactories)
+      {
         foreach (var semantic in semantics)
         {
           _permissions[i] = new OperationPermission<IPermission>(semantic, permissionFactory.ParameterIndex,
@@ -61,6 +62,7 @@ internal void InitializePermissions(int parameterCount, OperationSemantic[] sema
           _hasPermissionForParameter[permissionFactory.ParameterIndex] = true;
           i++;
         }
+      }
     }
 
     /// <summary>
@@ -89,43 +91,60 @@ internal bool HasPermissionForParameter(int parameterIndex)
     internal void RequirePermission(MemberInfo member, OperationSemantic semantic, int parameterIndex, object securable)
     {
       if (evaluatingPermissions)
+      {
         return;
+      }
 
       if (SecurityContext.Current == null)
+      {
         return;
+      }
 
       var subject = SecurityContext.Current.Subject;
       var policy = SecurityContext.Current.Policy;
 
       if (policy == null)
+      {
         return;
-
+      }
 
       try
       {
         evaluatingPermissions = true;
 
         foreach (var permission in _permissions)
+        {
           if ((permission.Semantic == OperationSemantic.Default || permission.Semantic == semantic) &&
               permission.ParameterIndex == parameterIndex)
+          {
             if (!policy.Evaluate(subject, permission.Permission, securable))
             {
               SecurityContext.Current.ExceptionHandler?.OnSecurityException(member, semantic, securable,
                 SecurityContext.Current.Subject, permission.Permission);
 
               string memberKind;
               if (member is FieldInfo)
+              {
                 memberKind = "field";
+              }
               else if (member is PropertyInfo)
+              {
                 memberKind = "property";
+              }
               else if (member is MethodBase)
+              {
                 memberKind = "method";
+              }
               else
+              {
                 throw new ArgumentOutOfRangeException(nameof(member));
+              }
 
               throw new SecurityException(
                 $"Cannot {semantic.ToString().ToLowerInvariant()} the {memberKind} {member.DeclaringType.Name}.{member.Name}: the subject '{subject.Name}' does not have the {permission.Permission.Name} permission on the object '{securable}'.");
             }
+          }
+        }
       }
       finally
       {

Framework/PostSharp.Samples.Authorization/Framework/MethodAuthorizationAspect.cs

@@ -32,10 +32,14 @@ public void RuntimeInitialize(MethodBase method)
     public void OnEntry(MethodExecutionArgs args)
     {
       if (HasPermissionForParameter(0))
+      {
         RequirePermission(args.Method, OperationSemantic.Default, 0, args.Instance);
+      }
 
       for (var i = 0; i < args.Arguments.Count; i++)
+      {
         RequirePermission(args.Method, OperationSemantic.Default, i + 1, args.Arguments[i]);
+      }
     }
 
     public void OnExit(MethodExecutionArgs args)

Framework/PostSharp.Samples.Authorization/Framework/Permission.cs

@@ -24,16 +24,36 @@ public Permission(string name)
 
     bool IEquatable<IPermission>.Equals(IPermission other)
     {
-      if (ReferenceEquals(null, other)) return false;
-      if (ReferenceEquals(this, other)) return true;
+      if (ReferenceEquals(null, other))
+      {
+        return false;
+      }
+
+      if (ReferenceEquals(this, other))
+      {
+        return true;
+      }
+
       return string.Equals(Name, other.Name);
     }
 
     public override bool Equals(object obj)
     {
-      if (ReferenceEquals(null, obj)) return false;
-      if (ReferenceEquals(this, obj)) return true;
-      if (obj.GetType() != GetType()) return false;
+      if (ReferenceEquals(null, obj))
+      {
+        return false;
+      }
+
+      if (ReferenceEquals(this, obj))
+      {
+        return true;
+      }
+
+      if (obj.GetType() != GetType())
+      {
+        return false;
+      }
+
       return Equals((Permission) obj);
     }