[Potato Uno]

PHP Code:

stock bool CopyFile(const char[] Source, const char[] Destination, bool read_valve_fs=false)
{
    Handle f = OpenFile(Source, "r", read_valve_fs);
    Handle g = OpenFile(Destination, "w");
    
    int Buffer[1024];
    while (!IsEndOfFile(f))
    {
        int Size = ReadFile(f, Buffer, sizeof(Buffer), 1);
        WriteFile(g, Buffer, Size, 1);
    }
    
    delete f;
    delete g;
} 


Tried it with a bunch of MvM population files and the stock works correctly (use_valve_fs = false).

I tried it on a nav file from a VPK (read_valve_fs = true) and it didn't do the job properly, and I can't figure out why it doesn't work.

Test code: (don't criticize this code since it's a TEST.)

PHP Code:

public Action Trash(int iClient, int nArgs)
{
    char SrcPath[255], DstPath[255];
    Format(SrcPath, sizeof(SrcPath), "maps/cp_dustbowl.nav");
    Format(DstPath, sizeof(DstPath), "addons/sourcemod/configs/population/test/cp_dustbowl.nav");
    CopyFile(SrcPath, DstPath, true);
    return Plugin_Handled;
} 


File size of the nav in the vpk file: 1,170,989 bytes
File size of the copied nav file: 1,173,968 bytes

What's the deal with the 2979 byte discrepancy? What am I doing wrong?

If there is a better way to do this or a way to improve the stock, I am all ears for it. This has to work on all file types (including text files like .pop and binary files like .nav).

[Potato Uno]

Oh, it's also causing the watchdog timer to kill it if I read from a VPK file. Is this a bug or an actual thing?

[Impact123]

Try to open the files in binary mode like

PHP Code:

Handle f = OpenFile(Source, "rb", read_valve_fs); 


Here's some info about the watchdog in case that helps.

[Potato Uno]

That worked, thanks!

The watchdog timer is due to the fact that I read 1 byte at a time which caused the loop to run way too frequently. Changing it to 4 fixed the problem.