ClaytonIndustries/QuickSave

Function "QuickSaveRaw.SaveBytes" performs the wrong check to throw exception

Closed this issue · 1 comments

stackOverflower92 commented

The function "QuickSaveRaw.SaveBytes" is missing a "not" operator when checking if the operation was succesful.

File: QuickSaveRaw.cs
Line: 73

public static void SaveBytes(string filename, byte[] content)
{
    if (FileAccess.SaveBytes(filename, true, content)) // <- Here. Shouldn't it be "if (!FileAccess.[...])" ?
    {
        throw new QuickSaveException("Failed to write to file");
    }
}
ClaytonIndustries commented

Good spot, I'll get it fixed

  • 🎉1