diff options
| author | Chick Markley | 2017-05-18 12:23:40 -0700 |
|---|---|---|
| committer | GitHub | 2017-05-18 12:23:40 -0700 |
| commit | 9c50af20027801d8623edd1db2c63c4eb449b3ae (patch) | |
| tree | bee5ddcab07ca0e498c52a53f0b8d3e1c3b22293 /src/main/scala/firrtl/ExecutionOptionsManager.scala | |
| parent | d824c60c9643973e0ae9cddc5007b3d9592f8a52 (diff) | |
Upgrade Logging facility (#488)
* Upgrade Logging facility
Make thread-safe
Make logging by package name work
Use caching of class names to level for performance
Make some tests to show this working
* quick fix for dynamic logging variable
* A number of changes based on Adam's suggestions
Default LoggerState
But there is an invoke method now to handle threading issues. This should be propagated to other
projects Driver.execute methods
* Add built-in support for string capture of Logging
* Usability fixes for logging stuff. Settings made to the logger prior to execute/invoke will be passed along if possible.
* A couple style fixes
Comment and privatize Logger state
* Name and save string buffers used for logging
* Fix default logging state setting
Fix logging test, did not have change to command argument
* comment out logging in InlineInstanceTests
* Changed invoke to makeScope
Nested makeScopes share same state object
Removed earlier named string buffer implementation
* Better name for captor get data
* Add trace tests to make sure it works too
* Fix call into logger settings
Diffstat (limited to 'src/main/scala/firrtl/ExecutionOptionsManager.scala')
| -rw-r--r-- | src/main/scala/firrtl/ExecutionOptionsManager.scala | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/main/scala/firrtl/ExecutionOptionsManager.scala b/src/main/scala/firrtl/ExecutionOptionsManager.scala index 2e95ee0c..76b497ec 100644 --- a/src/main/scala/firrtl/ExecutionOptionsManager.scala +++ b/src/main/scala/firrtl/ExecutionOptionsManager.scala @@ -29,11 +29,11 @@ abstract class HasParser(applicationName: String) { * circuit and then set the topName from that if it has not already been set. */ case class CommonOptions( - topName: String = "", - targetDirName: String = ".", - globalLogLevel: LogLevel.Value = LogLevel.Error, - logToFile: Boolean = false, - logClassNames: Boolean = false, + topName: String = "", + targetDirName: String = ".", + globalLogLevel: LogLevel.Value = LogLevel.None, + logToFile: Boolean = false, + logClassNames: Boolean = false, classLogLevels: Map[String, LogLevel.Value] = Map.empty) extends ComposableOptions { def getLogFileName(optionsManager: ExecutionOptionsManager): String = { |