Make SvgExtentions.Traverse/TraverseDepthFirst internal

- avoids polluting the external API
- fixes svg-net#889

Source/SvgExtentions.cs

@@ -85,7 +85,7 @@ public static void ApplyRecursiveDepthFirst(this SvgElement elem, Action<SvgElem
                 action(e);
         }
 
-        public static IEnumerable<T> Traverse<T>(this IEnumerable<T> items, Func<T, IEnumerable<T>> childrenSelector)
+        internal static IEnumerable<T> Traverse<T>(this IEnumerable<T> items, Func<T, IEnumerable<T>> childrenSelector)
         {
             if (childrenSelector == null)
                 throw new ArgumentNullException(nameof(childrenSelector));
@@ -100,10 +100,10 @@ public static IEnumerable<T> Traverse<T>(this IEnumerable<T> items, Func<T, IEnu
             }
         }
 
-        public static IEnumerable<T> Traverse<T>(this T root, Func<T, IEnumerable<T>> childrenSelector)
+        internal static IEnumerable<T> Traverse<T>(this T root, Func<T, IEnumerable<T>> childrenSelector)
             => Enumerable.Repeat(root, 1).Traverse(childrenSelector);
 
-        public static IEnumerable<T> TraverseDepthFirst<T>(this IEnumerable<T> items, Func<T, IEnumerable<T>> childrenSelector)
+        internal static IEnumerable<T> TraverseDepthFirst<T>(this IEnumerable<T> items, Func<T, IEnumerable<T>> childrenSelector)
         {
             if (childrenSelector == null)
                 throw new ArgumentNullException(nameof(childrenSelector));
@@ -118,7 +118,7 @@ public static IEnumerable<T> TraverseDepthFirst<T>(this IEnumerable<T> items, Fu
             }
         }
 
-        public static IEnumerable<T> TraverseDepthFirst<T>(this T root, Func<T, IEnumerable<T>> childrenSelector)
+        internal static IEnumerable<T> TraverseDepthFirst<T>(this T root, Func<T, IEnumerable<T>> childrenSelector)
             => Enumerable.Repeat(root, 1).TraverseDepthFirst(childrenSelector);
     }
 }

doc/ReleaseNotes.md

@@ -4,24 +4,28 @@ The release versions are NuGet releases.
 ## Unreleased (master)
 
 ### Changes
-* change namespace of SvgSymbol from Svg.Document_Structure to Svg. (see [PR #556](https://github.com/svg-net/SVG/pull/556))
-* changed default behavior of DTD resolution so external references are not resolved by default; to mitigate XXE vulnerability. (see [PR #870](https://github.com/svg-net/SVG/pull/870))
-* changed default behavior so external references to images, text definitions, and other resources are not resolved by default; to improve safety of rendering untrusted files. (see [PR #873](https://github.com/svg-net/SVG/pull/873))
-* fixed ISvgRenderer.SmoothingMode reset in method SvgVisualElement.RenderFillAndStroke. (see [PR #883](https://github.com/svg-net/SVG/pull/883))
+* changed namespace of `SvgSymbol` from `Svg.Document_Structure` to `Svg` (see [PR #556](https://github.com/svg-net/SVG/pull/556))
+* mitigated XXE vulnerability: changed default behavior of DTD resolution so external references are not resolved by default
+  (see [#869](https://github.com/svg-net/SVG/issues/869))
+* improved safety of rendering untrusted files: changed default behavior so external references to images, text definitions, 
+  and other resources are not resolved by default (see [#872](https://github.com/svg-net/SVG/issues/872))
+* made `SvgExtentions.Traverse` and `SvgExtentions.TraverseDepthFirst` internal to avoid polluting the API
+  (see [#889](https://github.com/svg-net/SVG/issues/889))
 
 ### Enhancements
-* minimize XmlTextReader customization (see [PR #836](https://github.com/svg-net/SVG/pull/836))
+* minimize `XmlTextReader` customization (see [PR #836](https://github.com/svg-net/SVG/pull/836))
 * manage namespaces and prefixes (see [#604](https://github.com/svg-net/SVG/issues/604))
 
 ### Fixes
-* fixed filled polyline not displayed with stroke-width=0 (see [#785](https://github.com/svg-net/SVG/issues/785))
-* fixed unimplemented filter classes issue (see [#768](https://github.com/svg-net/SVG/issues/768))
-* fixed StackOverFlowException (see [#755](https://github.com/svg-net/SVG/issues/755))
-* fixed different prefix is assigned using XmlTextWriter (see [#817](https://github.com/svg-net/SVG/issues/817))
+* fixed filled polyline not displayed with `stroke-width=0` (see [#785](https://github.com/svg-net/SVG/issues/785))
+* added basic implementation of filter classes (see [#768](https://github.com/svg-net/SVG/issues/768))
+* prevent stack overflow in size calculation for empty SVG (see [#755](https://github.com/svg-net/SVG/issues/755))
+* fixed different prefix is assigned using `XmlTextWriter` (see [#817](https://github.com/svg-net/SVG/issues/817))
 * fixed scaling if opacity is not 1 (see [#863](https://github.com/svg-net/SVG/issues/863))
-* fixed error occurs with empty SVG (see [PR #827](https://github.com/svg-net/SVG/pull/827))
-* fixed unnecessary dependency on System.ValueTuple (see [#879](https://github.com/svg-net/SVG/issues/879))
-* fixed x and y in outermost svg (see [#886](https://github.com/svg-net/SVG/issues/886))
+* fixed unnecessary dependency on `System.ValueTuple` (see [#879](https://github.com/svg-net/SVG/issues/879))
+* prevent `ISvgRenderer.SmoothingMode` reset when `RequiresSmoothRendering` is `true`
+  (see [#882](https://github.com/svg-net/SVG/issues/882))
+* ignore `x` and `y` attributes in outermost svg as per standard (see [#886](https://github.com/svg-net/SVG/issues/886))
 
 ## [Version 3.2.3](https://www.nuget.org/packages/Svg/3.2.3)