Apparently, Filter.prefix is pretty strict on the appearance order of %s... #78

Merged
merged 1 commit into from May 20, 2012

Projects

None yet

2 participants

@rystraum
Contributor
rystraum commented May 4, 2012

..., %d and %d for the file name, line number and column number in that the %s has to be appear first in the string before any of the %d. Otherwise, you get

Footnotes #Footnotes::Notes::ControllerNote:0x9a5a650 Exception: invalid value for Integer(): "/opt/ror/tribes/app/controllers/dailies_controller.rb"

errors all over the place. I ran into this when I tried writing my own gedit protocol wrapper where I assumed the format to be:

foo://l=%d&%s

and ran into problems.

Granted that I'm probably an edge case and not a lot of people write their own filter prefixes so this is probably not worth writing code for. Just wanted it to appear in the docs somewhere to spare someone else the trouble I've gone through.

@rystraum rystraum Apparently, Filter.prefix is pretty strict on the appearance order of…
… %s, %d and %d for the file name, line number and column number in that the %s has to be appear first in the string before any of the %d. Otherwise, you get


    Footnotes #<Footnotes::Notes::ControllerNote:0x9a5a650> Exception: invalid value for Integer(): "/opt/ror/tribes/app/controllers/dailies_controller.rb"

errors all over the place. I ran into this when I tried writing my own gedit protocol wrapper where I assumed the format to be:

    foo://l=%d&%s

and ran into problems.

Granted that I'm probably an edge case and not a lot of people write their own filter prefixes so this is probably not worth writing code for. Just wanted it to appear in the docs somewhere to spare someone else the trouble I've gone through.
a58f41a
@romanvbabenko romanvbabenko merged commit 54aa382 into josevalim:master May 20, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment