Use checked NonZero constructor in obligation forest NodeIndex

SimonSapin · SimonSapin · commit 13d17adf6059 · 2017-07-22T20:38:40.000+02:00
… to remove an `unsafe` block.
diff --git a/src/librustc_data_structures/obligation_forest/node_index.rs b/src/librustc_data_structures/obligation_forest/node_index.rs
@@ -19,7 +19,7 @@ pub struct NodeIndex {
 impl NodeIndex {
     pub fn new(value: usize) -> NodeIndex {
         assert!(value < (u32::MAX as usize));
-        unsafe { NodeIndex { index: NonZero::new_unchecked((value as u32) + 1) } }
+        NodeIndex { index: NonZero::new((value as u32) + 1).unwrap() }
     }
 
     pub fn get(self) -> usize {

Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@ pub struct NodeIndex {`
`19`	`19`	`impl NodeIndex {`
`20`	`20`	`pub fn new(value: usize) -> NodeIndex {`
`21`	`21`	`assert!(value < (u32::MAX as usize));`
`22`		`- unsafe { NodeIndex { index: NonZero::new_unchecked((value as u32) + 1) } }`
	`22`	`+ NodeIndex { index: NonZero::new((value as u32) + 1).unwrap() }`
`23`	`23`	`}`
`24`	`24`
`25`	`25`	`pub fn get(self) -> usize {`