ronantakizawa/codereview-bench huggingface dataset:
Repository: files-community/Files
File: src/Files.App/Actions/Content/Tags/OpenAllTaggedActions.cs (C#)

Code before:

		public string Description
			=> "OpenAllTaggedItemsDescription".GetLocalizedResource();

		public RichGlyph Glyph
			=> new("\uE71D");

		public bool IsExecutable => 
			_pageContext.ShellPage is not null &&
			_tagsContext.TaggedItems.Any();

		public OpenAllTaggedActions()
		{
			_pageContext = Ioc.Default.GetRequiredService<IContentPageContext>();
			_tagsContext = Ioc.Default.GetRequiredService<ITagsContext>();

			_pageContext.PropertyChanged += Context_PropertyChanged;
			_tagsContext.PropertyChanged += Context_PropertyChanged;
		}

		public async Task ExecuteAsync()
		{
			var filePaths = _tagsContext.TaggedItems.Where(item => !item.isFolder).Select(f => f.path);
			var folderPaths = _tagsContext.TaggedItems.Where(item => item.isFolder).Select(f => f.path);

			foreach (var file in filePaths)
			{
			    await NavigationHelpers.OpenPath(file, _pageContext.ShellPage!);
			}
			
			foreach (var folder in folderPaths)
			{
			    await NavigationHelpers.OpenPathInNewTab(folder,false);
			}
		}

		private void Context_PropertyChanged(object? sender, PropertyChangedEventArgs e)
		{
			switch (e.PropertyName)
			{
				case nameof(IContentPageContext.ShellPage):
				case nameof(ITagsContext.TaggedItems):
					OnPropertyChanged(nameof(IsExecutable));
					break;
			}
		}
	}
}


Review comment (suggestion):
```suggestion
			foreach (var path in filePaths)
```

Code after:

		public string Description
			=> "OpenAllTaggedItemsDescription".GetLocalizedResource();

		public RichGlyph Glyph
			=> new("\uE71D");

		public bool IsExecutable => 
			_pageContext.ShellPage is not null &&
			_tagsContext.TaggedItems.Any();

		public OpenAllTaggedActions()
		{
			_pageContext = Ioc.Default.GetRequiredService<IContentPageContext>();
			_tagsContext = Ioc.Default.GetRequiredService<ITagsContext>();

			_pageContext.PropertyChanged += Context_PropertyChanged;
			_tagsContext.PropertyChanged += Context_PropertyChanged;
		}

		public async Task ExecuteAsync()
		{
			var filePaths = _tagsContext.TaggedItems
				.Where(item => !item.isFolder)
				.Select(f => f.path)
				.ToList();
			
			var folderPaths = _tagsContext
				.TaggedItems
				.Where(item => item.isFolder)
				.Select(f => f.path)
				.ToList();
			
			await Task.WhenAll(filePaths.Select(path => NavigationHelpers.OpenPath(path, _pageContext.ShellPage!)));

			foreach (var path in folderPaths) 
				await NavigationHelpers.OpenPathInNewTab(path, false);
		}

		private void Context_PropertyChanged(object? sender, PropertyChangedEventArgs e)
		{
			switch (e.PropertyName)
			{
				case nameof(IContentPageContext.ShellPage):
				case nameof(ITagsContext.TaggedItems):
					OnPropertyChanged(nameof(IsExecutable));
					break;
			}
		}
	}
}
Repository: files-community/Files
File: src/Files.App/Actions/Content/Tags/OpenAllTaggedActions.cs (C#)

Code before:
			=> new("\uE71D");

		public bool IsExecutable => 
			_pageContext.ShellPage is not null &&
			_tagsContext.TaggedItems.Any();

		public OpenAllTaggedActions()
		{
			_pageContext = Ioc.Default.GetRequiredService<IContentPageContext>();
			_tagsContext = Ioc.Default.GetRequiredService<ITagsContext>();

			_pageContext.PropertyChanged += Context_PropertyChanged;
			_tagsContext.PropertyChanged += Context_PropertyChanged;
		}

		public async Task ExecuteAsync()
		{
			var filePaths = _tagsContext.TaggedItems.Where(item => !item.isFolder).Select(f => f.path);
			var folderPaths = _tagsContext.TaggedItems.Where(item => item.isFolder).Select(f => f.path);

			foreach (var file in filePaths)
			{
			    await NavigationHelpers.OpenPath(file, _pageContext.ShellPage!);
			}
			
			foreach (var folder in folderPaths)
			{
			    await NavigationHelpers.OpenPathInNewTab(folder,false);
			}
		}

		private void Context_PropertyChanged(object? sender, PropertyChangedEventArgs e)
		{
			switch (e.PropertyName)
			{
				case nameof(IContentPageContext.ShellPage):
				case nameof(ITagsContext.TaggedItems):
					OnPropertyChanged(nameof(IsExecutable));
					break;
			}
		}
	}
}


