From ed69d6b0370bd033e78accf6d7a397de25b4ca05 Mon Sep 17 00:00:00 2001
From: Steven Fackler <sfackler@gmail.com>
Date: Sat, 5 Nov 2016 13:40:53 -0700
Subject: [PATCH] Add Stack::pop

---
 openssl/src/stack.rs | 25 +++++++++++++------------
 1 file changed, 13 insertions(+), 12 deletions(-)

diff --git a/openssl/src/stack.rs b/openssl/src/stack.rs
index 50694919..dae42ca8 100644
--- a/openssl/src/stack.rs
+++ b/openssl/src/stack.rs
@@ -40,18 +40,7 @@ impl<T: Stackable> Stack<T> {
 impl<T: Stackable> Drop for Stack<T> {
     fn drop(&mut self) {
         unsafe {
-            loop {
-                let ptr = OPENSSL_sk_pop(self.as_stack());
-
-                if ptr.is_null() {
-                    break;
-                }
-
-                // Build the owned version of the object just to run
-                // its `drop` implementation and delete the item.
-                T::from_ptr(ptr as *mut _);
-            }
-
+            while let Some(_) = self.pop() {}
             OPENSSL_sk_free(self.0 as *mut _);
         }
     }
@@ -209,6 +198,18 @@ impl<T: Stackable> StackRef<T> {
         }
     }
 
+    /// Removes the last element from the stack and returns it.
+    pub fn pop(&mut self) -> Option<T> {
+        unsafe {
+            let ptr = OPENSSL_sk_pop(self.as_stack());
+            if ptr.is_null() {
+                None
+            } else {
+                Some(T::from_ptr(ptr as *mut _))
+            }
+        }
+    }
+
     unsafe fn _get(&self, idx: usize) -> *mut T::CType {
         OPENSSL_sk_value(self.as_stack(), idx as c_int) as *mut _
     }