#22 Fix for COBOL syntax highlighting and tagmanager

[stephen connolly]

Support for the COBOL language seems to be broken. I modified filetypes.cobol to function properly and added comments to document the supported variables in the [keywords] section. As well, for the tag parser cobol.c needed some tuning to better isolate only valid cases.

[Colomban Wendling]

The changes to the COBOL parser break the simple.cbl test case, could you please either fix the test case or the parser? Also additional test cases would be great.

Also, did the parser support free-format, and does the update? I pretty much don't know anything about COBOL, just trying to avoid some regressions :)

[stephen connolly]

I later saw your instructions for submissions. Updated work is in pull request #320. Please let me know of anything else that may be required. Look forward to hearing from you again. Great work. We use geany ( my patched version ) at work as out COBOL IDE and maintain more than 80 COBOL applications comprising more than 320000 lines of code.

Stephen

On 11/4/14, Colomban Wendling colombanw@users.sf.net wrote:

The changes to the COBOL parser break the simple.cbl test case, could you please either fix the test case or the parser? Also additional test cases would be great.

Also, did the parser support free-format, and does the update? I pretty much don't know anything about COBOL, just trying to avoid some regressions :)

patches:#22 Fix for COBOL syntax highlighting and tagmanager

Status: open

Group: Unstable (example)

Created: Sat Aug 02, 2014 11:50 AM UTC by stephen connolly

Last Updated: Sat Aug 02, 2014 11:50 AM UTC

Owner: nobody

Support for the COBOL language seems to be broken. I modified filetypes.cobol to function properly and added comments to document the supported variables in the [keywords] section. As well, for the tag parser cobol.c needed some tuning to better isolate only valid cases.

Sent from sourceforge.net because you indicated interest in https://sourceforge.net/p/geany/patches/22/(https://sourceforge.net/p/geany/patches/22)

To unsubscribe from further messages, please visit https://sourceforge.net/auth/subscriptions/(https://sourceforge.net/auth/subscriptions)

[Colomban Wendling]

Oops, not I look like a fool don't I ^^
So, this moved to https://github.com/geany/geany/pull/320