mem: Page Table long lines
Review Request #2461 - Created Sept. 30, 2014 and submitted
| Information | |
|---|---|
| Alexandru Dutu | |
| gem5 | |
| default | |
| Reviewers | |
| Default | |
Changeset 10425:ad9f6339024a --------------------------- mem: Page Table long lines Trimmed down all the lines greater than 78 characters.
Quick regression testing done.
Issue Summary
3
1
2
0
| Description | From | Last Updated | Status |
|---|---|---|---|
| I'm not sure I'm a massive fan...but then again, I don't quite know what to suggest instead. | Andreas Hansson | Sept. 30, 2014, 1:45 p.m. | Open |
Posted (Sept. 30, 2014, 12:57 p.m.)
Should this not be folded into the original patch?
Posted (Sept. 30, 2014, 1:45 p.m.)
-
src/mem/multi_level_page_table_impl.hh (Diff revision 1) -
I'm not sure I'm a massive fan...but then again, I don't quite know what to suggest instead.
Review request changed
Updated (Sept. 30, 2014, 2:25 p.m.)
Change Summary:
Modified the long for statement.
Description: |
|
||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Diff: |
Revision 2 (+25 -12) |
-
src/mem/multi_level_page_table_impl.hh (Diff revision 2) -
i - 1
-
src/mem/page_table.cc (Diff revision 2) -
Same here as the other
Some very minor things. No need to re-post once they are fixed.
