simplify the diff a bit

Author: lukesandberg

crates/next-core/src/next_image/module.rs

@@ -58,10 +58,13 @@ impl StructuredImageModuleType {
         let static_asset = StaticUrlJsModule::new(*source).to_resolved().await?;
         Ok(module_asset_context
             .process(
-                Vc::upcast(StructuredImageFileSource::new(
-                    *source,
-                    blur_placeholder_mode,
-                )),
+                Vc::upcast(
+                    StructuredImageFileSource {
+                        image: source,
+                        blur_placeholder_mode,
+                    }
+                    .cell(),
+                ),
                 ReferenceType::Internal(ResolvedVc::cell(fxindexmap!(
                     rcstr!("IMAGE") => ResolvedVc::upcast(static_asset)
                 ))),

crates/next-core/src/next_image/source_asset.rs

@@ -25,25 +25,10 @@ fn blur_options() -> Vc<BlurPlaceholderOptions> {
 
 /// An source asset that transforms an image into javascript code which exports
 /// an object with meta information like width, height and a blur placeholder.
-#[turbo_tasks::value]
+#[turbo_tasks::value(shared)]
 pub struct StructuredImageFileSource {
-    image: ResolvedVc<Box<dyn Source>>,
-    blur_placeholder_mode: BlurPlaceholderMode,
-}
-
-#[turbo_tasks::value_impl]
-impl StructuredImageFileSource {
-    #[turbo_tasks::function]
-    pub fn new(
-        image: ResolvedVc<Box<dyn Source>>,
-        blur_placeholder_mode: BlurPlaceholderMode,
-    ) -> Vc<Self> {
-        Self {
-            image,
-            blur_placeholder_mode,
-        }
-        .cell()
-    }
+    pub image: ResolvedVc<Box<dyn Source>>,
+    pub blur_placeholder_mode: BlurPlaceholderMode,
 }
 
 #[turbo_tasks::value_impl]

turbopack/crates/turbo-tasks/src/trait_ref.rs

@@ -1,4 +1,4 @@
-use std::{fmt::Debug, future::Future, marker::PhantomData, ptr::DynMetadata};
+use std::{fmt::Debug, future::Future, marker::PhantomData};
 
 use anyhow::Result;
 use serde::{Deserialize, Serialize};