Oh derp, you're right. I changed everything to LogLevel except there.

I'd been thinking of making Level an enum, but that stops people from using
other levels.
On May 2, 2014 7:44 PM, "Huon Wilson" <notifications@github.com> wrote:

> In src/liblog/lib.rs:
>
> >  }
> >
> >  struct DefaultLogger {
> >      handle: LineBufferedWriter<io::stdio::StdWriter>,
> >  }
> >
> > +struct LevelStr(u32);
>
> Seems like external loggers might find it useful too.
>
> Maybe the log level could just be
>
> pub struct LogLevel {
>     pub level: u32}
>
> which implements Show as a string, but still provides easy access to the
> internal number. (I guess the field could be private too, and offer a
> retrieval method.)
>
> (In any case, with or without any of these changes, this should be storing
> a LogLevel, not a u32.)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mozilla/rust/pull/13912/files#r12254268>
> .
>