Sounds good to me, this is likely a rare code path anyways
On May 22, 2014 5:10 AM, "Tony Wasserka" <notifications@github.com> wrote:

> In Source/Core/Core/HW/EXI_DeviceMemoryCard.cpp:
>
> >
> > -	pFile.WriteBytes(data->memcardContent, data->memcardSize);
> > +	if (!File::Exists(strDirectoryName)) // first use of memcard folder, migrate automatically
> > +	{
> > +		MigrateFromMemcardFile(strDirectoryName + DIR_SEP, card_index);
> > +	}
> > +	else if (!File::IsDirectory(strDirectoryName))
> > +	{
> > +		if (File::Rename(strDirectoryName, strDirectoryName + ".original"))
> > +		{
> > +			PanicAlertT("%s was not a directory, moved to *.original", strDirectoryName.c_str());
>
> This does not deserve a PanicAlert since it doesn't prevent Dolphin from
> keeping running by crashing or anything like that (or if it does, this is
> not apparent from the code).
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dolphin-emu/dolphin/pull/243/files#r12946383>
> .
>