Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

Fix code analysis problem introduced with #154 #156

Merged
merged 1 commit into from
Dec 4, 2019

Conversation

chick
Copy link
Collaborator

@chick chick commented Dec 4, 2019

The main fix is to correct an eta expansion notation in shl processing.
Did a minor bit of code cleanup found during fix.
Fix some too long lines etc.

Fix some too long lines etc.
@chick chick added the cleanup Fixing up stale code label Dec 4, 2019
@chick chick requested a review from ucbjrl December 4, 2019 19:31
@chick chick requested a review from a team as a code owner December 4, 2019 19:31
@chick chick self-assigned this Dec 4, 2019
Copy link
Contributor

@ucbjrl ucbjrl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm - in general, I prefer pure formatting changes in their own PR

@chick chick merged commit 547d249 into master Dec 4, 2019
@ucbjrl ucbjrl modified the milestones: 1.1.x, 1.1.3 Feb 4, 2020
ucbjrl pushed a commit that referenced this pull request Feb 19, 2020
Fix code analysis problem introduced with  #154

(cherry picked from commit 547d249)
ucbjrl added a commit that referenced this pull request Feb 19, 2020
* Merge pull request #150 from freechipsproject/fix-test-dir-name-making

Fixed bug in Treadles internal default test directory naming

(cherry picked from commit eefbdb9)

* Merge pull request #153 from freechipsproject/datastore-plugin-match-in-repl

Fixes incomplete match specification of a datastore plugin in repl

(cherry picked from commit 1fad437)

* Merge pull request #152 from freechipsproject/better-stop-result

Provide API for getting stop result in from treadle tester

(cherry picked from commit cf60353)

* Merge pull request #154 from freechipsproject/riscv-mini-fixes

Fixes for problem discovered in riscv-mini

(cherry picked from commit 5e7e480)

* Merge pull request #155 from freechipsproject/printf-format-like-verilator

Makes treadle printf output formatted more like verilator does

(cherry picked from commit ad79d0c)

* Merge pull request #156 from freechipsproject/fix-shl-eta-expansion

Fix code analysis problem introduced with  #154

(cherry picked from commit 547d249)

* Merge pull request #157 from freechipsproject/fix-vcd-root-name

The vcd filename can be wrong

(cherry picked from commit d8b18b9)

* Merge pull request #159 from freechipsproject/fix-printf-match-verilator-2

Make printf behavior more closely match verilator

(cherry picked from commit ea3b725)

* Bump minor version

Co-authored-by: Chick Markley <chick@qrhino.com>
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cleanup Fixing up stale code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants