Ensure that all comments are complete sentences as per guidelines

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

Author: Alan Jowett

libs/execution_context/ebpf_core.c

@@ -38,7 +38,7 @@ const NPI_MODULEID ebpf_general_helper_function_module_id = {
 static ebpf_pinning_table_t* _ebpf_core_map_pinning_table = NULL;
 
 // Assume enabled until we can query it.
-// Extern variable defined in ebpf_core_jit.h
+// Extern variable defined in ebpf_core_jit.h.
 bool ebpf_platform_hypervisor_code_integrity_enabled = true;
 static bool _ebpf_platform_test_signing_enabled = true;
 

tests/libs/util/socket_helper.cpp

@@ -126,13 +126,13 @@ _base_socket::~_base_socket() { clean_up_socket(socket); }
 void
 _base_socket::get_local_address(_Out_ PSOCKADDR& address, _Out_ int& address_length) const
 {
-    // Query the current local address from the socket
+    // Query the current local address from the socket.
     int error = getsockname(socket, (PSOCKADDR)&local_address, &local_address_size);
     if (error != 0) {
         FAIL("Failed to query local address of socket with error: " << WSAGetLastError());
     }
 
-    // Return the freshly queried address
+    // Return the freshly queried address.
     address = (PSOCKADDR)&local_address;
     address_length = local_address_size;
 }
@@ -486,11 +486,11 @@ _datagram_server_socket::_datagram_server_socket(
         FAIL("datagram_client_socket class only supports sockets of type SOCK_DGRAM or SOCK_RAW and protocols of type "
              "IPPROTO_UDP, IPPROTO_IPV4 or IPPROTO_IPV6)");
 
-    // Enable redirect context for UDP sockets
+    // Enable redirect context for UDP sockets.
     if (protocol == IPPROTO_UDP) {
         DWORD option_value = 1;
 
-        // Enable IPv4 redirect context only for IPv4 and Dual stack sockets
+        // Enable IPv4 redirect context only for IPv4 and Dual stack sockets.
         if (family == IPv4 || family == Dual) {
             int result = setsockopt(
                 socket,
@@ -503,7 +503,7 @@ _datagram_server_socket::_datagram_server_socket(
             }
         }
 
-        // Enable IPv6 redirect context only for IPv6 and Dual stack sockets
+        // Enable IPv6 redirect context only for IPv6 and Dual stack sockets.
         if (family == IPv6 || family == Dual) {
             int result = setsockopt(
                 socket,
@@ -533,15 +533,15 @@ _datagram_server_socket::post_async_receive()
         FAIL("WSACreateEvent failed with error: " << WSAGetLastError());
     }
 
-    // Set up WSAMSG structure for WSARecvMsg
+    // Set up WSAMSG structure for WSARecvMsg.
     recv_msg.name = (LPSOCKADDR)&sender_address;
     recv_msg.namelen = sender_address_size;
     recv_msg.lpBuffers = &wsa_recv_buffer;
     recv_msg.dwBufferCount = 1;
     recv_msg.Control = wsa_control_buffer;
     recv_msg.dwFlags = 0;
 
-    // Post an asynchronous receive using WSARecvMsg to get ancillary data
+    // Post an asynchronous receive using WSARecvMsg to get ancillary data.
     error = receive_message(socket, &recv_msg, nullptr, &overlapped, nullptr);
 
     if (error != 0) {

tools/bpf2c/bpf_code_generator.cpp

@@ -522,7 +522,7 @@ bpf_code_generator::parse_btf_maps_section(const unsafe_string& name)
             if (map.name.empty()) {
                 map.name = "__anonymous_" + std::to_string(++anonymous_map_count);
             }
-            // Skip the global variable maps as they are handled seperately.
+            // Skip the global variable maps as they are handled separately.
             if (map.name == ".bss" || map.name == ".data" || map.name == ".rodata") {
                 continue;
             }

tools/netsh/resource.h

@@ -36,7 +36,7 @@
 #define EBPF_FILE_DESCRIPTION "eBPF for Windows Netsh Helper"
 #define EBPF_FILE_NAME "ebpfnetsh.dll"
 
-// Next default values for new objects
+// Next default values for new objects.
 //
 #ifdef APSTUDIO_INVOKED
 #ifndef APSTUDIO_READONLY_SYMBOLS

undocked/tests/sample/ext/drv/sample_ext.c

@@ -144,7 +144,7 @@ _sample_ebpf_extension_program_info_provider_detach_client(_In_ const void* prov
 static void
 _sample_ebpf_extension_program_info_provider_cleanup_binding_context(_Frees_ptr_ void* provider_binding_context);
 
-// Sample eBPF extension Program Information NPI provider characteristics
+// Sample eBPF extension Program Information NPI provider characteristics.
 
 const NPI_PROVIDER_CHARACTERISTICS _sample_ebpf_extension_program_info_provider_characteristics = {
     0,
@@ -227,7 +227,7 @@ _sample_ebpf_extension_hook_provider_detach_client(_In_ const void* provider_bin
 static void
 _sample_ebpf_extension_hook_provider_cleanup_binding_context(_Frees_ptr_ void* provider_binding_context);
 
-// Sample eBPF extension Hook NPI provider characteristics
+// Sample eBPF extension Hook NPI provider characteristics.
 ebpf_attach_provider_data_t _sample_ebpf_extension_attach_provider_data = {
     EBPF_ATTACH_PROVIDER_DATA_HEADER, EBPF_PROGRAM_TYPE_SAMPLE_GUID, BPF_ATTACH_TYPE_SAMPLE, BPF_LINK_TYPE_UNSPEC};
 

undocked/tests/sample/ext/drv/sample_ext_drv.c

@@ -22,12 +22,12 @@
 #define HELPER_DATA_1 1
 #define HELPER_DATA_2 2
 
-// Driver global variables
+// Driver global variables.
 static DEVICE_OBJECT* _sample_ebpf_ext_driver_device_object;
 static BOOLEAN _sample_ebpf_ext_driver_unloading_flag = FALSE;
 
 //
-// Pre-Declarations
+// Pre-Declarations.
 //
 static EVT_WDF_IO_QUEUE_IO_DEVICE_CONTROL _sample_ebpf_ext_driver_io_device_control;
 DRIVER_INITIALIZE DriverEntry;
@@ -124,7 +124,7 @@ _sample_ebpf_ext_driver_initialize_objects(
     status = WdfDeviceCreate(&device_initialize, WDF_NO_OBJECT_ATTRIBUTES, device);
 
     if (!NT_SUCCESS(status)) {
-        // do not free if any other call
+        // Do not free if any other call
         // after WdfDeviceCreate fails.
         WdfDeviceInitFree(device_initialize);
         device_initialize = NULL;

undocked/tests/sample/ext/inc/sample_ext_ioctls.h

@@ -16,7 +16,7 @@
 #define SAMPLE_EBPF_EXT_DEVICE_WIN32_NAME L"\\\\.\\" SAMPLE_EBPF_EXT_DEVICE_BASE_NAME
 
 //
-// IOCTL Codes
+// IOCTL Codes.
 //
 
 typedef enum _sample_ebpf_ext_control_code