chore: apply group_imports = "StdExternalCrate"[1]

Unmerge a few excessively complex import statements.

The option itself is unstable, so it is commented in rustfmt.toml.

[1]: https://rust-lang.github.io/rustfmt/?version=master&search=#group_imports

Author: bavshin-f5

examples/async.rs

@@ -1,3 +1,9 @@
+use std::ffi::{c_char, c_void};
+use std::ptr::{addr_of, addr_of_mut};
+use std::sync::atomic::AtomicBool;
+use std::sync::Arc;
+use std::time::Instant;
+
 use ngx::core;
 use ngx::ffi::{
     nginx_version, ngx_array_push, ngx_command_t, ngx_conf_t, ngx_cycle, ngx_event_t, ngx_http_core_module,
@@ -7,11 +13,6 @@ use ngx::ffi::{
 };
 use ngx::http::{self, HTTPModule, MergeConfigError};
 use ngx::{http_request_handler, ngx_log_debug_http, ngx_null_command, ngx_string};
-use std::ffi::{c_char, c_void};
-use std::ptr::{addr_of, addr_of_mut};
-use std::sync::atomic::AtomicBool;
-use std::sync::Arc;
-use std::time::Instant;
 use tokio::runtime::Runtime;
 
 struct Module;

examples/awssig.rs

@@ -1,3 +1,6 @@
+use std::ffi::{c_char, c_void};
+use std::ptr::addr_of;
+
 use http::HeaderMap;
 use ngx::core;
 use ngx::ffi::{
@@ -8,8 +11,6 @@ use ngx::ffi::{
 };
 use ngx::http::*;
 use ngx::{http_request_handler, ngx_log_debug_http, ngx_null_command, ngx_string};
-use std::ffi::{c_char, c_void};
-use std::ptr::addr_of;
 
 struct Module;
 

examples/curl.rs

@@ -1,3 +1,6 @@
+use std::ffi::{c_char, c_void};
+use std::ptr::addr_of;
+
 use ngx::core;
 use ngx::ffi::{
     nginx_version, ngx_array_push, ngx_command_t, ngx_conf_t, ngx_http_core_module, ngx_http_handler_pt,
@@ -6,8 +9,6 @@ use ngx::ffi::{
 };
 use ngx::http::{self, HTTPModule, MergeConfigError};
 use ngx::{http_request_handler, ngx_log_debug_http, ngx_null_command, ngx_string};
-use std::ffi::{c_char, c_void};
-use std::ptr::addr_of;
 
 struct Module;
 

examples/httporigdst.rs

@@ -1,3 +1,6 @@
+use std::ffi::{c_char, c_int, c_void};
+use std::ptr::addr_of;
+
 use ngx::core;
 use ngx::ffi::{
     in_port_t, nginx_version, ngx_conf_t, ngx_connection_local_sockaddr, ngx_http_add_variable, ngx_http_module_t,
@@ -6,8 +9,6 @@ use ngx::ffi::{
 };
 use ngx::http::{self, HTTPModule};
 use ngx::{http_variable_get, ngx_http_null_variable, ngx_log_debug_http, ngx_null_string, ngx_string};
-use std::ffi::{c_char, c_int, c_void};
-use std::ptr::addr_of;
 
 const IPV4_STRLEN: usize = INET_ADDRSTRLEN as usize;
 

examples/upstream.rs

@@ -6,30 +6,26 @@
  * The NGINX authors are grateful to @gabihodoroaga for their contributions
  * to the community at large.
  */
-use ngx::{
-    core::{Pool, Status},
-    ffi::{
-        nginx_version, ngx_atoi, ngx_command_t, ngx_conf_log_error, ngx_conf_t, ngx_connection_t,
-        ngx_event_free_peer_pt, ngx_event_get_peer_pt, ngx_http_module_t, ngx_http_upstream_init_peer_pt,
-        ngx_http_upstream_init_pt, ngx_http_upstream_init_round_robin, ngx_http_upstream_module,
-        ngx_http_upstream_srv_conf_t, ngx_http_upstream_t, ngx_int_t, ngx_module_t, ngx_peer_connection_t, ngx_str_t,
-        ngx_uint_t, NGX_CONF_NOARGS, NGX_CONF_TAKE1, NGX_CONF_UNSET, NGX_ERROR, NGX_HTTP_MODULE, NGX_HTTP_UPS_CONF,
-        NGX_LOG_EMERG, NGX_RS_HTTP_SRV_CONF_OFFSET, NGX_RS_MODULE_SIGNATURE,
-    },
-    http::{
-        ngx_http_conf_get_module_srv_conf, ngx_http_conf_upstream_srv_conf_immutable,
-        ngx_http_conf_upstream_srv_conf_mutable, HTTPModule, Merge, MergeConfigError, Request,
-    },
-    http_upstream_init_peer_pt,
-    log::DebugMask,
-    ngx_log_debug_http, ngx_log_debug_mask, ngx_null_command, ngx_string,
+use std::ffi::{c_char, c_void};
+use std::mem;
+use std::ptr::addr_of;
+use std::slice;
+
+use ngx::core::{Pool, Status};
+use ngx::ffi::{
+    nginx_version, ngx_atoi, ngx_command_t, ngx_conf_log_error, ngx_conf_t, ngx_connection_t, ngx_event_free_peer_pt,
+    ngx_event_get_peer_pt, ngx_http_module_t, ngx_http_upstream_init_peer_pt, ngx_http_upstream_init_pt,
+    ngx_http_upstream_init_round_robin, ngx_http_upstream_module, ngx_http_upstream_srv_conf_t, ngx_http_upstream_t,
+    ngx_int_t, ngx_module_t, ngx_peer_connection_t, ngx_str_t, ngx_uint_t, NGX_CONF_NOARGS, NGX_CONF_TAKE1,
+    NGX_CONF_UNSET, NGX_ERROR, NGX_HTTP_MODULE, NGX_HTTP_UPS_CONF, NGX_LOG_EMERG, NGX_RS_HTTP_SRV_CONF_OFFSET,
+    NGX_RS_MODULE_SIGNATURE,
 };
-use std::{
-    ffi::{c_char, c_void},
-    mem,
-    ptr::addr_of,
-    slice,
+use ngx::http::{
+    ngx_http_conf_get_module_srv_conf, ngx_http_conf_upstream_srv_conf_immutable,
+    ngx_http_conf_upstream_srv_conf_mutable, HTTPModule, Merge, MergeConfigError, Request,
 };
+use ngx::log::DebugMask;
+use ngx::{http_upstream_init_peer_pt, ngx_log_debug_http, ngx_log_debug_mask, ngx_null_command, ngx_string};
 
 #[derive(Clone, Copy, Debug)]
 #[repr(C)]

rustfmt.toml

@@ -1 +1,2 @@
+# group_imports = "StdExternalCrate" # unstable
 max_width = 120

src/core/buffer.rs

@@ -1,7 +1,7 @@
-use crate::ffi::*;
-
 use std::slice;
 
+use crate::ffi::*;
+
 /// The `Buffer` trait provides methods for working with an nginx buffer (`ngx_buf_t`).
 pub trait Buffer {
     /// Returns a raw pointer to the underlying `ngx_buf_t` of the buffer.

src/core/pool.rs

@@ -1,9 +1,9 @@
-use crate::core::buffer::{Buffer, MemoryBuffer, TemporaryBuffer};
-use crate::ffi::*;
-
 use std::ffi::c_void;
 use std::{mem, ptr};
 
+use crate::core::buffer::{Buffer, MemoryBuffer, TemporaryBuffer};
+use crate::ffi::*;
+
 /// Wrapper struct for an `ngx_pool_t` pointer, providing methods for working with memory pools.
 pub struct Pool(*mut ngx_pool_t);
 

src/core/status.rs

@@ -1,6 +1,7 @@
-use crate::ffi::*;
 use std::fmt;
 
+use crate::ffi::*;
+
 /// Status
 ///
 /// Rust native wrapper for NGINX status codes.

src/core/string.rs

@@ -1,9 +1,9 @@
-use crate::ffi::*;
-
 use std::borrow::Cow;
 use std::slice;
 use std::str::{self, Utf8Error};
 
+use crate::ffi::*;
+
 /// Static string initializer for [`ngx_str_t`].
 ///
 /// The resulting byte string is always nul-terminated (just like a C string).