forks
/
arsd
mirror of https://github.com/adamdruppe/arsd.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

image, png: some forgotten imports

This commit is contained in:
Ketmar Dark 2018-03-21 13:51:27 +02:00 committed by Adam D. Ruppe
parent 3a63dbebcc
commit d9c2195882
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
image.d
View File

@ -231,7 +231,7 @@ public MemoryImage loadImageFromFile(T:const(char)[]) (T filename) {
if (fsz < 4) throw new Exception("cannot determine file format");
if (fsz > int.max/8) throw new Exception("image data too big");
auto data = new ubyte[](cast(uint)fsz);
scope(exit) GC.free(data.ptr); // this should be safe, as image will copy data to it's internal storage
scope(exit) { import core.memory : GC; GC.free(data.ptr); } // this should be safe, as image will copy data to it's internal storage
fl.rawReadExact(data);
return loadImageFromMemory(data);
case ImageFileFormat.Png: static if (is(T == string)) return readPng(filename); else return readPng(filename.idup);
@ -279,7 +279,7 @@ public MemoryImage loadImageFromFile (VFile fl) {
if (fsz < 4) throw new Exception("cannot determine file format");
if (fsz > int.max/8) throw new Exception("image data too big");
auto data = new ubyte[](cast(uint)fsz);
scope(exit) { GC.free(data.ptr); } // this should be safe, as image will copy data to it's internal storage
scope(exit) { import core.memory : GC; GC.free(data.ptr); } // this should be safe, as image will copy data to it's internal storage
fl.rawReadExact(data);
return loadImageFromMemory(data);
}

2
png.d
View File

@ -473,7 +473,7 @@ void writeImageToPngFile(in char[] filename, TrueColorImage image) {
fputc((c.checksum & 0x000000ff) >> 0, fp);
}
GC.free(com.ptr); // there is a reference to this in the PNG struct, but it is going out of scope here too, so who cares
{ import core.memory : GC; GC.free(com.ptr); } // there is a reference to this in the PNG struct, but it is going out of scope here too, so who cares
// just wanna make sure this crap doesn't stick around
}