Commit 3c74e772 by Jamie Madill

Revert "Use a std::stringstream for Program's InfoLog."

Missing one character for the log length. BUG=angleproject:307,angleproject:992 This reverts commit 8ae74e11. Change-Id: Ifb3658c324cb0aa56ee31352a0fc0e6759979f7b Reviewed-on: https://chromium-review.googlesource.com/269848Reviewed-by: 's avatarJamie Madill <jmadill@chromium.org> Tested-by: 's avatarJamie Madill <jmadill@chromium.org>
parent 27464aa9
...@@ -55,31 +55,38 @@ AttributeBindings::~AttributeBindings() ...@@ -55,31 +55,38 @@ AttributeBindings::~AttributeBindings()
{ {
} }
InfoLog::InfoLog() InfoLog::InfoLog() : mInfoLog(NULL)
{ {
} }
InfoLog::~InfoLog() InfoLog::~InfoLog()
{ {
delete[] mInfoLog;
} }
size_t InfoLog::getLength() const
int InfoLog::getLength() const
{ {
return mStream.str().length(); if (!mInfoLog)
{
return 0;
}
else
{
return strlen(mInfoLog) + 1;
}
} }
void InfoLog::getLog(GLsizei bufSize, GLsizei *length, char *infoLog) void InfoLog::getLog(GLsizei bufSize, GLsizei *length, char *infoLog)
{ {
size_t index = 0; int index = 0;
if (bufSize > 0) if (bufSize > 0)
{ {
const std::string str(mStream.str()); if (mInfoLog)
if (!str.empty())
{ {
index = std::min(static_cast<size_t>(bufSize) - 1, str.length()); index = std::min(bufSize - 1, (int)strlen(mInfoLog));
memcpy(infoLog, str.c_str(), index); memcpy(infoLog, mInfoLog, index);
} }
infoLog[index] = '\0'; infoLog[index] = '\0';
...@@ -87,7 +94,7 @@ void InfoLog::getLog(GLsizei bufSize, GLsizei *length, char *infoLog) ...@@ -87,7 +94,7 @@ void InfoLog::getLog(GLsizei bufSize, GLsizei *length, char *infoLog)
if (length) if (length)
{ {
*length = static_cast<GLsizei>(index); *length = index;
} }
} }
...@@ -109,7 +116,7 @@ void InfoLog::appendSanitized(const char *message) ...@@ -109,7 +116,7 @@ void InfoLog::appendSanitized(const char *message)
} }
while (found != std::string::npos); while (found != std::string::npos);
mStream << message << "\n"; append("%s", msg.c_str());
} }
void InfoLog::append(const char *format, ...) void InfoLog::append(const char *format, ...)
...@@ -121,27 +128,52 @@ void InfoLog::append(const char *format, ...) ...@@ -121,27 +128,52 @@ void InfoLog::append(const char *format, ...)
va_list vararg; va_list vararg;
va_start(vararg, format); va_start(vararg, format);
std::string tempString(FormatString(format, vararg)); size_t infoLength = vsnprintf(NULL, 0, format, vararg);
va_end(vararg);
char *logPointer = NULL;
if (!mInfoLog)
{
mInfoLog = new char[infoLength + 2];
logPointer = mInfoLog;
}
else
{
size_t currentlogLength = strlen(mInfoLog);
char *newLog = new char[currentlogLength + infoLength + 2];
strcpy(newLog, mInfoLog);
delete[] mInfoLog;
mInfoLog = newLog;
logPointer = mInfoLog + currentlogLength;
}
va_start(vararg, format);
vsnprintf(logPointer, infoLength, format, vararg);
va_end(vararg); va_end(vararg);
mStream << tempString << "\n"; logPointer[infoLength] = 0;
strcpy(logPointer + infoLength, "\n");
} }
void InfoLog::reset() void InfoLog::reset()
{ {
if (mInfoLog)
{
delete [] mInfoLog;
mInfoLog = NULL;
}
} }
VariableLocation::VariableLocation() VariableLocation::VariableLocation()
: name(), : name(), element(0), index(0)
element(0),
index(0)
{ {
} }
VariableLocation::VariableLocation(const std::string &name, unsigned int element, unsigned int index) VariableLocation::VariableLocation(const std::string &name, unsigned int element, unsigned int index)
: name(name), : name(name), element(element), index(index)
element(element),
index(index)
{ {
} }
...@@ -160,8 +192,8 @@ Program::Program(rx::ProgramImpl *impl, ResourceManager *manager, GLuint handle) ...@@ -160,8 +192,8 @@ Program::Program(rx::ProgramImpl *impl, ResourceManager *manager, GLuint handle)
mValidated(false), mValidated(false),
mTransformFeedbackVaryings(), mTransformFeedbackVaryings(),
mTransformFeedbackBufferMode(GL_NONE), mTransformFeedbackBufferMode(GL_NONE),
mFragmentShader(nullptr), mFragmentShader(NULL),
mVertexShader(nullptr), mVertexShader(NULL),
mLinked(false), mLinked(false),
mDeleteStatus(false), mDeleteStatus(false),
mRefCount(0), mRefCount(0),
...@@ -178,12 +210,12 @@ Program::~Program() ...@@ -178,12 +210,12 @@ Program::~Program()
{ {
unlink(true); unlink(true);
if (mVertexShader != nullptr) if (mVertexShader != NULL)
{ {
mVertexShader->release(); mVertexShader->release();
} }
if (mFragmentShader != nullptr) if (mFragmentShader != NULL)
{ {
mFragmentShader->release(); mFragmentShader->release();
} }
...@@ -228,7 +260,7 @@ bool Program::detachShader(Shader *shader) ...@@ -228,7 +260,7 @@ bool Program::detachShader(Shader *shader)
} }
mVertexShader->release(); mVertexShader->release();
mVertexShader = nullptr; mVertexShader = NULL;
} }
else if (shader->getType() == GL_FRAGMENT_SHADER) else if (shader->getType() == GL_FRAGMENT_SHADER)
{ {
...@@ -238,7 +270,7 @@ bool Program::detachShader(Shader *shader) ...@@ -238,7 +270,7 @@ bool Program::detachShader(Shader *shader)
} }
mFragmentShader->release(); mFragmentShader->release();
mFragmentShader = nullptr; mFragmentShader = NULL;
} }
else UNREACHABLE(); else UNREACHABLE();
...@@ -355,13 +387,13 @@ void Program::unlink(bool destroy) ...@@ -355,13 +387,13 @@ void Program::unlink(bool destroy)
if (mFragmentShader) if (mFragmentShader)
{ {
mFragmentShader->release(); mFragmentShader->release();
mFragmentShader = nullptr; mFragmentShader = NULL;
} }
if (mVertexShader) if (mVertexShader)
{ {
mVertexShader->release(); mVertexShader->release();
mVertexShader = nullptr; mVertexShader = NULL;
} }
} }
...@@ -520,7 +552,7 @@ Error Program::saveBinary(GLenum *binaryFormat, void *binary, GLsizei bufSize, G ...@@ -520,7 +552,7 @@ Error Program::saveBinary(GLenum *binaryFormat, void *binary, GLsizei bufSize, G
GLint Program::getBinaryLength() const GLint Program::getBinaryLength() const
{ {
GLint length; GLint length;
Error error = saveBinary(nullptr, nullptr, std::numeric_limits<GLint>::max(), &length); Error error = saveBinary(NULL, NULL, std::numeric_limits<GLint>::max(), &length);
if (error.isError()) if (error.isError())
{ {
return 0; return 0;
...@@ -551,7 +583,7 @@ unsigned int Program::getRefCount() const ...@@ -551,7 +583,7 @@ unsigned int Program::getRefCount() const
int Program::getInfoLogLength() const int Program::getInfoLogLength() const
{ {
return static_cast<int>(mInfoLog.getLength()); return mInfoLog.getLength();
} }
void Program::getInfoLog(GLsizei bufSize, GLsizei *length, char *infoLog) void Program::getInfoLog(GLsizei bufSize, GLsizei *length, char *infoLog)
......
...@@ -21,7 +21,6 @@ ...@@ -21,7 +21,6 @@
#include <GLSLANG/ShaderLang.h> #include <GLSLANG/ShaderLang.h>
#include <vector> #include <vector>
#include <sstream>
#include <string> #include <string>
#include <set> #include <set>
...@@ -67,14 +66,14 @@ class InfoLog : angle::NonCopyable ...@@ -67,14 +66,14 @@ class InfoLog : angle::NonCopyable
InfoLog(); InfoLog();
~InfoLog(); ~InfoLog();
size_t getLength() const; int getLength() const;
void getLog(GLsizei bufSize, GLsizei *length, char *infoLog); void getLog(GLsizei bufSize, GLsizei *length, char *infoLog);
void appendSanitized(const char *message); void appendSanitized(const char *message);
void append(const char *info, ...); void append(const char *info, ...);
void reset(); void reset();
private: private:
std::stringstream mStream; char *mInfoLog;
}; };
// Struct used for correlating uniforms/elements of uniform arrays to handles // Struct used for correlating uniforms/elements of uniform arrays to handles
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment