Skip to content
This repository was archived by the owner on Apr 30, 2021. It is now read-only.

if fmt is table and has __tostring__ then use it#8

Open
orian wants to merge 1 commit intoNeopallium:masterfrom
orian:patch-1
Open

if fmt is table and has __tostring__ then use it#8
orian wants to merge 1 commit intoNeopallium:masterfrom
orian:patch-1

Conversation

@orian
Copy link

@orian orian commented May 20, 2015

This makes logger behaviour more in sync with standard print function.

This makes logger behaviour more in sync with standard print function.
@Tieske
Copy link

Tieske commented Jan 10, 2020

just browsing through this...

There is no __tostring__ functionality in Lua, that must be some non-standard implementation. There only is the __tostring metamethod that can be put on a metatable. And that metamethod is already supported, as the logger in the end falls back on a tostring call which will invoke the __tostring method.

So this should not be merged imo.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants