build(deps): bump dependencies to latest version (#15)

* build(deps): update dependencies

Signed-off-by: Luca Georges Francois <luca@quartz.technology>

* fix: update scanners to reflect dependencies bump

Signed-off-by: Luca Georges Francois <luca@quartz.technology>

---------

Signed-off-by: Luca Georges Francois <luca@quartz.technology>

Author: 0xpanoramix

Cargo.toml

@@ -14,7 +14,7 @@ categories = ["command-line-utilities", "development-tools"]
 [dependencies]
 anyhow = "1.0.72"
 clap = { version = "4", features = ["derive"] }
-syn-solidity = "0.3.0"
+syn-solidity = "0.4.2"
 syn = { version = "2.0.26", features = ["full"] }
 proc-macro2 = { version = "1.0.65", features = ["span-locations"]}
 colored = "2.0.4"

src/scanners/implementations/missing_comments.rs

@@ -1,6 +1,4 @@
-use syn_solidity::{
-    Item, ItemContract, ItemEnum, ItemError, ItemEvent, ItemFunction, ItemStruct, ItemUdt,
-};
+use syn_solidity::{Item, ItemContract, ItemEnum, ItemError, ItemEvent, ItemFunction, ItemStruct, ItemUdt, Spanned};
 
 use crate::scanners::{
     result::{Reporter, Severity},

src/scanners/implementations/mutable_functions.rs

@@ -2,7 +2,7 @@ use crate::scanners::{
     result::{Reporter, Severity},
     Scanner,
 };
-use syn_solidity::{FunctionAttribute, Item, ItemContract, ItemFunction, Mutability};
+use syn_solidity::{FunctionAttribute, Item, ItemContract, ItemFunction, Mutability, Spanned};
 
 #[derive(Default)]
 pub struct MutableFunctions {}
@@ -19,8 +19,7 @@ impl MutableFunctions {
 
     /// Reports if a function is able to mutate the contract state.
     fn scan_function(&self, function: &ItemFunction, reporter: &mut Reporter) {
-        // TODO: There is probably a cleaner way to check this.
-        if function.kind.as_str() == "modifier" {
+        if function.kind.is_modifier() {
             return;
         }
 
@@ -31,7 +30,7 @@ impl MutableFunctions {
         ];
 
         for ifa in immutable_function_attributes {
-            if function.attributes.get(ifa).is_some() {
+            if function.attributes.contains(ifa) {
                 return;
             }
         }

src/scanners/implementations/mutable_variables.rs

@@ -2,7 +2,7 @@ use crate::scanners::{
     result::{Reporter, Severity},
     Scanner,
 };
-use syn_solidity::{Item, ItemContract, VariableAttribute, VariableDefinition};
+use syn_solidity::{Item, ItemContract, Spanned, VariableAttribute, VariableDefinition};
 
 #[derive(Default)]
 pub struct MutableVariables {}
@@ -25,7 +25,7 @@ impl MutableVariables {
         ];
 
         for iva in immutable_variable_attributes {
-            if variable.attributes.0.get(iva).is_some() {
+            if variable.attributes.0.contains(iva) {
                 return;
             }
         }

tests/contracts/SimpleContract.sol

@@ -6,7 +6,7 @@ error BrokenQuartz();
 
 enum QuartzType {
     Amethyst,
-    Agate,
+    Agate
 }
 
 struct Quartz {