<div dir="auto"><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Chris Muller <<a href="mailto:asqueaker@gmail.com" target="_blank" rel="noreferrer">asqueaker@gmail.com</a>> schrieb am Mi., 29. Jan. 2020, 23:40:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div dir="ltr">On Wed, Jan 29, 2020 at 1:04 AM Jakob Reschke <<a href="mailto:forums.jakob@resfarm.de" rel="noreferrer noreferrer" target="_blank">forums.jakob@resfarm.de</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto"><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Chris Muller <<a href="mailto:asqueaker@gmail.com" rel="noreferrer noreferrer noreferrer" target="_blank">asqueaker@gmail.com</a>> schrieb am Mi., 29. Jan. 2020, 03:57:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><br></div><div>It's important this feature does not get inherited by SaveDialog since the functionality could be harmful if used there</div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto">How so? What is bad about wittingly ignoring non-changes during save? You called it pollution yourself.</div></div></blockquote><div><br></div><div>Jakob, your language is confusing because "Ignore" is the other command there that, yes, would cause them to not be committed.</div><div><br></div><div>But what is being added is a 'filter out unchanged methods from the list...', which, IIUC, would only remove them from the list, making them invisible, but not "Ignore" them.  So, if used on the Save dialog, would they be included in the commit?</div></div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Alright if it does not work there it should be fixed in the second iteration of course. I didn't test it. Ignoring instead of removing on save sounds reasonable, then you can change your mind later (although in this case, why would you...).</div><div dir="auto"><br></div><div dir="auto">I suppose this can be covered by a unit test.</div></div>