Code::Blocks Forums

Developer forums (C::B DEVELOPMENT STRICTLY!) => Development => Topic started by: FlipFlop on May 07, 2013, 12:23:51 pm

Title: Added new features. How to share it?
Post by: FlipFlop on May 07, 2013, 12:23:51 pm
Hello,
I have added new features to codeblocks:

1) Copy/Cut and Paste now are the same of Visual Studio:
    If no selection, copy/cut copy and cut all the line under caret. Then, paste insert a new line.

2) Wrapped a possibly bug of scintilla: FindColumn don't work.

How I share the modifications?

thanks.
Title: Re: Added new features. How to share it?
Post by: oBFusCATed on May 07, 2013, 01:23:17 pm
Here is the guide how to make a patch: http://wiki.codeblocks.org/index.php?title=Creating_a_patch_to_submit_to_BerliOS_%28Patch_Tracker%29

And here is my personal opinion:
1. This is the most annoying feature of visual studio, so if you'd like to have it in C::B you'd need to make it configurable.
    But I don't see a use case for it, we have the vastly superior Ctrl+D (Duplicate line or duplicate selection).
2. This should be reported to the Scintilla devs, we don't support it and we prefer to minimize our changes to it.
Title: Re: Added new features. How to share it?
Post by: FlipFlop on May 07, 2013, 03:06:39 pm
1) The functionality (cut/copy - paste) is very convenient if you have to copy entire lines from one point to another. Ctrl - D copy a line below and not in another point of the code. For my development team this feature is very much appreciated.

If it seems appropriate, I share these changes.

thanks,
  FF.

Title: Re: Added new features. How to share it?
Post by: dmoore on May 07, 2013, 04:18:00 pm
Maybe something to put into the EditorTweaks plugin. Much better than putting into the core application.

Never mind -- this is a patch to existing copy/paste behavior so that wouldn't work. This should probably be a scintilla patch anyway.
Title: Re: Added new features. How to share it?
Post by: Sertalk on May 08, 2013, 01:22:51 pm
Quote
2) Wrapped a possibly bug of scintilla: FindColumn don't work.

Thanks for this future , it's that what I need .