Clean-up attribute names and paranthesis

vmuriart · vmuriart · commit d1241f8afd54 · 2017-02-07T13:04:21.000-07:00
diff --git a/src/runtime/converter.cs b/src/runtime/converter.cs
@@ -10,7 +10,7 @@ namespace Python.Runtime
     /// <summary>
     /// Performs data conversions between managed types and Python types.
     /// </summary>
-    [SuppressUnmanagedCodeSecurityAttribute()]
+    [SuppressUnmanagedCodeSecurity]
     internal class Converter
     {
         private Converter()
diff --git a/src/runtime/interop.cs b/src/runtime/interop.cs
@@ -12,7 +12,7 @@ namespace Python.Runtime
     /// runtime. Generally, the definitions here need to be kept up to date
     /// when moving to new Python versions.
     /// </summary>
-    [Serializable()]
+    [Serializable]
     [AttributeUsage(AttributeTargets.All)]
     public class DocStringAttribute : Attribute
     {
@@ -30,7 +30,7 @@ public string DocString
         private string docStr;
     }
 
-    [Serializable()]
+    [Serializable]
     [AttributeUsage(AttributeTargets.Method | AttributeTargets.Delegate)]
     internal class PythonMethodAttribute : Attribute
     {
@@ -39,7 +39,7 @@ public PythonMethodAttribute()
         }
     }
 
-    [Serializable()]
+    [Serializable]
     [AttributeUsage(AttributeTargets.Method | AttributeTargets.Delegate)]
     internal class ModuleFunctionAttribute : Attribute
     {
@@ -48,7 +48,7 @@ public ModuleFunctionAttribute()
         }
     }
 
-    [Serializable()]
+    [Serializable]
     [AttributeUsage(AttributeTargets.Method | AttributeTargets.Delegate)]
     internal class ForbidPythonThreadsAttribute : Attribute
     {
@@ -58,7 +58,7 @@ public ForbidPythonThreadsAttribute()
     }
 
 
-    [Serializable()]
+    [Serializable]
     [AttributeUsage(AttributeTargets.Property)]
     internal class ModulePropertyAttribute : Attribute
     {
diff --git a/src/runtime/moduleobject.cs b/src/runtime/moduleobject.cs
@@ -349,34 +349,34 @@ internal void InitializePreload()
             }
         }
 
-        [ModuleFunctionAttribute]
+        [ModuleFunction]
         public static bool getPreload()
         {
             return preload;
         }
 
-        [ModuleFunctionAttribute]
+        [ModuleFunction]
         public static void setPreload(bool preloadFlag)
         {
             preload = preloadFlag;
         }
 
-        //[ModulePropertyAttribute]
+        //[ModuleProperty]
         public static bool SuppressDocs
         {
             get { return _SuppressDocs; }
             set { _SuppressDocs = value; }
         }
 
-        //[ModulePropertyAttribute]
+        //[ModuleProperty]
         public static bool SuppressOverloads
         {
             get { return _SuppressOverloads; }
             set { _SuppressOverloads = value; }
         }
 
-        [ModuleFunctionAttribute]
-        [ForbidPythonThreadsAttribute]
+        [ModuleFunction]
+        [ForbidPythonThreads]
         public static Assembly AddReference(string name)
         {
             AssemblyManager.UpdatePath();
@@ -402,15 +402,15 @@ public static Assembly AddReference(string name)
             return assembly;
         }
 
-        [ModuleFunctionAttribute]
-        [ForbidPythonThreadsAttribute]
+        [ModuleFunction]
+        [ForbidPythonThreads]
         public static string FindAssembly(string name)
         {
             AssemblyManager.UpdatePath();
             return AssemblyManager.FindAssembly(name);
         }
 
-        [ModuleFunctionAttribute]
+        [ModuleFunction]
         public static string[] ListAssemblies(bool verbose)
         {
             AssemblyName[] assnames = AssemblyManager.ListAssemblies();
@@ -429,7 +429,7 @@ public static string[] ListAssemblies(bool verbose)
             return names;
         }
 
