Apply suggestions from code review

Co-authored-by: Steven Roberts <[email protected]>
LLNL · Aug 29, 2024 · af67f18 · af67f18
1 parent 6a7b4a8
commit af67f18
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/doc/shared/RecentChanges.rst b/doc/shared/RecentChanges.rst
@@ -15,7 +15,7 @@ Fixed the loading of ARKStep's default first order explicit method.
 Fixed a CMake bug regarding usage of missing "print_warning" macro
 that was only triggered when the deprecated ``CUDA_ARCH`` option was used.
 
-Fixed a memory leak that could occur if if ARKodeSetDefaults is called
+Fixed a memory leak that could occur if :c:func:`ARKodeSetDefaults` is called
 repeatedly.
 
 **Deprecation Notices**
diff --git a/src/arkode/arkode_arkstep_io.c b/src/arkode/arkode_arkstep_io.c
@@ -724,7 +724,6 @@ int arkStep_SetDefaults(ARKodeMem ark_mem)
       return (ARK_MEM_FAIL);
     }
   }
-  ark_mem->hadapt_mem->hcontroller = NULL;
   ark_mem->hadapt_mem->hcontroller = SUNAdaptController_PID(ark_mem->sunctx);
   if (ark_mem->hadapt_mem->hcontroller == NULL)
   {