do not use container

Signed-off-by: Ric Li <ming3.li@intel.com>
OpenVisualCloud · Jan 12, 2024 · 27c85a1 · 27c85a1
1 parent 168b975
commit 27c85a1
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
diff --git a/.github/workflows/ecosystem.yml b/.github/workflows/ecosystem.yml
@@ -38,8 +38,6 @@ jobs:
     if: ${{ github.repository == 'OpenVisualCloud/Media-Transport-Library' && needs.changes.outputs.changed == 'true' }}
     runs-on: ${{ matrix.os }}
     timeout-minutes: 60
-    container:
-      image: ubuntu:latest
     strategy:
       matrix:
         os: [ubuntu-latest]
@@ -89,6 +87,7 @@ jobs:
         run: |
           cd rust/
           cargo build --verbose
+          cargo clippy
           cd imtl-sys
           cargo build --verbose
           cargo test --verbose
diff --git a/rust/src/imtl/mtl.rs b/rust/src/imtl/mtl.rs
@@ -254,7 +254,7 @@ impl Mtl {
         let mut c_param = unsafe { c_param.assume_init() };
 
         let handle = unsafe { sys::mtl_init(&mut c_param as *mut _) };
-        if handle == std::ptr::null_mut() {
+        if handle.is_null() {
             bail!("Failed to initialize MTL")
         } else {
             self.handle = Some(handle);

diff --git a/rust/src/imtl/video.rs b/rust/src/imtl/video.rs
@@ -307,7 +307,7 @@ impl VideoTx {
         let mut ops = unsafe { ops.assume_init() };
 
         let handle = unsafe { sys::st20_tx_create(mtl.handle().unwrap(), &mut ops as *mut _) };
-        if handle == std::ptr::null_mut() {
+        if handle.is_null() {
             bail!("Failed to initialize MTL")
         } else {
             self.handle = Some(handle);
@@ -495,7 +495,7 @@ impl VideoRx {
         let mut ops = unsafe { ops.assume_init() };
 
         let handle = unsafe { sys::st20_rx_create(mtl.handle().unwrap(), &mut ops as *mut _) };
-        if handle == std::ptr::null_mut() {
+        if handle.is_null() {
             bail!("Failed to initialize MTL")
         } else {
             self.handle = Some(handle);