AddrRange: Simplify Range by removing stream input/output

Information
Submitter:	Andreas Hansson
Repository:	gem5
Branch:	default
Bugs:
Depends On:
Reviewers
Groups:	Default
People:

Description

Changeset 9186:caba9377b12a
---------------------------
AddrRange: Simplify Range by removing stream input/output

This patch simplifies the Range class in preparation for the
introduction of a more specific AddrRange class that allows
interleaving/striping.

The only place where the parsing was used was in the unit test.

Testing Done

util/regress all passing (disregarding t1000 and eio)

Issue Summary

Description	From	Last Updated	Status
maybe keep the test and just don't use the parsing?	Ali Saidi	Sept. 12, 2012, 2:19 a.m.	Dropped

i hate to see unit tests go. How about we keep the tests but loose the parsing?

src/unittest/rangetest.cc (Diff revision 1)

maybe keep the test and just don't use the parsing?

Show all issues

Andreas Hansson Sept. 11, 2012, 2:44 a.m. (Sept. 11, 2012, 2:44 a.m.)
```
Will do
```

Andreas Hansson Sept. 11, 2012, 8:29 p.m. (Sept. 11, 2012, 8:29 p.m.)

After quite some time spent on trying to reach a good solution my conclusion is that it is not worth keeping this test. Most of the operators that are used are pruned in the Range<T> -> AddrRange patch that follows, and thus the whole test pretty much disappears. I would rather suggest to stick to the rangemaptest that also includes the basic range functionality and is updated to accommodate the AddrRange class in the follow-up patch.

Sounds reasonable?

Ali Saidi Sept. 11, 2012, 10:45 p.m. (Sept. 11, 2012, 10:45 p.m.)
```
yup
```

Ship It!

You have a pending review.

Review Board 2.0.15

This change has been marked as submitted.

Screenshots

Files

Issue Summary

Status: Closed (submitted)