StripAll should be treated as equivalent to StripOthers in save()

scotchi · scotchi · commit de25bc611163 · 2019-09-12T07:57:16.000+02:00
diff --git a/taglib/mpeg/mpegfile.cpp b/taglib/mpeg/mpegfile.cpp
@@ -244,7 +244,7 @@ bool MPEG::File::save(int tags, StripTags strip, ID3v2::Version version, Duplica
 
   // Remove all the tags not going to be saved.
 
-  if(strip == StripOthers)
+  if(strip == StripOthers || strip == StripAll)
     File::strip(~tags, false);
 
   if(ID3v2 & tags) {
diff --git a/taglib/riff/wav/wavfile.cpp b/taglib/riff/wav/wavfile.cpp
@@ -169,7 +169,7 @@ bool RIFF::WAV::File::save(TagTypes tags, StripTags strip, ID3v2::Version versio
     return false;
   }
 
-  if(strip == StripOthers)
+  if(strip == StripOthers || strip == StripAll)
     File::strip(static_cast<TagTypes>(AllTags & ~tags));
 
   if(tags & ID3v2) {

Original file line number	Diff line number	Diff line change
`@@ -169,7 +169,7 @@ bool RIFF::WAV::File::save(TagTypes tags, StripTags strip, ID3v2::Version versio`
`169`	`169`	`return false;`
`170`	`170`	`}`
`171`	`171`
`172`		`- if(strip == StripOthers)`
	`172`	`+ if(strip == StripOthers \|\| strip == StripAll)`
`173`	`173`	`File::strip(static_cast<TagTypes>(AllTags & ~tags));`
`174`	`174`
`175`	`175`	`if(tags & ID3v2) {`