Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
reorder some struct declarations in profiler
  • Loading branch information
Tim Kuehn committed Aug 29, 2013
1 parent cf4107e commit 1d2c038
Showing 1 changed file with 9 additions and 8 deletions.
17 changes: 9 additions & 8 deletions src/components/util/time.rs
Expand Up @@ -52,14 +52,6 @@ pub enum ProfilerCategory {
// FIXME(rust#8803): workaround for lack of CTFE function on enum types to return length
NumBuckets,
}
type ProfilerBuckets = TreeMap<ProfilerCategory, ~[float]>;

// back end of the profiler that handles data aggregation and performance metrics
pub struct Profiler {
port: Port<ProfilerMsg>,
buckets: ProfilerBuckets,
last_msg: Option<ProfilerMsg>,
}

impl ProfilerCategory {
// convenience function to not have to cast every time
Expand Down Expand Up @@ -99,6 +91,15 @@ impl ProfilerCategory {
}
}

type ProfilerBuckets = TreeMap<ProfilerCategory, ~[float]>;

// back end of the profiler that handles data aggregation and performance metrics
pub struct Profiler {
port: Port<ProfilerMsg>,
buckets: ProfilerBuckets,
last_msg: Option<ProfilerMsg>,
}

impl Profiler {
pub fn create(port: Port<ProfilerMsg>) {
let port = Cell::new(port);
Expand Down

5 comments on commit 1d2c038

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

saw approval from metajack
at tikue@1d2c038

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

merging tikue/servo/master = 1d2c038 into auto

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

tikue/servo/master = 1d2c038 merged ok, testing candidate = 65c6435

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

fast-forwarding master to auto = 65c6435

Please sign in to comment.