Skip to content

Commit

Permalink
Another followup on #6009.
Browse files Browse the repository at this point in the history
Odd that my earlier make checks did not catch this.
  • Loading branch information
pnkfelix committed Sep 4, 2013
1 parent 6fbb64a commit c326ff0
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/libstd/condition.rs
Original file line number Diff line number Diff line change
Expand Up @@ -193,7 +193,8 @@ mod test {
// Issue #6009
mod m {
condition! {
sadness: int -> int;
// #6009, #8215: should this truly need a `pub` for access from n?
pub sadness: int -> int;
}

mod n {
Expand Down

9 comments on commit c326ff0

@bors
Copy link
Contributor

@bors bors commented on c326ff0 Sep 4, 2013

Choose a reason for hiding this comment

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

saw approval from alexcrichton
at pnkfelix@c326ff0

@bors
Copy link
Contributor

@bors bors commented on c326ff0 Sep 4, 2013

Choose a reason for hiding this comment

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

merging pnkfelix/rust/fsk-followup-on-6009-rebased = c326ff0 into auto

@bors
Copy link
Contributor

@bors bors commented on c326ff0 Sep 4, 2013

Choose a reason for hiding this comment

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

pnkfelix/rust/fsk-followup-on-6009-rebased = c326ff0 merged ok, testing candidate = c2e2ba5d

@bors
Copy link
Contributor

@bors bors commented on c326ff0 Sep 4, 2013

@bors
Copy link
Contributor

@bors bors commented on c326ff0 Sep 4, 2013

Choose a reason for hiding this comment

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

saw approval from alexcrichton
at pnkfelix@c326ff0

@bors
Copy link
Contributor

@bors bors commented on c326ff0 Sep 4, 2013

Choose a reason for hiding this comment

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

merging pnkfelix/rust/fsk-followup-on-6009-rebased = c326ff0 into auto

@bors
Copy link
Contributor

@bors bors commented on c326ff0 Sep 4, 2013

Choose a reason for hiding this comment

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

pnkfelix/rust/fsk-followup-on-6009-rebased = c326ff0 merged ok, testing candidate = b161e09

@bors
Copy link
Contributor

@bors bors commented on c326ff0 Sep 5, 2013

@bors
Copy link
Contributor

@bors bors commented on c326ff0 Sep 5, 2013

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 = b161e09

Please sign in to comment.