Merge pull request #15 from ten3roberts/update-ci

Update ci toolchain

.github/workflows/ci.yml

@@ -14,112 +14,65 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - name: Install nightly
-        uses: actions-rs/toolchain@v1
-        with:
-          toolchain: stable
-          override: true
-          profile: minimal
-
+      - uses: dtolnay/rust-toolchain@stable
       - uses: taiki-e/install-action@nextest
       - name: Run cargo test
-        uses: actions-rs/cargo@v1
-        with:
-          command: nextest
-          args: run --all-features
+        run: cargo nextest run --all-features
 
   test_miri:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - name: Install nightly
-        uses: actions-rs/toolchain@v1
+      - uses: dtolnay/rust-toolchain@master
         with:
           toolchain: nightly
-          override: true
           components: miri
 
       - uses: taiki-e/install-action@nextest
 
       - name: Run cargo miri
-        uses: actions-rs/cargo@v1
-        with:
-          command: miri
-          args: nextest run -j16 --no-default-features --features std,serde,flume,derive
+        run: cargo miri nextest run -j16 --no-default-features --features std,serde,flume,derive
 
   test_nostd:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - name: Install nightly
-        uses: actions-rs/toolchain@v1
-        with:
-          toolchain: stable
-          override: true
-          profile: minimal
-
+      - uses: dtolnay/rust-toolchain@stable
       - uses: taiki-e/install-action@nextest
 
       - name: Run cargo test
-        uses: actions-rs/cargo@v1
-        with:
-          command: nextest
-          args: run --no-default-features
+        run: cargo nextest run --no-default-features
 
   lint_doc:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - name: Install nightly
-        uses: actions-rs/toolchain@v1
-        with:
-          toolchain: stable
-          override: true
-          profile: minimal
-
+      - uses: dtolnay/rust-toolchain@stable
       - name: Run cargo test
-        uses: actions-rs/cargo@v1
-        with:
-          command: doc
-          args: --all-features
+        run: cargo doc --all-features
 
   test_doc:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - name: Install nightly
-        uses: actions-rs/toolchain@v1
-        with:
-          toolchain: stable
-          override: true
-          profile: minimal
-
+      - uses: dtolnay/rust-toolchain@stable
       - name: Run cargo test (doc)
-        uses: actions-rs/cargo@v1
-        with:
-          command: test
-          args: --doc --all-features
+        run: cargo test --doc --all-features
 
   coverage:
     runs-on: ubuntu-latest
 
     steps:
       - uses: actions/checkout@v2
-      - name: Install nightly
-        uses: actions-rs/toolchain@v1
+      - uses: dtolnay/rust-toolchain@master
         with:
           toolchain: nightly
-          override: true
-          profile: minimal
           components: miri
 
       - uses: taiki-e/install-action@cargo-tarpaulin
 
       - name: Run cargo tarpaulin
-        uses: actions-rs/cargo@v1
-        with:
-          command: tarpaulin
-          args: --all-features --out Xml --engine llvm
+        run: cargo tarpaulin --all-features --out Xml --engine llvm
 
       - name: Upload to codecov.io
         uses: codecov/codecov-action@v2

.github/workflows/deploy.yml

@@ -13,9 +13,9 @@ jobs:
     if: github.event_name == 'push'
     steps:
       - uses: actions/checkout@v2
-      - uses: dtolnay/rust-toolchain@1.70.0
+      - uses: dtolnay/rust-toolchain@master
         with:
-          target: wasm32-unknown-unknown
+          targets: wasm32-unknown-unknown
       - run: cargo build --release --package asteroids --target wasm32-unknown-unknown --manifest-path=asteroids/Cargo.toml
       - run: mv ./target/wasm32-unknown-unknown/release/asteroids.wasm ./asteroids/public
       - name: Upload Artefact

src/component.rs

@@ -286,11 +286,6 @@ impl<T: ComponentValue> Component<T> {
     pub fn name(&self) -> &'static str {
         self.vtable.name
     }
