Fix comment, thanks @saksham189!

rcurtin · rcurtin · commit c42b9f7670a0 · 2020-03-23T08:25:33.000-04:00
diff --git a/src/mlpack/methods/ann/brnn_impl.hpp b/src/mlpack/methods/ann/brnn_impl.hpp
@@ -66,8 +66,9 @@ template<typename OutputLayerType, typename MergeLayerType,
 BRNN<OutputLayerType, MergeLayerType, MergeOutputType,
     InitializationRuleType, CustomLayers...>::~BRNN()
 {
-  // Remove mergeLayer from the forward and backward RNNs so it doesn't get
-  // deleted.  This assumes that mergeLayer is the last layer!
+  // Remove the last layers from the forward and backward RNNs, as they are held
+  // in mergeLayer.  So, when we use DeleteVisitor with mergeLayer, those two
+  // layers will be properly (and not doubly) freed.
   forwardRNN.network.pop_back();
   backwardRNN.network.pop_back();
 
