Change all XXXs to FIXMEs

[brson]

Our development policy requires marking all FIXME's with issue numbers, and this is enforced by the tidy script. Having issues for all the FIXME's is particularly useful since they often consist of small tasks you can do at leisure or that new contributors can pick off easily.

Alas, it is awkward to open issues for bugs that one is still in the process of creating, and I have started marking problems with XXX.

With suitable advances in robot technology we could streamline the process: human write unnumbered FIXMEs; robot periodically find them, parse descriptions, open issues on github, patch issue numbers into code and submit a pull request.

To support the patching we would probably require FIXME to be written something like FIXME (####), so that the patched version is guaranteed to be under the line length limit.

[catamorphism]

Good idea. With suitable advances in robot technology, the robot could also fix the bugs ;-)

[catamorphism]

Nominating -- I think it would be great to have this before "maturity #5 - production ready", and then we would also not need both XXXs and FIXMEs.

[emberian]

Would still be handy

[graydon]

accepted for backwards-compatible milestone

[huonw]

Triage: loads of XXXs, and @bors is still only merging the bug fixes.

$ git grep '// XXX' | wc -l
141
$ git grep '// FIXME' | wc -l
421

[steveklabnik]

To be clear, this is basically 'open an issue for every XXX, put text in the issue, switch to FIXME with a number'?

I'm willing to do that, it's less than two hundred of them.

[jdm]

@steveklabnik: Correct.

[catamorphism]

@steveklabnik That would be great. In the process, you'll probably discover some that are already fixed or don't make sense; feel free to remove them completely or ask the person listed in git blame according to your judgment.

[reedlepee123]

I would like to work on it!

[salemtalha]

@huonw Since it seems this has been left alone, I'd like to tackle this, but I have a question (really new here): are these issues meant to be created manually? Some of the descriptions next to the XXX comments are not descriptive enough to stand on their own.

[emberian]

if there isn't much information in the XXX, there isn't much to put in the
issue. you should live in the comment after the fixme.

On Sat, Jan 25, 2014 at 1:50 PM, salemtalha notifications@github.comwrote:

Since it seems this has been left alone, I'd like to tackle this, but I
have a question (really new here): are these issues meant to be created
manually? Some of the descriptions next to the XXX comments are not
descriptive enough to stand on their own.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/3303#issuecomment-33296578
.

[salemtalha]

@cmr Could you clarify your last statement a bit? Not sure I follow.

[emberian]

leave, not live. replace the XXX with FIXME #XXXXX, but leave the
explanation there

On Sat, Jan 25, 2014 at 2:01 PM, salemtalha notifications@github.comwrote:

Could you clarify your last statement a bit? Not sure I follow.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/3303#issuecomment-33296815
.

[salemtalha]

@cmr So if I understand correctly: For all instances of "// XXX: comment", if the comment is explanatory enough then create an issue with the comment included and change the source code to "// FIXME #12345: comment". But if the comment doesn't make much sense, should I literally replace it with "// FIXME #XXX: comment"? I read the tidy script and I believe it rejects FIXMEs without numeric issue numbers.

[salemtalha]

@brson I came up with a quick script that maybe does what you're meaning, what do you think?

#!/usr/bin/env python

import sys, fileinput, subprocess, re, requests, json

def report_error_name_no(name, no, s):
    global err
    print("%s:%d: %s" % (name, no, s))  
    err=1

def report_err(s):
    report_error_name_no(fileinput.filename(), fileinput.filelineno(), s)

def report_warn(s):
    print("%s:%d: %s" % (fileinput.filename(), fileinput.filelineno(), s))

file_names = [s for s in sys.argv[1:] if (not s.endswith("_gen.rs")) 
    and (not ".#" in s)]

try:
    for line in fileinput.input(file_names, inplace=1):
        if "// FIXME" in line:
            comment = line.partition(':')[2]
            url = 'https://api.github.com/repos/mozilla/rust/issues'
            payload = {
                'title' : comment,
                'body' : ''
            }
            headers = { 'content-type' : 'application/json' }
            r = requests.post(url, data=json.dumps(payload), headers=headers)

            if r.status_code == 201:
                sys.stdout.write(line.replace("FIXME", "FIXME(#" + r['number'] + ")")
            else:
                sys.stdout.write(line)
        else:
             sys.stdout.write(line)

except UnicodeDecodeError, e:
    report_err("UTF-8 decoding error " + str(e))


sys.exit(err)