This repository was archived by the owner on Dec 23, 2025. It is now read-only.
Add -D option to specifiy file to receive trace output.#34
Open
brouhaha wants to merge 1 commit intoEtchedPixels:masterfrom
Open
Add -D option to specifiy file to receive trace output.#34brouhaha wants to merge 1 commit intoEtchedPixels:masterfrom
brouhaha wants to merge 1 commit intoEtchedPixels:masterfrom
Conversation
Owner
|
Makes a lot of sense. Will have a think about how best to handle this. |
Author
|
I didn't expect that you'd necessarily want to accept this PR as-is. On some projects, I've used something like this: That has the drawback of requiring a function call even if the trace is masked, which is OK for some things but probably not anything in the core of a simulator. A macro could be used instead, though I'm not a huge fan of macros. |
|
I don't think you really need this. I use
I use a Mac but it looks like the almost identical syntax works on Windows as well. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add -D option to specifiy file to receive trace output, though the default is still stderr. While redirection of stderr could be used without this change, the intent here is to allow stderr to be used for only error conditions. It is possible that some such output should go to both stderr and the trace file, though this commit does not do that.