News:

The new Release 25.03 is out! You can download binaries for Windows and many major Linux distros here .

Main Menu

The 06 August 2013 build (9246) is out.

Started by killerbot, August 08, 2013, 11:43:49 AM

Previous topic - Next topic

Jenna

Quote from: ollydbg on October 12, 2013, 02:46:52 PM
Quote from: ollydbg on August 12, 2013, 03:51:58 AM
Quote from: jens on August 12, 2013, 01:36:10 AM
@ollydbg:
can you test the attached patch ?

Just a quick and rough one. It also zooms the folding margin, but not the changebar and marker margin.
The changebar-margin is just 4 px small, so it does not really matter, and the maerker margin would need tweaking of all plugins that use it, and I do not have the time at the moment.

Hi, jens, thanks, I just test this patch, and it fix the problem I reported.

Jen, can you commit this patch to trunk? Thanks.

Committed to trunk (svn r9398).

Jens

ollydbg

Quote from: jens on October 12, 2013, 06:27:39 PM
Quote from: ollydbg on October 12, 2013, 02:46:52 PM
Quote from: ollydbg on August 12, 2013, 03:51:58 AM
Quote from: jens on August 12, 2013, 01:36:10 AM
@ollydbg:
can you test the attached patch ?

Just a quick and rough one. It also zooms the folding margin, but not the changebar and marker margin.
The changebar-margin is just 4 px small, so it does not really matter, and the maerker margin would need tweaking of all plugins that use it, and I do not have the time at the moment.

Hi, jens, thanks, I just test this patch, and it fix the problem I reported.

Jen, can you commit this patch to trunk? Thanks.

Committed to trunk (svn r9398).

Jens
Thank you.
If some piece of memory should be reused, turn them to variables (or const variables).
If some piece of operations should be reused, turn them to functions.
If they happened together, then turn them to classes.