Review comment (suggestion):
```suggestion
			foreach (var path in folderPaths)
```
I think we can also remove the curly brackets

Code after:
			=> new("\uE71D");

		public bool IsExecutable => 
			_pageContext.ShellPage is not null &&
			_tagsContext.TaggedItems.Any();

		public OpenAllTaggedActions()
		{
			_pageContext = Ioc.Default.GetRequiredService<IContentPageContext>();
			_tagsContext = Ioc.Default.GetRequiredService<ITagsContext>();

			_pageContext.PropertyChanged += Context_PropertyChanged;
			_tagsContext.PropertyChanged += Context_PropertyChanged;
		}

		public async Task ExecuteAsync()
		{
			var filePaths = _tagsContext.TaggedItems
				.Where(item => !item.isFolder)
				.Select(f => f.path)
				.ToList();
			
			var folderPaths = _tagsContext
				.TaggedItems
				.Where(item => item.isFolder)
				.Select(f => f.path)
				.ToList();
			
			await Task.WhenAll(filePaths.Select(path => NavigationHelpers.OpenPath(path, _pageContext.ShellPage!)));

			foreach (var path in folderPaths) 
				await NavigationHelpers.OpenPathInNewTab(path, false);
		}

		private void Context_PropertyChanged(object? sender, PropertyChangedEventArgs e)
		{
			switch (e.PropertyName)
			{
				case nameof(IContentPageContext.ShellPage):
				case nameof(ITagsContext.TaggedItems):
					OnPropertyChanged(nameof(IsExecutable));
					break;
			}
		}
	}
}
Repository: files-community/Files
File: src/Files.App/Actions/Content/Tags/OpenAllTaggedActions.cs (C#)

Code before:
			_pageContext.ShellPage is not null &&
			_tagsContext.TaggedItems.Any();

		public OpenAllTaggedActions()
		{
			_pageContext = Ioc.Default.GetRequiredService<IContentPageContext>();
			_tagsContext = Ioc.Default.GetRequiredService<ITagsContext>();

			_pageContext.PropertyChanged += Context_PropertyChanged;
			_tagsContext.PropertyChanged += Context_PropertyChanged;
		}

		public async Task ExecuteAsync()
		{
			var filePaths = _tagsContext.TaggedItems
				.Where(item => !item.isFolder)
				.Select(f => f.path)
				.ToList();
			
			var folderPaths = _tagsContext
				.TaggedItems
				.Where(item => item.isFolder)
				.Select(f => f.path)
				.ToList();
			
			await Task.WhenAll(filePaths.Select(path=> NavigationHelpers.OpenPath(path, _pageContext.ShellPage!)));

			foreach (var path in folderPaths) 
				await NavigationHelpers.OpenPathInNewTab(path, false);
		}

		private void Context_PropertyChanged(object? sender, PropertyChangedEventArgs e)
		{
			switch (e.PropertyName)
			{
				case nameof(IContentPageContext.ShellPage):
				case nameof(ITagsContext.TaggedItems):
					OnPropertyChanged(nameof(IsExecutable));
					break;
			}
		}
	}
}


Review comment (refactor):
```suggestion
			await Task.WhenAll(filePaths.Select(path => NavigationHelpers.OpenPath(path, _pageContext.ShellPage!)));
```

