New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feedback for *which* namespace is already linked. #1175

Merged
merged 2 commits into from Feb 10, 2015

Conversation

Projects
None yet
3 participants
@Azaezel
Contributor

Azaezel commented Feb 10, 2015

should probably look into enhancing assertwarn to be able to take a formatted string at some point...

feedback for *which* namespace is already linked. should probably loo…
…k into enhancing assertwarn to be able to take a formatted string at some point...
@crabmusket

This comment has been minimized.

Show comment
Hide comment
@crabmusket

crabmusket Feb 10, 2015

Collaborator

Would enjoy a PR for this that looks more like:

if( mNameSpace )
{
   Con::warnf(...);
   return;
}
Collaborator

crabmusket commented on 793d7bf Feb 10, 2015

Would enjoy a PR for this that looks more like:

if( mNameSpace )
{
   Con::warnf(...);
   return;
}

@crabmusket crabmusket added this to the 3.7 milestone Feb 10, 2015

crabmusket added a commit that referenced this pull request Feb 10, 2015

Merge pull request #1175 from Azaezel/linkNamespacesDebugOutput
feedback for *which* namespace is already linked.

@crabmusket crabmusket merged commit 7d4aeb8 into GarageGames:development Feb 10, 2015

@crabmusket crabmusket deleted the Azaezel:linkNamespacesDebugOutput branch Feb 10, 2015

@jamesu

This comment has been minimized.

Show comment
Hide comment
@jamesu

jamesu Feb 12, 2015

Contributor

I just use avar("%s", ....)

Contributor

jamesu commented Feb 12, 2015

I just use avar("%s", ....)

@Azaezel Azaezel restored the Azaezel:linkNamespacesDebugOutput branch Mar 30, 2015

@Azaezel Azaezel deleted the Azaezel:linkNamespacesDebugOutput branch May 29, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment