[Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [40536] trunk/blender: Minor: Other UI strings typos and tweaks.

Bastien Montagne montagne29 at wanadoo.fr
Mon Oct 10 08:15:18 CEST 2011


Hi Campbell,

I’m no more committing those kind of changes since about a week (as requested
by Sergey), but I had already changed quite a bunch of files this way…


Nearly all time, it was just a matter of putting the tooltip string part
on next line, I only had to cut a few of those strings (but having lines
of sometimes over 200 chars isn’t good at all, imho).

Anyway, I’m 100% ok to try to find a general rule about this (I used a 130
limit for my edits).

Regards,
Bastien

On Mon Oct 10 06:36:47 CEST, Campbell Barton
<ideasman42 at gmail.com>  wrote:

> Hi Bastien, noticed you're breaking lines up while changing descriptions.
> This may be beneficial but I'd prefer it happen after discussion with
> other devs.
>
> For now I'd prefer you don't make these changes in RNA (of course you
> can make these changes in you're own code).
>
> After 2.60 we could discuss having a set line length for blender
> maintained code - 120 for eg.
> Personally I like being able to pipe grep searches in the code - which
> doesn't work for multiple line splitting but maybe I just need to
> learn grep better
>
> I'm interested in improving our code but rather it happen after some
> discussion rather then each dev making changes that seem like a good
> idea at the time.
>
> --
> - Campbell



More information about the Bf-committers mailing list