core: Stamp out XXXes (comments only)

src/libcore/os.rs

@@ -145,7 +145,7 @@ pub mod win32 {
 /*
 Accessing environment variables is not generally threadsafe.
 This uses a per-runtime lock to serialize access.
-XXX: It would probably be appropriate to make this a real global
+FIXME #4726: It would probably be appropriate to make this a real global
 */
 fn with_env_lock<T>(f: &fn() -> T) -> T {
     use private::global::global_data_clone_create;

src/libcore/private/finally.rs

@@ -31,7 +31,7 @@ pub trait Finally<T> {
 }
 
 impl<T> &fn() -> T: Finally<T> {
-    // XXX: Should not require a mode here
+    // FIXME #4518: Should not require a mode here
     fn finally(&self, +dtor: &fn()) -> T {
         let _d = Finallyalizer {
             dtor: dtor
@@ -88,7 +88,7 @@ fn test_retval() {
 
 #[test]
 fn test_compact() {
-    // XXX Should be able to use a fn item instead
+    // FIXME #4727: Should be able to use a fn item instead
     // of a closure for do_some_fallible_work,
     // but it's a type error.
     let do_some_fallible_work: &fn() = || { };

src/libcore/private/global.rs

@@ -160,7 +160,7 @@ fn get_global_state() -> Exclusive<GlobalState> {
 
     const POISON: int = -1;
 
-    // XXX: Doing atomic_cxchg to initialize the global state
+    // FIXME #4728: Doing atomic_cxchg to initialize the global state
     // lazily, which wouldn't be necessary with a runtime written
     // in Rust
     let global_ptr = unsafe { rust_get_global_data_ptr() };

src/libcore/private/weak_task.rs

@@ -30,7 +30,7 @@ use ops::Drop;
 
 type ShutdownMsg = ();
 
-// XXX: This could be a PortOne but I've experienced bugginess
+// FIXME #4729: This could be a PortOne but I've experienced bugginess
 // with oneshot pipes and try_send
 pub unsafe fn weaken_task(f: &fn(Port<ShutdownMsg>)) {
     let service = global_data_clone_create(global_data_key,