Passing FileName and LineNumber about willow

[nsinklier]

Hey @cnoon, thanks for the quick response! I definitely agree that it should be attached to every LogLevel, I just didn't want to suggest something so invasive outright :)

I was thinking that since I'm going to make these changes anyway, I could just send you the code to look at. Then, if you think it would be beneficial we could incorporate it, otherwise I'll just keep it for myself. I've only spent about an hour and it's almost done, so I don't think the change will be too time consuming, it's just a lot of lines of code changed.

[cnoon]

Hi @nsinklier,

This is certainly something we've been experimenting with as well and have decided to place it on top of Willow itself for now. If we wanted to do this directly in Willow, we'd probably want to support file, function, line, and column in each of the Logger APIs which would then forward those to the LogWriter APIs. That would allow writers to decide how to use them if they wanted.

Given the scope of this change, I'm not sure it's something you'd actually like to take on yourself since it would be a MAJOR version change. It's certainly something that we've been noodling on for a while.

Thoughts?

I personally think that the position belongs to a log message. Without the position, log messages are only half as valuable. That's why I implemented this feature request and opened a pull request.

Hey @cnoon, any plans to merge PR #60 in the next time?