Skip to content

Commit

Permalink
Fixed "unused variable" errors and trailing whitespace.
Browse files Browse the repository at this point in the history
  • Loading branch information
vadimcn committed Dec 12, 2013
1 parent 544ed03 commit d4d1310
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/librustc/back/manifest.rs
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ use driver::session::Session;
use std::path::Path;

#[cfg(not(windows))]
pub fn postprocess_executable(sess: Session, filename: &Path) {}
pub fn postprocess_executable(_sess: Session, _filename: &Path) {}

#[cfg(windows)]
pub fn postprocess_executable(sess: Session, filename: &Path) {
Expand Down Expand Up @@ -76,7 +76,7 @@ mod windows {
unsafe{ transmute(id) }
}

pub fn embed_manifest(filename: &Path,
pub fn embed_manifest(filename: &Path,
manifest: &str) -> Result<(),~str> {
unsafe {
let hUpdate = as_utf16_p(filename.as_str().unwrap(), |path| {
Expand Down

13 comments on commit d4d1310

@bors
Copy link
Contributor

@bors bors commented on d4d1310 Dec 12, 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 vadimcn@d4d1310

@bors
Copy link
Contributor

@bors bors commented on d4d1310 Dec 12, 2013

Choose a reason for hiding this comment

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

merging vadimcn/rust/manifest = d4d1310 into auto

@bors
Copy link
Contributor

@bors bors commented on d4d1310 Dec 12, 2013

Choose a reason for hiding this comment

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

vadimcn/rust/manifest = d4d1310 merged ok, testing candidate = 937ae504

@bors
Copy link
Contributor

@bors bors commented on d4d1310 Dec 12, 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 d4d1310 Dec 12, 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 vadimcn@d4d1310

@bors
Copy link
Contributor

@bors bors commented on d4d1310 Dec 12, 2013

Choose a reason for hiding this comment

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

merging vadimcn/rust/manifest = d4d1310 into auto

@bors
Copy link
Contributor

@bors bors commented on d4d1310 Dec 12, 2013

Choose a reason for hiding this comment

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

vadimcn/rust/manifest = d4d1310 merged ok, testing candidate = 4722b67b

@bors
Copy link
Contributor

@bors bors commented on d4d1310 Dec 12, 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 d4d1310 Dec 13, 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 vadimcn@d4d1310

@bors
Copy link
Contributor

@bors bors commented on d4d1310 Dec 13, 2013

Choose a reason for hiding this comment

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

merging vadimcn/rust/manifest = d4d1310 into auto

@bors
Copy link
Contributor

@bors bors commented on d4d1310 Dec 13, 2013

Choose a reason for hiding this comment

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

vadimcn/rust/manifest = d4d1310 merged ok, testing candidate = 09bb3c0

@bors
Copy link
Contributor

@bors bors commented on d4d1310 Dec 13, 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 d4d1310 Dec 13, 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 = 09bb3c0

Please sign in to comment.