-        [ModuleFunctionAttribute]
+        [ModuleFunction]
         public static int _AtExit()
         {
             return Runtime.AtExit();
diff --git a/src/runtime/runtime.cs b/src/runtime/runtime.cs
@@ -5,7 +5,7 @@
 
 namespace Python.Runtime
 {
-    [SuppressUnmanagedCodeSecurity()]
+    [SuppressUnmanagedCodeSecurity]
     static class NativeMethods
     {
 #if MONO_LINUX || MONO_OSX

Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ namespace Python.Runtime`
`10`	`10`	`/// <summary>`
`11`	`11`	`/// Performs data conversions between managed types and Python types.`
`12`	`12`	`/// </summary>`
`13`		`- [SuppressUnmanagedCodeSecurityAttribute()]`
	`13`	`+ [SuppressUnmanagedCodeSecurity]`
`14`	`14`	`internal class Converter`
`15`	`15`	`{`
`16`	`16`	`private Converter()`
Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@ namespace Python.Runtime`
`12`	`12`	`/// runtime. Generally, the definitions here need to be kept up to date`
`13`	`13`	`/// when moving to new Python versions.`
`14`	`14`	`/// </summary>`
`15`		`- [Serializable()]`
	`15`	`+ [Serializable]`
`16`	`16`	`[AttributeUsage(AttributeTargets.All)]`
`17`	`17`	`public class DocStringAttribute : Attribute`
`18`	`18`	`{`
`@@ -30,7 +30,7 @@ public string DocString`
`30`	`30`	`private string docStr;`
`31`	`31`	`}`
`32`	`32`
`33`		`- [Serializable()]`
	`33`	`+ [Serializable]`
`34`	`34`	`[AttributeUsage(AttributeTargets.Method \| AttributeTargets.Delegate)]`
`35`	`35`	`internal class PythonMethodAttribute : Attribute`
`36`	`36`	`{`
`@@ -39,7 +39,7 @@ public PythonMethodAttribute()`
`39`	`39`	`}`
`40`	`40`	`}`
`41`	`41`
`42`		`- [Serializable()]`
	`42`	`+ [Serializable]`
`43`	`43`	`[AttributeUsage(AttributeTargets.Method \| AttributeTargets.Delegate)]`
`44`	`44`	`internal class ModuleFunctionAttribute : Attribute`
`45`	`45`	`{`
`@@ -48,7 +48,7 @@ public ModuleFunctionAttribute()`
`48`	`48`	`}`
`49`	`49`	`}`
`50`	`50`
`51`		`- [Serializable()]`
	`51`	`+ [Serializable]`
`52`	`52`	`[AttributeUsage(AttributeTargets.Method \| AttributeTargets.Delegate)]`
`53`	`53`	`internal class ForbidPythonThreadsAttribute : Attribute`
`54`	`54`	`{`
`@@ -58,7 +58,7 @@ public ForbidPythonThreadsAttribute()`
`58`	`58`	`}`
`59`	`59`
`60`	`60`
`61`		`- [Serializable()]`
	`61`	`+ [Serializable]`
`62`	`62`	`[AttributeUsage(AttributeTargets.Property)]`
`63`	`63`	`internal class ModulePropertyAttribute : Attribute`
`64`	`64`	`{`
Original file line number	Diff line number	Diff line change
`@@ -349,34 +349,34 @@ internal void InitializePreload()`
`349`	`349`	`}`
`350`	`350`	`}`
`351`	`351`
`352`		`- [ModuleFunctionAttribute]`
	`352`	`+ [ModuleFunction]`
`353`	`353`	`public static bool getPreload()`
`354`	`354`	`{`
`355`	`355`	`return preload;`
`356`	`356`	`}`
`357`	`357`
`358`		`- [ModuleFunctionAttribute]`
	`358`	`+ [ModuleFunction]`
`359`	`359`	`public static void setPreload(bool preloadFlag)`
`360`	`360`	`{`
`361`	`361`	`preload = preloadFlag;`
`362`	`362`	`}`
`363`	`363`
`364`		`- //[ModulePropertyAttribute]`
	`364`	`+ //[ModuleProperty]`
`365`	`365`	`public static bool SuppressDocs`
`366`	`366`	`{`
`367`	`367`	`get { return _SuppressDocs; }`
`368`	`368`	`set { _SuppressDocs = value; }`
`369`	`369`	`}`
`370`	`370`
`371`		`- //[ModulePropertyAttribute]`
	`371`	`+ //[ModuleProperty]`
`372`	`372`	`public static bool SuppressOverloads`
`373`	`373`	`{`
`374`	`374`	`get { return _SuppressOverloads; }`
`375`	`375`	`set { _SuppressOverloads = value; }`
`376`	`376`	`}`
`377`	`377`
`378`		`- [ModuleFunctionAttribute]`
`379`		`- [ForbidPythonThreadsAttribute]`
	`378`	`+ [ModuleFunction]`
	`379`	`+ [ForbidPythonThreads]`
`380`	`380`	`public static Assembly AddReference(string name)`
`381`	`381`	`{`
`382`	`382`	`AssemblyManager.UpdatePath();`
`@@ -402,15 +402,15 @@ public static Assembly AddReference(string name)`
`402`	`402`	`return assembly;`
`403`	`403`	`}`
`404`	`404`
`405`		`- [ModuleFunctionAttribute]`
`406`		`- [ForbidPythonThreadsAttribute]`
	`405`	`+ [ModuleFunction]`
	`406`	`+ [ForbidPythonThreads]`
`407`	`407`	`public static string FindAssembly(string name)`
`408`	`408`	`{`
`409`	`409`	`AssemblyManager.UpdatePath();`
`410`	`410`	`return AssemblyManager.FindAssembly(name);`
`411`	`411`	`}`
`412`	`412`
`413`		`- [ModuleFunctionAttribute]`
	`413`	`+ [ModuleFunction]`
`414`	`414`	`public static string[] ListAssemblies(bool verbose)`
`415`	`415`	`{`
`416`	`416`	`AssemblyName[] assnames = AssemblyManager.ListAssemblies();`
`@@ -429,7 +429,7 @@ public static string[] ListAssemblies(bool verbose)`
`429`	`429`	`return names;`
`430`	`430`	`}`
`431`	`431`
`432`		`- [ModuleFunctionAttribute]`
	`432`	`+ [ModuleFunction]`
`433`	`433`	`public static int _AtExit()`
`434`	`434`	`{`
`435`	`435`	`return Runtime.AtExit();`
Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@`
`5`	`5`
`6`	`6`	`namespace Python.Runtime`
`7`	`7`	`{`
`8`		`- [SuppressUnmanagedCodeSecurity()]`
	`8`	`+ [SuppressUnmanagedCodeSecurity]`
`9`	`9`	`static class NativeMethods`
`10`	`10`	`{`
`11`	`11`	`#if MONO_LINUX \|\| MONO_OSX`