Comments on: Wavefront Obj Mesh Loader

By: A Tiny Wavefront Object Loader – Part I | Benjamin Sommer Weblog

A Tiny Wavefront Object Loader – Part I | Benjamin Sommer Weblog — Sun, 12 Feb 2012 16:01:11 +0000

[...] One unique vertex being reused by several faces should only be stored in one place. Take the post by limegarden.net, where a triangular mesh is returned – a mesh where all vertex, normal and texture [...]

By: Wouter Lindenhof

Wouter Lindenhof — Mon, 31 Jan 2011 13:03:26 +0000

Yep, the wavefront obj file format (and the accompanying wavefront mtl format) have many more capabilities then the above loader supports. The above loader only supports a few simple things.

However for what I needed the above was good enough. Hmm… Might be fun to write an fully capable OBJ mesh loader.

By: Alexander Smirnov

Alexander Smirnov — Mon, 31 Jan 2011 09:44:21 +0000

Man… You should read wavefront obj file format specification, it’s not that simple.

By: Wouter Lindenhof

Wouter Lindenhof — Sun, 26 Sep 2010 06:48:37 +0000

Thanks for your feedback. I had completely forgotten that I could use streams. I will fix that as soon as possible (and give credits).

It’s true that because of RAII (Resource Acquisition Is Initialization) you don’t need to close stream, so I could leave it out, but I like to clean things up when I no longer need them. The reason for that is because it can reduce the size of memory leaks in case of exceptions.

By: Dale Weiler

Dale Weiler — Sat, 25 Sep 2010 06:17:24 +0000

you don’t need to close it, or do that memset with STL, and why c_str(), again you don’t need that.

try removing lines 82-94 and replacing them with what I have below:


std::string fileBuffer;                         // char required memset (C function!)
std::stringstream strStream;                    // Normal string stream
std::ifstream filestream(filename);             //no c_str() required

while(std::getline(filestream, fileBuffer))     //this is the gold since when it's finished it auto closes
                                                // which means line 118 can come right out (more SPEED THIS WAY!)

By: Wouter Lindenhof

Wouter Lindenhof — Tue, 22 Jun 2010 14:48:49 +0000

Yes, it should (and it was). Thanks for pointing it out.

By: Onno

Onno — Tue, 22 Jun 2010 09:14:42 +0000

small question on line 85:

should “&&” not be “&&”?