Apply suggestions from code review

Co-authored-by: Joshua Lampert <51029046+JoshuaLampert@users.noreply.github.com>

Author: cwittens

src/equations/kdv_1d.jl

@@ -53,7 +53,7 @@ function KdVEquation1D(; gravity, D = 1.0, eta0 = 0.0)
     KdVEquation1D(gravity, D, eta0)
 end
 
-# KdV equations have stiff third-order derivative terms that benefit IMEX methods
+# KdV equations have stiff third-order derivative terms that benefit from IMEX methods
 have_stiff_terms(::KdVEquation1D) = Val{true}()
 
 """

src/semidiscretization.jl

@@ -269,7 +269,7 @@ function _check_split_rhs_implementation(semi)
     args = (nothing, nothing, nothing, mesh, equations, initial_condition,
             boundary_conditions, source_terms, solver, cache)
 
-    # # Check if methods are applicable
+    # Check if methods are applicable
     if !applicable(rhs!, args..., :stiff)
         throw(ArgumentError("Split RHS method with :stiff argument not implemented for $equation_name."))
     end

test/test_kdv_1d.jl

@@ -68,5 +68,5 @@ end
                         cons_error=[2.220446049250313e-15],
                         change_waterheight=-2.220446049250313e-15)
 
-    @test_allocations_splitform(semi, sol, allocs=5_000)
+    @test_allocations_split_ode(semi, sol, allocs=5_000)
 end

test/test_util.jl

@@ -177,13 +177,13 @@ macro test_allocations(semi, sol, allocs)
 end
 
 """
-    @test_allocations_splitform(semi, sol, allocs)
+    @test_allocations_split_ode(semi, sol, allocs)
 
 Test that the memory allocations of `DispersiveShallowWater.rhs_split_stiff!` 
 and `DispersiveShallowWater.rhs_split_nonstiff!` are below `allocs`
 (e.g., from type instabilities).
 """
-macro test_allocations_splitform(semi, sol, allocs)
+macro test_allocations_split_ode(semi, sol, allocs)
     quote
         t = $sol.t[end]
         q = $sol.u[end]