Code after:
			_pageContext.ShellPage is not null &&
			_tagsContext.TaggedItems.Any();

		public OpenAllTaggedActions()
		{
			_pageContext = Ioc.Default.GetRequiredService<IContentPageContext>();
			_tagsContext = Ioc.Default.GetRequiredService<ITagsContext>();

			_pageContext.PropertyChanged += Context_PropertyChanged;
			_tagsContext.PropertyChanged += Context_PropertyChanged;
		}

		public async Task ExecuteAsync()
		{
			var filePaths = _tagsContext.TaggedItems
				.Where(item => !item.isFolder)
				.Select(f => f.path)
				.ToList();
			
			var folderPaths = _tagsContext
				.TaggedItems
				.Where(item => item.isFolder)
				.Select(f => f.path)
				.ToList();
			
			await Task.WhenAll(filePaths.Select(path => NavigationHelpers.OpenPath(path, _pageContext.ShellPage!)));

			foreach (var path in folderPaths) 
				await NavigationHelpers.OpenPathInNewTab(path, false);
		}

		private void Context_PropertyChanged(object? sender, PropertyChangedEventArgs e)
		{
			switch (e.PropertyName)
			{
				case nameof(IContentPageContext.ShellPage):
				case nameof(ITagsContext.TaggedItems):
					OnPropertyChanged(nameof(IsExecutable));
					break;
			}
		}
	}
}
Repository: files-community/Files
File: src/Files.App/Utils/Storage/Operations/FileOperationsHelpers.cs (C#)

Code before:
				{
					taskbar.SetProgressValue(OwnerWindow, (ulong)Progress, 100);
				}
				else
				{
					taskbar.SetProgressState(OwnerWindow, Shell32.TBPFLAG.TBPF_NOPROGRESS);
				}
			}

			public void WaitForCompletion()
			{
				operationsCompletedEvent.WaitOne();
			}

			protected override void Dispose(bool disposing)
			{
				if (disposing)
				{
					operationsCompletedEvent?.Dispose();
					if (taskbar is not null)
						Marshal.ReleaseComObject(taskbar);
				}
			}
		}

		public static string GetIncrementalName(bool overWriteOnCopy,string? filePathToCheck)
		{
			if (filePathToCheck == null)
				return null;
			else
			{
				if ((!Path.Exists(filePathToCheck)) || overWriteOnCopy)
					return Path.GetFileName(filePathToCheck);
				else
				{
					int i = 2;
					string genfilepath = string.Concat([filePathToCheck, " - (", i.ToString(), ")"]);
					while (Path.Exists(genfilepath))
					{
						i = i+1;
						genfilepath = string.Concat([filePathToCheck, " - (", i.ToString(), ")"]);
					}
					return Path.GetFileName(genfilepath);
				}				
			}
		}
	}
}


Review comment (suggestion):
A space is needed after the coma:
```suggestion
		public static string GetIncrementalName(bool overWriteOnCopy, string? filePathToCheck)
```

Code after:
				if (operations.Any())
				{
					taskbar.SetProgressValue(OwnerWindow, (ulong)Progress, 100);
				}
				else
				{
					taskbar.SetProgressState(OwnerWindow, Shell32.TBPFLAG.TBPF_NOPROGRESS);
				}
			}

			public void WaitForCompletion()
			{
				operationsCompletedEvent.WaitOne();
			}

			protected override void Dispose(bool disposing)
			{
				if (disposing)
				{
					operationsCompletedEvent?.Dispose();
					if (taskbar is not null)
						Marshal.ReleaseComObject(taskbar);
				}
			}
		}

		private static string GetIncrementalName(bool overWriteOnCopy, string? filePathToCheck, string? filePathToCopy)
		{
			if (filePathToCheck == null)
				return null;			

			if ((!Path.Exists(filePathToCheck)) || overWriteOnCopy || filePathToCheck == filePathToCopy)
				return Path.GetFileName(filePathToCheck);

			var index = 2;
			var filePath = filePathToCheck;
			if (Path.HasExtension(filePathToCheck))
				filePath = filePathToCheck.Substring(0, filePathToCheck.LastIndexOf("."));

			Func<int, string> genFilePath = x => string.Concat([filePath, " (", x.ToString(), ")", Path.GetExtension(filePathToCheck)]);

			while (Path.Exists(genFilePath(index)))
				index++;

			return Path.GetFileName(genFilePath(index));
		}		
	}
}
Repository: files-community/Files
File: src/Files.App/Utils/Storage/Operations/FileOperationsHelpers.cs (C#)

Code before:
				operationsCompletedEvent.WaitOne();
			}

			protected override void Dispose(bool disposing)
			{
				if (disposing)
				{
					operationsCompletedEvent?.Dispose();
					if (taskbar is not null)
						Marshal.ReleaseComObject(taskbar);
				}
			}
		}

		public static string GetIncrementalName(bool overWriteOnCopy,string? filePathToCheck)
		{
			if (filePathToCheck == null)
				return null;
			else
			{
				if ((!Path.Exists(filePathToCheck)) || overWriteOnCopy)
					return Path.GetFileName(filePathToCheck);
				else
				{
					int i = 2;
					string genfilepath = string.Concat([filePathToCheck, " - (", i.ToString(), ")"]);
					while (Path.Exists(genfilepath))
					{
						i = i+1;
						genfilepath = string.Concat([filePathToCheck, " - (", i.ToString(), ")"]);
					}
					return Path.GetFileName(genfilepath);
				}				
			}
		}
	}
}


