User forums > Using Code::Blocks
EditorTweaks: new option 'Convert Matching Braces' : little bug
MortenMacFly:
--- Quote from: dmoore on October 27, 2012, 10:19:10 pm ---Anyone else tested it?
--- End quote ---
I've applied it in my local copy but didn't test.
dmoore:
--- Quote from: Alpha on October 26, 2012, 11:02:21 pm ---Here is a candidate patch.
--- End quote ---
Having troubles applying this on linux. There are different line endings in at least one of the target files (I thought we had done something to fix this in our SVN). Do you mind attaching a patch file Alpha?
dmoore:
Also, is there a good reason to put the brace completion stuff in cbEditor instead of the plugin base? I thought the point of making smart indent a set of plugins was to reduce the amount of cruft in the editor code.
Alpha:
Attached.
--- Quote from: dmoore on October 29, 2012, 12:50:22 am ---I thought we had done something to fix this in our SVN.
--- End quote ---
Most files are set to match system EOL style, however, it appears as if the files added by Smart Indent are CR LF only.
--- Quote from: dmoore on October 29, 2012, 01:13:38 am ---Also, is there a good reason to put the brace completion stuff in cbEditor instead of the plugin base?
--- End quote ---
I put it there so the editor has access to use it as fall-back if no plugin supports the current lexer. The Smart Indent plugins for languages that I am less familiar with just call this more-generic method, whereas the C++ plugin has its own specialized implementation (that I previously wrote - although, it occasionally has some issues, so I may have to look back at it again).
I did this under the same idea that cbEditor contains a minimal auto-indent (just copy the previous line's indentation). Not sure if this was correct thinking...
dmoore:
--- Quote from: Alpha on October 29, 2012, 01:28:06 am ---I put it there so the editor has access to use it as fall-back if no plugin supports the current lexer.
--- End quote ---
That makes some sense.
Thx for the patch.
Navigation
[0] Message Index
[#] Next page
[*] Previous page
Go to full version