Fix clippy issues

Author: JR-1991

src/bin/cli.rs

@@ -380,10 +380,7 @@ fn convert(args: ConvertArgs) -> Result<(), Box<dyn Error>> {
         Templates::Linkml => serialize_linkml(model, args.output.as_ref())?,
         Templates::Internal => render_internal_schema(&model)?,
         Templates::JsonLd => {
-            let root = match args.root {
-                Some(root) => Some(root),
-                None => None,
-            };
+            let root = args.root;
             serde_json::to_string_pretty(&model.json_ld_header(root.as_deref())?).unwrap()
         }
         _ => render_jinja_template(&args.template, &mut model, Some(&config))?,

src/datamodel.rs

@@ -321,7 +321,7 @@ impl DataModel {
                 continue;
             }
             if let Some(duplicate_obj) = self.objects.iter().find(|o| o.name == other_obj.name) {
-                if !duplicate_obj.same_hash(&other_obj) {
+                if !duplicate_obj.same_hash(other_obj) {
                     error!(
                         "[{}] {}: Object {} is defined more than once.",
                         "Merge".bold(),
@@ -338,7 +338,7 @@ impl DataModel {
                 continue;
             }
             if let Some(duplicate_enm) = self.enums.iter().find(|e| e.name == other_enm.name) {
-                if !duplicate_enm.same_hash(&other_enm) {
+                if !duplicate_enm.same_hash(other_enm) {
                     error!(
                         "[{}] {}: Enumeration {} is defined more than once.",
                         "Merge".bold(),

src/jsonld/export.rs

@@ -186,8 +186,7 @@ fn build_detailed_term_def(
 ) -> TermDef {
     // Determine type using the first dtype, falling back to parent_object if needed
     let object_type = attr
-        .dtypes
-        .get(0)
+        .dtypes.first()
         .and_then(|dtype| find_sub_object(graph, dtype))
         .map(|_idx| "@id".to_string());
 

src/jsonld/schema.rs

@@ -61,31 +61,35 @@ impl JsonLdHeader {
         let import_url = import_url.into();
         match self.context.take() {
             None => {
-                let mut obj = SimpleContext::default();
-                obj.import = Some(import_url);
+                let obj = SimpleContext {
+                    import: Some(import_url),
+                    ..Default::default()
+                };
                 self.context = Some(JsonLdContext::Object(obj));
             }
             Some(JsonLdContext::Object(mut obj)) => {
                 if obj.import.is_none() {
                     obj.import = Some(import_url);
                     self.context = Some(JsonLdContext::Object(obj));
                 } else {
-                    let mut arr = Vec::new();
-                    arr.push(JsonLdContext::Object(obj));
-                    arr.push(JsonLdContext::Object(SimpleContext {
-                        import: Some(import_url),
-                        ..Default::default()
-                    }));
+                    let arr = vec![
+                        JsonLdContext::Object(obj),
+                        JsonLdContext::Object(SimpleContext {
+                            import: Some(import_url),
+                            ..Default::default()
+                        }),
+                    ];
                     self.context = Some(JsonLdContext::Array(arr));
                 }
             }
             Some(JsonLdContext::Iri(iri)) => {
-                let mut arr = Vec::new();
-                arr.push(JsonLdContext::Iri(iri));
-                arr.push(JsonLdContext::Object(SimpleContext {
-                    import: Some(import_url),
-                    ..Default::default()
-                }));
+                let arr = vec![
+                    JsonLdContext::Iri(iri),
+                    JsonLdContext::Object(SimpleContext {
+                        import: Some(import_url),
+                        ..Default::default()
+                    }),
+                ];
                 self.context = Some(JsonLdContext::Array(arr));
             }
             Some(JsonLdContext::Array(mut arr)) => {
@@ -117,6 +121,7 @@ pub enum TypeOrVec {
 /// or combining several contexts in a single document.
 #[derive(Debug, Clone, Serialize, Deserialize, Eq, PartialEq)]
 #[serde(untagged)]
+#[allow(clippy::large_enum_variant)]
 pub enum JsonLdContext {
     /// The context is a remote document specified by an IRI.
     Iri(String),