Remove disallowed-macros

Signed-off-by: Simon Davies <simongdavies@users.noreply.github.com>

Author: simongdavies

src/hyperlight_common/clippy.toml

@@ -17,16 +17,6 @@ limitations under the License.
 #![cfg_attr(not(any(test, debug_assertions)), warn(clippy::panic))]
 #![cfg_attr(not(any(test, debug_assertions)), warn(clippy::expect_used))]
 #![cfg_attr(not(any(test, debug_assertions)), warn(clippy::unwrap_used))]
-// clippy.toml disallows assert!/assert_eq!/assert_ne! via disallowed-macros.
-// That lint is active by default, so we suppress it globally here, then
-// selectively re-enable it for release host builds (feature = "std").
-// Guest targets (no std) are allowed asserts — panics are contained in the
-// micro-VM and cannot crash the host. Tests and debug builds are also allowed.
-#![allow(clippy::disallowed_macros)]
-#![cfg_attr(
-    not(any(test, debug_assertions, not(feature = "std"))),
-    warn(clippy::disallowed_macros)
-)]
 // We use Arbitrary during fuzzing, which requires std
 #![cfg_attr(not(feature = "fuzzing"), no_std)]
 

src/hyperlight_common/src/clippy.toml

@@ -90,7 +90,6 @@ impl<const NAME_SZ: usize, const DESC_SZ: usize> ElfNote<NAME_SZ, DESC_SZ> {
     ///
     /// Panics at compile time if `NAME_SZ` or `DESC_SZ` don't match
     /// `padded_name_size(name.len() + 1)` or `padded_desc_size(desc.len() + 1)`.
-    #[allow(clippy::disallowed_macros)] // These asserts are evaluated at compile time only (const fn).
     pub const fn new(name: &str, desc: &str, n_type: u32) -> Self {
         // NAME_SZ and DESC_SZ must match the padded sizes.
         assert!(

src/hyperlight_common/src/lib.rs

@@ -14,11 +14,6 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-#![expect(
-    clippy::disallowed_macros,
-    reason = "This is a benchmark file, so using disallowed macros is fine here."
-)]
-
 use std::sync::{Arc, Barrier, Mutex};
 use std::thread;
 use std::time::{Duration, Instant};

src/hyperlight_common/src/version_note.rs

@@ -71,8 +71,6 @@ limitations under the License.
 //!
 //! Core dump files are named `hl_core_<timestamp>.elf`.
 
-#![allow(clippy::disallowed_macros)]
-
 #[cfg(all(crashdump, target_os = "linux"))]
 use std::io::Write;
 

src/hyperlight_host/benches/benchmarks.rs

@@ -13,7 +13,6 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 See the License for the specific language governing permissions and
 limitations under the License.
 */
-#![allow(clippy::disallowed_macros)]
 use std::thread;
 
 use hyperlight_host::sandbox::SandboxConfiguration;

src/hyperlight_host/clippy.toml

@@ -13,7 +13,6 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 See the License for the specific language governing permissions and
 limitations under the License.
 */
-#![allow(clippy::disallowed_macros)]
 use std::thread;
 
 use hyperlight_host::{MultiUseSandbox, UninitializedSandbox};

src/hyperlight_host/examples/crashdump/main.rs

@@ -13,7 +13,6 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 See the License for the specific language governing permissions and
 limitations under the License.
 */
-#![allow(clippy::disallowed_macros)]
 extern crate hyperlight_host;
 
 use std::sync::{Arc, Barrier};