Small fixes

Author: tryashtar

fNbt/Tags/NbtContainerTag.cs

@@ -52,7 +52,7 @@ public void Clear()
         }
         public void Insert(int index, NbtTag item)
         {
-            PerformAction(new DescriptionHolder("Insert {0} into {1} at index {2}", this, item, index),
+            PerformAction(new DescriptionHolder("Insert {0} into {1} at index {2}", item, this, index),
                 () => DoInsert(index, item),
                 () => DoRemoveAt(index)
             );
@@ -64,7 +64,7 @@ public bool Remove(NbtTag item)
                 return false;
             return PerformAction(new DescriptionHolder("Remove {0} from {1}", item, this),
                 () => DoRemove(item),
-                () => { DoInsert(index, item); }
+                () => DoInsert(index, item)
             );
         }
         public void RemoveAt(int index)

fNbt/Tags/NbtTag.cs

@@ -15,13 +15,13 @@ public abstract class NbtTag : ICloneable {
         public abstract NbtTagType TagType { get; }
 
         /// <summary> Event raised when this tag or one of its children is changed. </summary>
-        public event EventHandler<NbtTag> Changed;
+        public event Action<NbtTag> Changed;
 
-        /// <summary> Event raised when this tag or one of its children has an undoable action performed on it. </summary>
-        public event EventHandler<UndoableAction> ActionPerformed;
+        /// <summary> Event raised when this tag has an undoable action performed on it. </summary>
+        public event Action<UndoableAction> ActionPerformed;
 
-        private void RaiseChanged(NbtTag tag) => Changed?.Invoke(this, tag);
-        private void RaiseActionPerformed(UndoableAction action) => ActionPerformed?.Invoke(this, action);
+        private void RaiseChanged(NbtTag tag) => Changed?.Invoke(tag);
+        private void RaiseActionPerformed(UndoableAction action) => ActionPerformed?.Invoke(action);
 
         /// <summary> Helper method for signaling changes to parent tags. </summary>
         protected T PerformAction<T>(DescriptionHolder description, Func<T> action, Action undo)
@@ -84,21 +84,17 @@ public string Name {
             get { return name; }
             set {
                 string current_name = name;
+                if (current_name == value)
+                    return;
                 PerformAction(new DescriptionHolder("Rename {0} from {1}", this, current_name),
                     () => SetName(value),
                     () => SetName(current_name)
                 );
-
             }
         }
 
         private void SetName(string value)
         {
-            if (name == value)
-            {
-                return;
-            }
-
             var parentAsCompound = Parent as NbtCompound;
             if (parentAsCompound != null)
             {