---

yaml
---
r: 2239
b: refs/heads/master
c: 4c7886d
h: refs/heads/master
i:
  2237: 50b4ff8
  2235: 3009fbd
  2231: 100b855
  2223: 57067a4
  2207: 941d27a
  2175: 7dcbab2
v: v3

Author: graydon

[refs]

@@ -1,2 +1,2 @@
 ---
-refs/heads/master: fef8314c2e018d2e35c8bd91d3181de038e44de2
+refs/heads/master: 4c7886de8008b07e892ea741be08291dae38f7bd

trunk/src/lib/_str.rs

@@ -5,6 +5,7 @@ import _vec.rustrt.vbuf;
 native "rust" mod rustrt {
     type sbuf;
     fn str_buf(str s) -> sbuf;
+    fn str_vec(str s) -> vec[u8];
     fn str_byte_len(str s) -> uint;
     fn str_alloc(uint n_bytes) -> str;
     fn str_from_vec(vec[mutable? u8] b) -> str;
@@ -126,10 +127,7 @@ fn buf(str s) -> sbuf {
 }
 
 fn bytes(str s) -> vec[u8] {
-    fn ith(str s, uint i) -> u8 {
-        ret s.(i);
-    }
-    ret _vec.init_fn[u8](bind ith(s, _), byte_len(s));
+    ret rustrt.str_vec(s);
 }
 
 fn from_bytes(vec[u8] v) : is_utf8(v) -> str {

trunk/src/rt/rust_builtin.cpp

@@ -219,6 +219,32 @@ str_buf(rust_task *task, rust_str *s)
     return (char const *)&s->data[0];
 }
 
+extern "C" CDECL rust_vec*
+str_vec(rust_task *task, rust_str *s)
+{
+    // FIXME: this should just upref s and return it, but we
+    // accidentally made too much of the language and runtime know
+    // and care about the difference between str and vec (trailing null);
+    // eliminate these differences and then rewrite this back to just
+    // the following:
+    //
+    // if (s->ref_count != CONST_REFCOUNT)
+    //    s->ref();
+    // return s;
+
+    rust_str *v =
+        vec_alloc_with_data(task,
+                            s->fill - 1,
+                            s->fill - 1,
+                            1,
+                            (s->fill - 1) ? (void*)s->data : NULL);
+    if (!v) {
+        task->fail(2);
+        return NULL;
+    }
+    return v;
+}
+
 extern "C" CDECL size_t
 str_byte_len(rust_task *task, rust_str *s)
 {