Review comment (suggestion):
`string.Concat([filePathToCheck, " - (", i.ToString(), ")"]);` could be wrapped into a lambda to be called twice to reduce duplication.

Code after:
			{
				operationsCompletedEvent.WaitOne();
			}

			protected override void Dispose(bool disposing)
			{
				if (disposing)
				{
					operationsCompletedEvent?.Dispose();
					if (taskbar is not null)
						Marshal.ReleaseComObject(taskbar);
				}
			}
		}

		private static string GetIncrementalName(bool overWriteOnCopy, string? filePathToCheck, string? filePathToCopy)
		{
			if (filePathToCheck == null)
				return null;			

			if ((!Path.Exists(filePathToCheck)) || overWriteOnCopy || filePathToCheck == filePathToCopy)
				return Path.GetFileName(filePathToCheck);

			var index = 2;
			var filePath = filePathToCheck;
			if (Path.HasExtension(filePathToCheck))
				filePath = filePathToCheck.Substring(0, filePathToCheck.LastIndexOf("."));

			Func<int, string> genFilePath = x => string.Concat([filePath, " (", x.ToString(), ")", Path.GetExtension(filePathToCheck)]);

			while (Path.Exists(genFilePath(index)))
				index++;

			return Path.GetFileName(genFilePath(index));
		}		
	}
}
Repository: files-community/Files
File: src/Files.App/Utils/Storage/Operations/FileOperationsHelpers.cs (C#)

Code before:
				if (disposing)
				{
					operationsCompletedEvent?.Dispose();
					if (taskbar is not null)
						Marshal.ReleaseComObject(taskbar);
				}
			}
		}

		public static string GetIncrementalName(bool overWriteOnCopy, string? filePathToCheck, string? filePathToCopy)
		{
			if (filePathToCheck == null)
				return null;			

			if ((!Path.Exists(filePathToCheck)) || overWriteOnCopy || filePathToCheck == filePathToCopy)
				return Path.GetFileName(filePathToCheck);

			int i = 2;
			string filePath = filePathToCheck;
			if (Path.HasExtension(filePathToCheck))
				filePath = filePathToCheck.Substring(0, filePathToCheck.LastIndexOf("."));

			Func<int, string> genFilePath = x => string.Concat([filePath, " (", x.ToString(), ")", Path.GetExtension(filePathToCheck)]);

			while (Path.HasExtension(filePathToCheck) ? File.Exists(genFilePath(i)) : Path.Exists(genFilePath(i)))
			{
				i = i+1;				
			}
			return Path.GetFileName(genFilePath(i));									
		}		
	}
}


Review comment (suggestion):
1. Remove brackets and insert a line break right before `return` statement.
2. Let's use `index++;` instead.

