This repository was archived by the owner on Jul 19, 2025. It is now read-only.

Commit 52f3c2c

committed

skip file message: format everything in one call

I noticed that some logs reporting skip file messages seemed to interleave multiple messages: That's not terribly surprising due to concurrency. This changes the message formatting a bit & does it all in one method call to avoid the threading issues.

1 parent 371a7cf commit 52f3c2cCopy full SHA for 52f3c2c

File tree

1 file changed

-2

lines changed

lib/cc/engine/analyzers
- analyzer_base.rb

1 file changed

-2

lines changed

`‎lib/cc/engine/analyzers/analyzer_base.rb‎`

Lines changed: 1 addition & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -18,8 +18,7 @@ def initialize(engine_config:)`
`18`	`18`	`def run(file)`
`19`	`19`	`process_file(file)`
`20`	`20`	`rescue *RESCUABLE_ERRORS => ex`
`21`		`- $stderr.puts("Skipping file #{file} due to exception:")`
`22`		`- $stderr.puts("(#{ex.class}) #{ex.message} #{ex.backtrace.join("\n")}")`
	`21`	`+ $stderr.puts("Skipping file #{file} due to exception (#{ex.class}): #{ex.message}\n#{ex.backtrace.join("\n")}")`
`23`	`22`	`rescue => ex`
`24`	`23`	`$stderr.puts("#{ex.class} error occurred processing file #{file}: aborting.")`
`25`	`24`	`raise ex`

0 commit comments

Comments

(0)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit 52f3c2c

File tree

1 file changed

1 file changed

`‎lib/cc/engine/analyzers/analyzer_base.rb‎`

0 commit comments