Re: file(line, col)=file(line:col) to follow convention of clang, gcc, dscanner etc

2015-02-03 Thread Jacob Carlborg via Digitalmars-d

On 2015-02-03 03:54, Nick Sabalausky wrote:


Frankly, I think any editor that's hardcoded to expect any one single
style is doing it wrong. Accepting either , or : should be trivial for
any editor, and hell, Programmer's Notepad 2 just uses an
optionally-customizable regex for error capture. I'd argue that should
be baseline for any editor.


Yeah, I guess I could create a pull request in the case of TextMate.

--
/Jacob Carlborg


Re: file(line, col)=file(line:col) to follow convention of clang, gcc, dscanner etc

2015-02-02 Thread Nick Sabalausky via Digitalmars-d

On 01/28/2015 04:52 PM, Timothee Cour via Digitalmars-d wrote:

See also https://github.com/Hackerpilot/Dscanner/issues/224
It breaks tooling that expects the main convention.



IIRC, file(line,col) is common in the windows world, and I've seen other 
variations, too.


Frankly, I think any editor that's hardcoded to expect any one single 
style is doing it wrong. Accepting either , or : should be trivial for 
any editor, and hell, Programmer's Notepad 2 just uses an 
optionally-customizable regex for error capture. I'd argue that should 
be baseline for any editor.


That said, getting all editors to be that flexible is probably about as 
likely as getting all CLI tools to output the same file/line format, so 
it's probably best (or at least, most realistic) to whip up a little 
streaming IO tool that takes (and optionally detects) one format and 
outputs another. (The Unix Philosophy is really growing on me...even if 
modern Linux seems surprisingly inconsistent about actually using the 
principle...)




Re: file(line, col)=file(line:col) to follow convention of clang, gcc, dscanner etc

2015-01-29 Thread Jacob Carlborg via Digitalmars-d

On 2015-01-28 22:52, Timothee Cour via Digitalmars-d wrote:

See also https://github.com/Hackerpilot/Dscanner/issues/224
It breaks tooling that expects the main convention.


Yeah, TextMate expects a colon as separator. That DMD uses a comma 
forced me to write a completely new error handler in TextMate.


--
/Jacob Carlborg


file(line,col)=file(line:col) to follow convention of clang,gcc,dscanner etc

2015-01-28 Thread Timothee Cour via Digitalmars-d
See also https://github.com/Hackerpilot/Dscanner/issues/224
It breaks tooling that expects the main convention.