Remove LayoutKind.Sequential attributes

These are the default in C# anyway, so it's just noise.
dotnet · Jul 1, 2020 · 0147a6f · 0147a6f
1 parent 8fa0596
commit 0147a6f
Show file tree

Hide file tree

Showing 125 changed files with 6 additions and 128 deletions.
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -196,8 +196,13 @@ anything else found in native header files for these reasons:
 | `WPARAM`    | `IntPtr`  |
 | `LRESULT`   | `IntPtr`  |
 
+### Structs
 
-### Struct field types
+Tip: Avoid adding `[StructLayout(LayoutKind.Sequential)]` to your structs,
+as [this is the default in C#](https://docs.microsoft.com/en-us/dotnet/api/system.runtime.interopservices.structlayoutattribute?view=netcore-3.1#remarks) and thus just more verbose than necessary.
+You may need the attribute to add additional properties however, such as forcing a particular `Pack` property or `CharSet`.
+
+#### Field types
 
 When defining a struct, we should take care to make sure the struct is 'pinnable' (i.e. all fields must be
 value types rather than reference types.) Benefits of structs being pinnable include:

diff --git a/src/AdvApi32/AdvApi32+ENUM_SERVICE_STATUS.cs b/src/AdvApi32/AdvApi32+ENUM_SERVICE_STATUS.cs
@@ -14,7 +14,6 @@ public partial class AdvApi32
         /// Contains the name of a service in a service control manager database and information about that service.
         /// It is used by the EnumDependentServices and EnumServicesStatus functions.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential, CharSet = CharSet.Unicode)]
         public struct ENUM_SERVICE_STATUS
         {
             /// <summary>

diff --git a/src/AdvApi32/AdvApi32+SC_ACTION.cs b/src/AdvApi32/AdvApi32+SC_ACTION.cs
@@ -13,7 +13,6 @@ public static partial class AdvApi32
         /// <summary>
         /// Represents an action that the service control manager can perform.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct SC_ACTION
         {
             /// <summary>

diff --git a/src/AdvApi32/AdvApi32+SERVICE_FAILURE_ACTIONS.cs b/src/AdvApi32/AdvApi32+SERVICE_FAILURE_ACTIONS.cs
@@ -13,7 +13,6 @@ public static partial class AdvApi32
         /// <summary>
         /// Represents the action the service controller should take on each failure of a service. A service is considered failed when it terminates without reporting a status of SERVICE_STOPPED to the service controller.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential, CharSet = CharSet.Unicode)]
         [OfferIntPtrPropertyAccessors]
         public unsafe partial struct SERVICE_FAILURE_ACTIONS
         {

diff --git a/src/AdvApi32/AdvApi32+SERVICE_STATUS.cs b/src/AdvApi32/AdvApi32+SERVICE_STATUS.cs
@@ -15,7 +15,6 @@ public partial class AdvApi32
         /// The ControlService, EnumDependentServices, EnumServicesStatus, and QueryServiceStatus functions use this structure.
         /// A service uses this structure in the <see cref="SetServiceStatus"/> function to report its current status to the service control manager.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential, CharSet = CharSet.Unicode)]
         public struct SERVICE_STATUS
         {
             /// <summary>

diff --git a/src/AdvApi32/AdvApi32+SERVICE_STATUS_PROCESS.cs b/src/AdvApi32/AdvApi32+SERVICE_STATUS_PROCESS.cs
@@ -15,7 +15,6 @@ public partial class AdvApi32
         /// The ControlService, EnumDependentServices, EnumServicesStatus, and QueryServiceStatus functions use this structure.
         /// A service uses this structure in the <see cref="SetServiceStatus"/> function to report its current status to the service control manager.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential, CharSet = CharSet.Unicode)]
         public struct SERVICE_STATUS_PROCESS
         {
             /// <summary>

diff --git a/src/AdvApi32/AdvApi32+SERVICE_TRIGGER_SPECIFIC_DATA_ITEM.cs b/src/AdvApi32/AdvApi32+SERVICE_TRIGGER_SPECIFIC_DATA_ITEM.cs
@@ -15,7 +15,6 @@ public partial class AdvApi32
         /// Contains trigger-specific data for a service trigger event.
         /// This structure is used by the <see cref="ServiceTrigger"/> structure for <see cref="ServiceTriggerType.SERVICE_TRIGGER_TYPE_CUSTOM"/>, <see cref="ServiceTriggerType.SERVICE_TRIGGER_TYPE_DEVICE_INTERFACE_ARRIVAL"/>, <see cref="ServiceTriggerType.SERVICE_TRIGGER_TYPE_FIREWALL_PORT_EVENT"/>, or <see cref="ServiceTriggerType.SERVICE_TRIGGER_TYPE_NETWORK_ENDPOINT"/> trigger events.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct SERVICE_TRIGGER_SPECIFIC_DATA_ITEM
         {
             /// <summary>

diff --git a/src/AdvApi32/AdvApi32+ServiceDelayedAutoStartInfo.cs b/src/AdvApi32/AdvApi32+ServiceDelayedAutoStartInfo.cs
@@ -13,7 +13,6 @@ public static partial class AdvApi32
         /// <summary>
         /// Contains the delayed auto-start setting of an auto-start service.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct ServiceDelayedAutoStartInfo
         {
             /// <summary>

diff --git a/src/AdvApi32/AdvApi32+ServiceDescription.cs b/src/AdvApi32/AdvApi32+ServiceDescription.cs
@@ -13,7 +13,6 @@ public static partial class AdvApi32
         /// <summary>
         /// Contains a service description.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct ServiceDescription
         {
             /// <summary>

diff --git a/src/AdvApi32/AdvApi32+ServiceLaunchProtectedInfo.cs b/src/AdvApi32/AdvApi32+ServiceLaunchProtectedInfo.cs
@@ -13,7 +13,6 @@ public static partial class AdvApi32
         /// <summary>
         /// Indicates a service protection type.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct ServiceLaunchProtectedInfo
         {
             /// <summary>

diff --git a/src/AdvApi32/AdvApi32+ServicePreferredNodeInfo.cs b/src/AdvApi32/AdvApi32+ServicePreferredNodeInfo.cs
@@ -13,7 +13,6 @@ public static partial class AdvApi32
         /// <summary>
         /// Represents the preferred node on which to run a service.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct ServicePreferredNodeInfo
         {
             /// <summary>

diff --git a/src/AdvApi32/AdvApi32+ServicePreshutdownInfo.cs b/src/AdvApi32/AdvApi32+ServicePreshutdownInfo.cs
@@ -13,7 +13,6 @@ public static partial class AdvApi32
         /// <summary>
         /// Contains preshutdown settings
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct ServicePreshutdownInfo
         {
             /// <summary>

diff --git a/src/AdvApi32/AdvApi32+ServiceRequiredPrivilegesInfo.cs b/src/AdvApi32/AdvApi32+ServiceRequiredPrivilegesInfo.cs
@@ -13,7 +13,6 @@ public static partial class AdvApi32
         /// <summary>
         /// Represents the required privileges for a service.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct ServiceRequiredPrivilegesInfo
         {
             /// <summary>

diff --git a/src/AdvApi32/AdvApi32+ServiceSidInfo.cs b/src/AdvApi32/AdvApi32+ServiceSidInfo.cs
@@ -13,7 +13,6 @@ public static partial class AdvApi32
         /// <summary>
         /// Represents a service security identifier (SID).
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct ServiceSidInfo
         {
             /// <summary>

diff --git a/src/AdvApi32/AdvApi32+ServiceTrigger.cs b/src/AdvApi32/AdvApi32+ServiceTrigger.cs
@@ -14,7 +14,6 @@ public static partial class AdvApi32
         /// <summary>
         /// Represents a service trigger event. This structure is used by the <see cref="ServiceTriggerInfo"/> structure.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct ServiceTrigger
         {
             /// <summary>

diff --git a/src/AdvApi32/AdvApi32+ServiceTriggerInfo.cs b/src/AdvApi32/AdvApi32+ServiceTriggerInfo.cs
@@ -14,7 +14,6 @@ public static partial class AdvApi32
         /// <summary>
         /// Contains trigger event information for a service. This structure is used by the <see cref="ChangeServiceConfig2(SafeServiceHandle, ServiceInfoLevel, void*)"/> and QueryServiceConfig2 functions.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct ServiceTriggerInfo
         {
             /// <summary>

diff --git a/src/BCrypt/BCrypt+BCRYPT_ALGORITHM_IDENTIFIER.cs b/src/BCrypt/BCrypt+BCRYPT_ALGORITHM_IDENTIFIER.cs
@@ -13,7 +13,6 @@ public partial class BCrypt
         /// <summary>
         /// Used with the <see cref="BCryptEnumAlgorithms(AlgorithmOperations, out int, out BCRYPT_ALGORITHM_IDENTIFIER*, BCryptEnumAlgorithmsFlags)"/> function to contain a cryptographic algorithm identifier.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential, CharSet = CharSet.Unicode)]
         [OfferIntPtrPropertyAccessors]
         public unsafe partial struct BCRYPT_ALGORITHM_IDENTIFIER
         {

diff --git a/src/BCrypt/BCrypt+BCRYPT_AUTHENTICATED_CIPHER_MODE_INFO.cs b/src/BCrypt/BCrypt+BCRYPT_AUTHENTICATED_CIPHER_MODE_INFO.cs
@@ -15,7 +15,6 @@ public partial class BCrypt
         /// Used with the <see cref="BCryptEncrypt(SafeKeyHandle, byte[], void*, byte[], BCryptEncryptFlags)"/> and <see cref="BCryptDecrypt(SafeKeyHandle, byte[], void*, byte[], BCryptEncryptFlags)"/> functions
         /// to contain additional information related to authenticated cipher modes.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         [OfferIntPtrPropertyAccessors]
         public unsafe partial struct BCRYPT_AUTHENTICATED_CIPHER_MODE_INFO
         {

diff --git a/src/BCrypt/BCrypt+BCRYPT_AUTH_TAG_LENGTHS_STRUCT.cs b/src/BCrypt/BCrypt+BCRYPT_AUTH_TAG_LENGTHS_STRUCT.cs
@@ -14,7 +14,6 @@ public partial class BCrypt
         /// <summary>
         /// defines the range of tag sizes that are supported by the provider. This structure is used with the <see cref="PropertyNames.BCRYPT_AUTH_TAG_LENGTH"/> property.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct BCRYPT_AUTH_TAG_LENGTHS_STRUCT : IEnumerable<int>
         {
             /// <summary>

diff --git a/src/BCrypt/BCrypt+BCRYPT_DH_KEY_BLOB.cs b/src/BCrypt/BCrypt+BCRYPT_DH_KEY_BLOB.cs
@@ -14,7 +14,6 @@ public partial class BCrypt
         /// A key blob format for transporting DH keys.
         /// Used as a header for a Diffie-Hellman public key or private key BLOB in memory.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct BCRYPT_DH_KEY_BLOB
         {
             /// <summary>

diff --git a/src/BCrypt/BCrypt+BCRYPT_DSA_KEY_BLOB.cs b/src/BCrypt/BCrypt+BCRYPT_DSA_KEY_BLOB.cs
@@ -17,7 +17,6 @@ public partial class BCrypt
         /// <remarks>
         /// The structure applies to DSA keys that equal or exceed 512 bits in length but are less than or equal to 1024 bits.
         /// </remarks>
-        [StructLayout(LayoutKind.Sequential)]
         public struct BCRYPT_DSA_KEY_BLOB
         {
             /// <summary>

diff --git a/src/BCrypt/BCrypt+BCRYPT_DSA_KEY_BLOB_V2.cs b/src/BCrypt/BCrypt+BCRYPT_DSA_KEY_BLOB_V2.cs
@@ -17,7 +17,6 @@ public partial class BCrypt
         /// <remarks>
         /// The structure applies to DSA keys that exceed 1024 bits in length but are less than or equal to 3072 bits.
         /// </remarks>
-        [StructLayout(LayoutKind.Sequential)]
         public struct BCRYPT_DSA_KEY_BLOB_V2
         {
             /// <summary>

diff --git a/src/BCrypt/BCrypt+BCRYPT_ECCKEY_BLOB.cs b/src/BCrypt/BCrypt+BCRYPT_ECCKEY_BLOB.cs
@@ -14,7 +14,6 @@ public partial class BCrypt
         /// A key blob format for transporting ECC keys.
         /// Used as a header for an elliptic curve public key or private key BLOB in memory.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct BCRYPT_ECCKEY_BLOB
         {
             /// <summary>

diff --git a/src/BCrypt/BCrypt+BCRYPT_KEY_BLOB.cs b/src/BCrypt/BCrypt+BCRYPT_KEY_BLOB.cs
@@ -14,7 +14,6 @@ public partial class BCrypt
         /// Represents a key blob header that identifies a key blob format for transporting keys.
         /// The base structure for all CNG key BLOBs. All CNG key BLOBs are based on this structure. For example, the <see cref="BCRYPT_RSAKEY_BLOB"/> structure is based on this structure.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct BCRYPT_KEY_BLOB
         {
             /// <summary>

diff --git a/src/BCrypt/BCrypt+BCRYPT_KEY_DATA_BLOB_HEADER.cs b/src/BCrypt/BCrypt+BCRYPT_KEY_DATA_BLOB_HEADER.cs
@@ -14,7 +14,6 @@ public partial class BCrypt
         /// <summary>
         /// Used to contain information about a key data BLOB. The key data BLOB must immediately follow this structure in memory.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct BCRYPT_KEY_DATA_BLOB_HEADER
         {
             /// <summary>

diff --git a/src/BCrypt/BCrypt+BCRYPT_KEY_LENGTHS_STRUCT.cs b/src/BCrypt/BCrypt+BCRYPT_KEY_LENGTHS_STRUCT.cs
@@ -15,7 +15,6 @@ public partial class BCrypt
         /// Defines the range of key sizes that are supported by the provider.
         /// This structure is used with the <see cref="PropertyNames.BCRYPT_KEY_LENGTHS"/> property.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct BCRYPT_KEY_LENGTHS_STRUCT : IEnumerable<int>
         {
             /// <summary>

diff --git a/src/BCrypt/BCrypt+BCRYPT_OAEP_PADDING_INFO.cs b/src/BCrypt/BCrypt+BCRYPT_OAEP_PADDING_INFO.cs
@@ -13,7 +13,6 @@ public partial class BCrypt
         /// <summary>
         /// The BCRYPT_OAEP_PADDING_INFO structure is used to provide options for the Optimal Asymmetric Encryption Padding (OAEP) scheme.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         [OfferIntPtrPropertyAccessors]
         public unsafe partial struct BCRYPT_OAEP_PADDING_INFO
         {

diff --git a/src/BCrypt/BCrypt+BCRYPT_PKCS1_PADDING_INFO.cs b/src/BCrypt/BCrypt+BCRYPT_PKCS1_PADDING_INFO.cs
@@ -13,7 +13,6 @@ public partial class BCrypt
         /// <summary>
         /// The BCRYPT_PKCS1_PADDING_INFO structure is used to provide options for the PKCS #1 padding scheme.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         [OfferIntPtrPropertyAccessors]
         public unsafe partial struct BCRYPT_PKCS1_PADDING_INFO
         {

diff --git a/src/BCrypt/BCrypt+BCRYPT_PSS_PADDING_INFO.cs b/src/BCrypt/BCrypt+BCRYPT_PSS_PADDING_INFO.cs
@@ -13,7 +13,6 @@ public partial class BCrypt
         /// <summary>
         /// The BCRYPT_PSS_PADDING_INFO structure is used to provide options for the Probabilistic Signature Scheme (PSS) padding scheme.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         [OfferIntPtrPropertyAccessors]
         public unsafe partial struct BCRYPT_PSS_PADDING_INFO
         {

diff --git a/src/BCrypt/BCrypt+BCRYPT_RSAKEY_BLOB.cs b/src/BCrypt/BCrypt+BCRYPT_RSAKEY_BLOB.cs
@@ -13,7 +13,6 @@ public partial class BCrypt
         /// <summary>
         /// A key blob format for transporting RSA keys.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct BCRYPT_RSAKEY_BLOB
         {
             /// <summary>

diff --git a/src/BCrypt/BCrypt+BCryptBuffer.cs b/src/BCrypt/BCrypt+BCryptBuffer.cs
@@ -14,7 +14,6 @@ public static partial class BCrypt
         /// <summary>
         /// Used to represent a generic CNG buffer.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         [OfferIntPtrPropertyAccessors]
         public unsafe partial struct BCryptBuffer
         {

diff --git a/src/BCrypt/BCrypt+BCryptBufferDesc.cs b/src/BCrypt/BCrypt+BCryptBufferDesc.cs
@@ -14,7 +14,6 @@ public static partial class BCrypt
         /// <summary>
         /// Used to contain a set of generic CNG buffers.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         [OfferIntPtrPropertyAccessors]
         public unsafe partial struct BCryptBufferDesc
         {

diff --git a/src/BCrypt/BCrypt+EccKeyBlob.cs b/src/BCrypt/BCrypt+EccKeyBlob.cs
@@ -11,7 +11,6 @@ namespace PInvoke
     /// </content>
     public partial class BCrypt
     {
-        [StructLayout(LayoutKind.Sequential)]
         public struct EccKeyBlob
         {
             /// <summary>

diff --git a/src/Crypt32/Crypt32+CRYPT_KEY_PROV_INFO.cs b/src/Crypt32/Crypt32+CRYPT_KEY_PROV_INFO.cs
@@ -14,7 +14,6 @@ public partial class Crypt32
         /// <summary>
         /// The CRYPT_KEY_PROV_INFO structure contains information about a key container within a cryptographic service provider (CSP).
         /// </summary>
-        [StructLayout(LayoutKind.Sequential, CharSet = CharSet.Unicode)]
         [OfferIntPtrPropertyAccessors]
         public unsafe partial struct CRYPT_KEY_PROV_INFO
         {

diff --git a/src/DbgHelp/DbgHelp+LOADED_IMAGE.cs b/src/DbgHelp/DbgHelp+LOADED_IMAGE.cs
@@ -19,7 +19,6 @@ public static partial class DbgHelp
         /// <summary>
         /// Contains information about the loaded image.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         [OfferIntPtrPropertyAccessors]
         public unsafe partial struct LOADED_IMAGE
         {

diff --git a/src/DwmApi/DwmApi+DWM_BLURBEHIND.cs b/src/DwmApi/DwmApi+DWM_BLURBEHIND.cs
@@ -14,7 +14,6 @@ public partial class DwmApi
         /// <summary>
         /// Specifies Desktop Window Manager (DWM) blur-behind properties. Used by the <see cref="DwmEnableBlurBehindWindow(IntPtr, DWM_BLURBEHIND*)"/> function.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public partial struct DWM_BLURBEHIND
         {
             /// <summary>

diff --git a/src/Fusion/Fusion+FUSION_INSTALL_REFERENCE.cs b/src/Fusion/Fusion+FUSION_INSTALL_REFERENCE.cs
@@ -14,7 +14,6 @@ public partial class Fusion
         /// <summary>
         /// Represents a reference that an application makes to an assembly that the application has installed in the global assembly cache.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         [OfferIntPtrPropertyAccessors]
         public unsafe partial struct FUSION_INSTALL_REFERENCE
         {

diff --git a/src/Fusion/MSCorEE+ASSEMBLY_INFO.cs b/src/Fusion/MSCorEE+ASSEMBLY_INFO.cs
@@ -13,7 +13,6 @@ public partial class Fusion
         /// <summary>
         /// Contains information about an assembly that is registered in the global assembly cache.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         [OfferIntPtrPropertyAccessors]
         public unsafe partial struct ASSEMBLY_INFO
         {

diff --git a/src/Hid/Hid+HiddAttributes.cs b/src/Hid/Hid+HiddAttributes.cs
@@ -13,7 +13,6 @@ public static partial class Hid
         /// <summary>
         /// The HIDD_ATTRIBUTES structure contains vendor information about a HIDClass device.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct HiddAttributes
         {
             /// <summary>

diff --git a/src/Kernel32/Kernel32+FILETIME.cs b/src/Kernel32/Kernel32+FILETIME.cs
@@ -20,7 +20,6 @@ public partial class Kernel32
         /// Xamarin.Android and Xamarin.iOS omit the type and it causes link failures.
         /// See https://github.com/dotnet/pinvoke/issues/232
         /// </remarks>
-        [StructLayout(LayoutKind.Sequential)]
         public struct FILETIME
         {
             /// <summary>

diff --git a/src/Kernel32/Kernel32+OSVERSIONINFOEX.cs b/src/Kernel32/Kernel32+OSVERSIONINFOEX.cs
@@ -13,7 +13,6 @@ public static partial class Kernel32
         /// <summary>
         /// The RTL_OSVERSIONINFOEXW structure contains operating system version information.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential, CharSet = CharSet.Unicode)]
         public unsafe partial struct OSVERSIONINFOEX
         {
             /// <summary>

diff --git a/src/Kernel32/Kernel32+OVERLAPPED.cs b/src/Kernel32/Kernel32+OVERLAPPED.cs
@@ -33,7 +33,6 @@ public partial class Kernel32
         /// using the WaitForMultipleObjects function.
         /// </para>
         /// </remarks>
-        [StructLayout(LayoutKind.Sequential)]
         public struct OVERLAPPED
         {
             /// <summary>

diff --git a/src/Kernel32/Kernel32+SECURITY_ATTRIBUTES.cs b/src/Kernel32/Kernel32+SECURITY_ATTRIBUTES.cs
@@ -16,7 +16,6 @@ public partial class Kernel32
         /// <summary>
         /// The SECURITY_ATTRIBUTES structure contains the security descriptor for an object and specifies whether the handle retrieved by specifying this structure is inheritable. This structure provides security settings for objects created by various functions, such as CreateFile, CreatePipe, CreateProcess, RegCreateKeyEx, or RegSaveKeyEx.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct SECURITY_ATTRIBUTES
         {
             /// <summary>

diff --git a/src/Kernel32/Kernel32+SECURITY_DESCRIPTOR.cs b/src/Kernel32/Kernel32+SECURITY_DESCRIPTOR.cs
@@ -11,7 +11,6 @@ namespace PInvoke
     /// </content>
     public partial class Kernel32
     {
-        [StructLayout(LayoutKind.Sequential)]
         public struct SECURITY_DESCRIPTOR
         {
             public byte Revision;

diff --git a/src/Kernel32/Kernel32+SYSTEMTIME.cs b/src/Kernel32/Kernel32+SYSTEMTIME.cs
@@ -33,7 +33,6 @@ public partial class Kernel32
         /// Because the system time can be adjusted either forward or backward, do not compare system time readings to determine elapsed time.
         /// Instead, use one of the methods described in Windows Time.
         /// </remarks>
-        [StructLayout(LayoutKind.Sequential)]
         public struct SYSTEMTIME
         {
             /// <summary>

diff --git a/src/Kernel32/Kernel32+WIN32_FIND_DATA.cs b/src/Kernel32/Kernel32+WIN32_FIND_DATA.cs
@@ -10,7 +10,6 @@ namespace PInvoke
     /// </content>
     public partial class Kernel32
     {
-        [StructLayout(LayoutKind.Sequential, CharSet = CharSet.Unicode)]
         public struct WIN32_FIND_DATA
         {
             /// <summary>

diff --git a/src/Kernel32/storebanned/Kernel32+ACL.cs b/src/Kernel32/storebanned/Kernel32+ACL.cs
@@ -10,7 +10,6 @@ namespace PInvoke
     /// </content>
     public partial class Kernel32
     {
-        [StructLayout(LayoutKind.Sequential)]
         public struct ACL
         {
             public byte AclRevision;

diff --git a/src/Kernel32/storebanned/Kernel32+CHAR_INFO.cs b/src/Kernel32/storebanned/Kernel32+CHAR_INFO.cs
@@ -14,7 +14,6 @@ public partial class Kernel32
         /// Specifies a Unicode or ANSI character and its attributes.
         /// This structure is used by console functions to read from and write to a console screen buffer.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct CHAR_INFO
         {
             /// <summary>

diff --git a/src/Kernel32/storebanned/Kernel32+CONSOLE_SCREEN_BUFFER_INFO.cs b/src/Kernel32/storebanned/Kernel32+CONSOLE_SCREEN_BUFFER_INFO.cs
@@ -13,7 +13,6 @@ public partial class Kernel32
         /// <summary>
         /// Contains information about a console screen buffer.
         /// </summary>
-        [StructLayout(LayoutKind.Sequential)]
         public struct CONSOLE_SCREEN_BUFFER_INFO
         {
             /// <summary>