-
-    /// Returns all metadata components
-    pub fn get_meta(&self) -> ComponentBuffer {
-        self.vtable.meta.get(self.desc())
-    }
 }
 
 impl<T: ComponentValue> Metadata<T> for Component<T> {
@@ -451,7 +446,7 @@ impl ComponentDesc {
         self.key.target.is_some()
     }
 
-    pub(crate) fn get_meta(&self) -> ComponentBuffer {
+    pub(crate) fn create_meta(&self) -> ComponentBuffer {
         self.vtable.meta.get(*self)
     }
 

src/macros.rs

@@ -109,27 +109,11 @@ macro_rules! component {
         $(#[$outer])*
         $vis fn $name($obj: $crate::Entity) -> $crate::Component<$ty> {
 
-            static COMPONENT_ID: ::core::sync::atomic::AtomicU32 = ::core::sync::atomic::AtomicU32::new($crate::entity::EntityIndex::MAX);
-            fn meta(_desc: $crate::component::ComponentDesc) -> $crate::buffer::ComponentBuffer {
-                let mut _buffer = $crate::buffer::ComponentBuffer::new();
-
-                <$crate::metadata::Name as $crate::metadata::Metadata<$ty>>::attach(_desc, &mut _buffer);
-                <$crate::Component<$ty> as $crate::metadata::Metadata<$ty>>::attach(_desc, &mut _buffer);
-
-                $(
-                    $(
-                        <$metadata as $crate::metadata::Metadata::<$ty>>::attach(_desc, &mut _buffer);
-                    )*
-                )*
-
-                _buffer
-            }
-
-            static META: $crate::vtable::LazyComponentBuffer = $crate::vtable::LazyComponentBuffer::new(meta);
-            static VTABLE: &$crate::vtable::ComponentVTable<$ty> =
-                &$crate::vtable::ComponentVTable::new(stringify!($name), &META);
             use $crate::entity::EntityKind;
             use $crate::relation::RelationExt;
+
+            static COMPONENT_ID: ::core::sync::atomic::AtomicU32 = ::core::sync::atomic::AtomicU32::new($crate::entity::EntityIndex::MAX);
+            static VTABLE: &$crate::vtable::ComponentVTable<$ty> = $crate::component_vtable!($name: $ty $(=> [$($metadata),*])?);
             $crate::Component::static_init(&COMPONENT_ID, EntityKind::COMPONENT, VTABLE).of($obj)
         }
 
@@ -142,26 +126,10 @@ macro_rules! component {
 
         $(#[$outer])*
         $vis fn $name() -> $crate::Component<$ty> {
-        static COMPONENT_ID: ::core::sync::atomic::AtomicU32 = ::core::sync::atomic::AtomicU32::new($crate::entity::EntityIndex::MAX);
-            fn meta(_desc: $crate::component::ComponentDesc) -> $crate::buffer::ComponentBuffer {
-                let mut _buffer = $crate::buffer::ComponentBuffer::new();
-
-                <$crate::metadata::Name as $crate::metadata::Metadata<$ty>>::attach(_desc, &mut _buffer);
-                <$crate::Component<$ty> as $crate::metadata::Metadata<$ty>>::attach(_desc, &mut _buffer);
-
-                $(
-                    $(
-                        <$metadata as $crate::metadata::Metadata::<$ty>>::attach(_desc, &mut _buffer);
-                    )*
-                )*
-
-                _buffer
-            }
-
-            static META: $crate::vtable::LazyComponentBuffer = $crate::vtable::LazyComponentBuffer::new(meta);
-            static VTABLE: &$crate::vtable::ComponentVTable<$ty> =
-                &$crate::vtable::ComponentVTable::new(stringify!($name), &META);
             use $crate::entity::EntityKind;
+
+            static COMPONENT_ID: ::core::sync::atomic::AtomicU32 = ::core::sync::atomic::AtomicU32::new($crate::entity::EntityIndex::MAX);
+            static VTABLE: &$crate::vtable::ComponentVTable<$ty> = $crate::component_vtable!($name: $ty $(=> [$($metadata),*])?);
             $crate::Component::static_init(&COMPONENT_ID, EntityKind::COMPONENT, VTABLE)
         }
 
@@ -204,11 +172,10 @@ macro_rules! component_vtable {
 
             }
 
-            static META: $crate::vtable::LazyComponentBuffer = $crate::vtable::LazyComponentBuffer::new(meta);
-            static VTABLE: &$crate::vtable::ComponentVTable<$ty> =
-                &$crate::vtable::ComponentVTable::new(stringify!($name), &META);
+            static VTABLE: $crate::vtable::ComponentVTable<$ty> =
+                $crate::vtable::ComponentVTable::new(stringify!($name), $crate::vtable::LazyComponentBuffer::new(meta));
 
-            VTABLE
+            &VTABLE
         }
 
     };

src/metadata/mod.rs

@@ -46,7 +46,7 @@ mod test {
             foo: String => [crate::Debuggable],
         }
 
-        let meta = foo().get_meta();
+        let meta = foo().desc().create_meta();
 
         assert!(meta.get(debuggable()).is_some());
         assert_eq!(meta.get(name()), Some(&"foo".into()));

src/vtable.rs

@@ -43,7 +43,7 @@ pub struct UntypedVTable {
     pub(crate) dangling: fn() -> NonNull<u8>,
     /// A metadata is a component which is attached to the component, such as
     /// metadata or name
-    pub(crate) meta: &'static LazyComponentBuffer,
+    pub(crate) meta: LazyComponentBuffer,
 }
 
 impl UntypedVTable {
@@ -55,7 +55,7 @@ impl UntypedVTable {
     /// Creates a new vtable of type `T`
     pub(crate) const fn new<T: ComponentValue>(
         name: &'static str,
-        meta: &'static LazyComponentBuffer,
+        meta: LazyComponentBuffer,
     ) -> Self {
         unsafe fn drop_ptr<T>(x: *mut u8) {
             x.cast::<T>().drop_in_place()
@@ -99,7 +99,7 @@ impl<T> core::ops::Deref for ComponentVTable<T> {
 
 impl<T: ComponentValue> ComponentVTable<T> {
     /// Creates a new *typed* vtable of `T`
-    pub const fn new(name: &'static str, meta: &'static LazyComponentBuffer) -> Self {
+    pub const fn new(name: &'static str, meta: LazyComponentBuffer) -> Self {
         Self {
             inner: UntypedVTable::new::<T>(name, meta),
             marker: PhantomData,

src/world.rs

@@ -342,7 +342,7 @@ impl World {
         }
 
         let id = desc.key().id;
-        let mut meta = desc.get_meta();
+        let mut meta = desc.create_meta();
         meta.set(component_info(), desc);
         meta.set(name(), desc.name().into());
 
@@ -917,7 +917,7 @@ impl World {
 
         let desc = component.desc();
 
-        let mut meta = desc.get_meta();
+        let mut meta = desc.create_meta();
         meta.set(component_info(), desc);
         meta.set(components::name(), desc.name().into());
 

tests/components.rs

@@ -11,15 +11,16 @@ use glam::{vec2, Vec2};
 fn custom_component() {
     let mut world = World::new();
 
-    static META: LazyComponentBuffer = LazyComponentBuffer::new(|desc| {
-        let mut buf = ComponentBuffer::new();
-        <Debuggable as Metadata<Vec2>>::attach(desc, &mut buf);
-        buf
-    });
-
-    static VTABLE: &ComponentVTable<Vec2> = &ComponentVTable::new("position", &META);
+    static VTABLE: ComponentVTable<Vec2> = ComponentVTable::new(
+        "position",
+        LazyComponentBuffer::new(|desc| {
+            let mut buf = ComponentBuffer::new();
+            <Debuggable as Metadata<Vec2>>::attach(desc, &mut buf);
+            buf
+        }),
+    );
 
-    let position = world.spawn_component(VTABLE);
+    let position = world.spawn_component(&VTABLE);
 
     let id = Entity::builder()
         .set(position, vec2(1.0, 6.4))