Thanks, I realized this and then forgot to make the change. In what module
can I find Servo opts?


On Wed, May 22, 2013 at 8:29 PM, Patrick Walton <notifications@github.com>wrote:

> In src/components/servo-util/time.rs:
>
> > +            buckets: [~[], ..12],
> > +            last_print: 0,
> > +        }
> > +    }
> > +
> > +    pub fn start(&mut self) {
> > +        loop {
> > +            let msg = self.port.recv();
> > +            self.handle_msg(msg);
> > +        }
> > +    }
> > +
> > +    priv fn handle_msg(&mut self, msg: (ProfilerMsg, uint)) {
> > +        let (prof_msg, t) = msg;
> > +        self.buckets[prof_msg as uint].push(t);
> > +        let verbose = getenv("SERVO_PROFILER");
>
> Check this when you start the profiler and cache the value. You don't want
> to keep hammering on the getenv call all the time, as it's expensive.
>
> Actually maybe this should be part of the Servo opts.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mozilla/servo/pull/475/files#r4353489>
> .
>