Skip to content

Commit

Permalink
appease the tidy gods with respect to a FIXME
Browse files Browse the repository at this point in the history
  • Loading branch information
nikomatsakis committed May 7, 2013
1 parent cd164cf commit 39a1190
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/librustc/middle/typeck/check/regionck.rs
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,7 @@ pub fn regionck_fn(fcx: @mut FnCtxt, blk: &ast::blk) {
}

fn regionck_visitor() -> rvt {
// FIXME(#3238) should use visit_pat, not visit_arm/visit_local,
// (*) FIXME(#3238) should use visit_pat, not visit_arm/visit_local,
// However, right now we run into an issue whereby some free
// regions are not properly related if they appear within the
// types of arguments that must be inferred. This could be
Expand All @@ -167,7 +167,7 @@ fn regionck_visitor() -> rvt {
visit::mk_vt(@visit::Visitor {visit_item: visit_item,
visit_expr: visit_expr,

//visit_pat: visit_pat, // (*) see FIXME above
//visit_pat: visit_pat, // (*) see above
visit_arm: visit_arm,
visit_local: visit_local,

Expand Down

5 comments on commit 39a1190

@bors
Copy link
Contributor

@bors bors commented on 39a1190 May 7, 2013

Choose a reason for hiding this comment

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

saw approval from nikomatsakis
at nikomatsakis@39a1190

@bors
Copy link
Contributor

@bors bors commented on 39a1190 May 7, 2013

Choose a reason for hiding this comment

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

merging nikomatsakis/rust/issue-5910-dyna-freeze = 39a1190 into auto

@bors
Copy link
Contributor

@bors bors commented on 39a1190 May 7, 2013

Choose a reason for hiding this comment

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

nikomatsakis/rust/issue-5910-dyna-freeze = 39a1190 merged ok, testing candidate = 05460fc

@bors
Copy link
Contributor

@bors bors commented on 39a1190 May 7, 2013

Choose a reason for hiding this comment

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

@bors
Copy link
Contributor

@bors bors commented on 39a1190 May 7, 2013

Choose a reason for hiding this comment

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

fast-forwarding incoming to auto = 05460fc

Please sign in to comment.