Code after:
			{
				if (disposing)
				{
					operationsCompletedEvent?.Dispose();
					if (taskbar is not null)
						Marshal.ReleaseComObject(taskbar);
				}
			}
		}

		private static string GetIncrementalName(bool overWriteOnCopy, string? filePathToCheck, string? filePathToCopy)
		{
			if (filePathToCheck == null)
				return null;			

			if ((!Path.Exists(filePathToCheck)) || overWriteOnCopy || filePathToCheck == filePathToCopy)
				return Path.GetFileName(filePathToCheck);

			var index = 2;
			var filePath = filePathToCheck;
			if (Path.HasExtension(filePathToCheck))
				filePath = filePathToCheck.Substring(0, filePathToCheck.LastIndexOf("."));

			Func<int, string> genFilePath = x => string.Concat([filePath, " (", x.ToString(), ")", Path.GetExtension(filePathToCheck)]);

			while (Path.Exists(genFilePath(index)))
				index++;

			return Path.GetFileName(genFilePath(index));
		}		
	}
}
Repository: files-community/Files
File: src/Files.App/Utils/Storage/Operations/FileOperationsHelpers.cs (C#)

Code before:
			{
				if (disposing)
				{
					operationsCompletedEvent?.Dispose();
					if (taskbar is not null)
						Marshal.ReleaseComObject(taskbar);
				}
			}
		}

		public static string GetIncrementalName(bool overWriteOnCopy, string? filePathToCheck, string? filePathToCopy)
		{
			if (filePathToCheck == null)
				return null;			

			if ((!Path.Exists(filePathToCheck)) || overWriteOnCopy || filePathToCheck == filePathToCopy)
				return Path.GetFileName(filePathToCheck);

			int index = 2;
			string filePath = filePathToCheck;
			if (Path.HasExtension(filePathToCheck))
				filePath = filePathToCheck.Substring(0, filePathToCheck.LastIndexOf("."));

			Func<int, string> genFilePath = x => string.Concat([filePath, " (", x.ToString(), ")", Path.GetExtension(filePathToCheck)]);

			while (Path.HasExtension(filePathToCheck) ? File.Exists(genFilePath(index)) : Path.Exists(genFilePath(index)))			
				index++;				

			return Path.GetFileName(genFilePath(index));									
		}		
	}
}


Review comment (suggestion):
```suggestion
			while (Path.Exists(genFilePath(index)))			
```
Some files don't have extensions, and it is enough to just use `Path.Exists()` in the first place.

Code after:
			protected override void Dispose(bool disposing)
			{
				if (disposing)
				{
					operationsCompletedEvent?.Dispose();
					if (taskbar is not null)
						Marshal.ReleaseComObject(taskbar);
				}
			}
		}

		private static string GetIncrementalName(bool overWriteOnCopy, string? filePathToCheck, string? filePathToCopy)
		{
			if (filePathToCheck == null)
				return null;			

			if ((!Path.Exists(filePathToCheck)) || overWriteOnCopy || filePathToCheck == filePathToCopy)
				return Path.GetFileName(filePathToCheck);

			var index = 2;
			var filePath = filePathToCheck;
			if (Path.HasExtension(filePathToCheck))
				filePath = filePathToCheck.Substring(0, filePathToCheck.LastIndexOf("."));

			Func<int, string> genFilePath = x => string.Concat([filePath, " (", x.ToString(), ")", Path.GetExtension(filePathToCheck)]);

			while (Path.Exists(genFilePath(index)))
				index++;

			return Path.GetFileName(genFilePath(index));
		}		
	}
}
Repository: files-community/Files
File: src/Files.App/Utils/Storage/Operations/FileOperationsHelpers.cs (C#)

Code before:
			public void WaitForCompletion()
			{
				operationsCompletedEvent.WaitOne();
			}

			protected override void Dispose(bool disposing)
			{
				if (disposing)
				{
					operationsCompletedEvent?.Dispose();
					if (taskbar is not null)
						Marshal.ReleaseComObject(taskbar);
				}
			}
		}

		public static string GetIncrementalName(bool overWriteOnCopy, string? filePathToCheck, string? filePathToCopy)
		{
			if (filePathToCheck == null)
				return null;			

			if ((!Path.Exists(filePathToCheck)) || overWriteOnCopy || filePathToCheck == filePathToCopy)
				return Path.GetFileName(filePathToCheck);

			int index = 2;
			string filePath = filePathToCheck;
			if (Path.HasExtension(filePathToCheck))
				filePath = filePathToCheck.Substring(0, filePathToCheck.LastIndexOf("."));

			Func<int, string> genFilePath = x => string.Concat([filePath, " (", x.ToString(), ")", Path.GetExtension(filePathToCheck)]);

			while (Path.Exists(genFilePath(index)))			
				index++;				

			return Path.GetFileName(genFilePath(index));									
		}		
	}
}


Review comment (suggestion):
```suggestion
			var index = 2;
			var filePath = filePathToCheck;
```

Code after:
			public void WaitForCompletion()
			{
				operationsCompletedEvent.WaitOne();
			}

			protected override void Dispose(bool disposing)
			{
				if (disposing)
				{
					operationsCompletedEvent?.Dispose();
					if (taskbar is not null)
						Marshal.ReleaseComObject(taskbar);
				}
			}
		}

		private static string GetIncrementalName(bool overWriteOnCopy, string? filePathToCheck, string? filePathToCopy)
		{
			if (filePathToCheck == null)
				return null;			

			if ((!Path.Exists(filePathToCheck)) || overWriteOnCopy || filePathToCheck == filePathToCopy)
				return Path.GetFileName(filePathToCheck);

			var index = 2;
			var filePath = filePathToCheck;
			if (Path.HasExtension(filePathToCheck))
				filePath = filePathToCheck.Substring(0, filePathToCheck.LastIndexOf("."));

			Func<int, string> genFilePath = x => string.Concat([filePath, " (", x.ToString(), ")", Path.GetExtension(filePathToCheck)]);

			while (Path.Exists(genFilePath(index)))
				index++;

			return Path.GetFileName(genFilePath(index));
		}		
	}
}
Repository: files-community/Files
File: src/Files.App/Utils/Storage/Operations/FileOperationsHelpers.cs (C#)

Code before:
				{
					taskbar.SetProgressValue(OwnerWindow, (ulong)Progress, 100);
				}
				else
				{
					taskbar.SetProgressState(OwnerWindow, Shell32.TBPFLAG.TBPF_NOPROGRESS);
				}
			}

			public void WaitForCompletion()
			{
				operationsCompletedEvent.WaitOne();
			}

			protected override void Dispose(bool disposing)
			{
				if (disposing)
				{
					operationsCompletedEvent?.Dispose();
					if (taskbar is not null)
						Marshal.ReleaseComObject(taskbar);
				}
			}
		}

		public static string GetIncrementalName(bool overWriteOnCopy, string? filePathToCheck, string? filePathToCopy)
		{
			if (filePathToCheck == null)
				return null;			

			if ((!Path.Exists(filePathToCheck)) || overWriteOnCopy || filePathToCheck == filePathToCopy)
				return Path.GetFileName(filePathToCheck);

			var index = 2;
			var filePath = filePathToCheck;
			if (Path.HasExtension(filePathToCheck))
				filePath = filePathToCheck.Substring(0, filePathToCheck.LastIndexOf("."));

			Func<int, string> genFilePath = x => string.Concat([filePath, " (", x.ToString(), ")", Path.GetExtension(filePathToCheck)]);

			while (Path.Exists(genFilePath(index)))			
				index++;				

			return Path.GetFileName(genFilePath(index));									
		}		
	}
}


Review comment (suggestion):
```suggestion
		private static string GetIncrementalName(bool overWriteOnCopy, string? filePathToCheck, string? filePathToCopy)
```
Sorry, I forgot one comment.
It should be `private` if it doesn't need to be public.

Code after:
					taskbar.SetProgressValue(OwnerWindow, (ulong)Progress, 100);
				}
				else
				{
					taskbar.SetProgressState(OwnerWindow, Shell32.TBPFLAG.TBPF_NOPROGRESS);
				}
			}

			public void WaitForCompletion()
			{
				operationsCompletedEvent.WaitOne();
			}

			protected override void Dispose(bool disposing)
			{
				if (disposing)
				{
					operationsCompletedEvent?.Dispose();
					if (taskbar is not null)
						Marshal.ReleaseComObject(taskbar);
				}
			}
		}

		private static string GetIncrementalName(bool overWriteOnCopy, string? filePathToCheck, string? filePathToCopy)
		{
			if (filePathToCheck == null)
				return null;			

			if ((!Path.Exists(filePathToCheck)) || overWriteOnCopy || filePathToCheck == filePathToCopy)
				return Path.GetFileName(filePathToCheck);

			var index = 2;
			var filePath = filePathToCheck;
			if (Path.HasExtension(filePathToCheck))
				filePath = filePathToCheck.Substring(0, filePathToCheck.LastIndexOf("."));

			Func<int, string> genFilePath = x => string.Concat([filePath, " (", x.ToString(), ")", Path.GetExtension(filePathToCheck)]);

			while (Path.Exists(genFilePath(index)))
				index++;

			return Path.GetFileName(genFilePath(index));
		}		
	}
}