Adding inline-translation of UI in Blender

This patch aims to add translation of Blender directly from the User Interface.

WARNING: Even though working, it’s currently still work in progress.

To do that, it splits the work in two:
* A small C operator finds the active button, get from it a bunch of strings (button/rna/enumitem labels and tips, RNA path, RNA context), and gives them to a py op defined in an addon.
* A py addon (current diff: http://www.pasteall.org/33552/diff ) reads the right po file, find in it the matching message ids, and popups an UI to edit the translations.

Current things I’m not sure about:
* It adds a new userpref path to point to /branches i18n dir (and may need another one to mo building tool in future :/ ).

[ideasman42, 2012-06-10 13:56:02]

not full review, looks ok generally - suggestion to change two areas.

http://codereview.appspot.com/6307069/diff/1/source/blender/editors/interface...
File source/blender/editors/interface/interface.c (right):

http://codereview.appspot.com/6307069/diff/1/source/blender/editors/interface...
source/blender/editors/interface/interface.c:3694: char
**uiButGetStrInfo(bContext *C, uiBut *but, int nbr, ...)
This finction is OK - but dont really like how all its callers have to then
access buf[n] - with hard coded constants.

This could be done differently... rather then use varargs

typedef struct UIStringInfo {
    int type;
    char *strinfo;
} UIStringInfo;


then pass varargs of these which each get filled in.

UIStringInfo label = {BUT_GET_LABEL, ""};

uiButGetStrInfo(....., label);

now use label.strinfo;

http://codereview.appspot.com/6307069/diff/1/source/blender/editors/interface...
File source/blender/editors/interface/interface_ops.c (right):

http://codereview.appspot.com/6307069/diff/1/source/blender/editors/interface...
source/blender/editors/interface/interface_ops.c:786: sprintf(path,
"%s%s/%s.po", root, uilng, uilng);
should use BLI_join_dirfile

http://codereview.appspot.com/6307069/diff/1/source/blender/editors/interface...
source/blender/editors/interface/interface_ops.c:792: sprintf(path,
"%s%s/%s.po", root, t, t);
should use BLI_join_dirfile

[ideasman42, 2012-06-10 14:11:51]

http://codereview.appspot.com/6307069/diff/1/source/blender/editors/interface...
File source/blender/editors/interface/interface.c (right):

http://codereview.appspot.com/6307069/diff/1/source/blender/editors/interface...
source/blender/editors/interface/interface.c:3694: char
**uiButGetStrInfo(bContext *C, uiBut *but, int nbr, ...)
On 2012/06/10 13:56:02, ideasman42 wrote:
> This finction is OK - but dont really like how all its callers have to then
> access buf[n] - with hard coded constants.
> 
> This could be done differently... rather then use varargs
> 
> typedef struct UIStringInfo {
>     int type;
>     char *strinfo;
> } UIStringInfo;
> 
> 
> then pass varargs of these which each get filled in.
> 
> UIStringInfo label = {BUT_GET_LABEL, ""};
> 
> uiButGetStrInfo(....., label);
> 
> now use label.strinfo;

correction - varargs would still be used.

http://codereview.appspot.com/6307069/diff/1/source/blender/editors/interface...
File source/blender/editors/interface/interface_ops.c (right):

http://codereview.appspot.com/6307069/diff/1/source/blender/editors/interface...
source/blender/editors/interface/interface_ops.c:863: return
WM_operatortype_find(EDTSRC_I18N_OP_NAME, 0) ? TRUE : FALSE;
suggest not to do this for the poll function, instead report an error when
running the op. search an op on poll is a bit odd.

this way users also get told they need an addon.

[mont29, 2012-06-10 20:18:22]

Adressed campbel’s comments.

[mont29, 2012-06-10 20:20:17]

http://codereview.appspot.com/6307069/diff/1/source/blender/editors/interface...
File source/blender/editors/interface/interface.c (right):

http://codereview.appspot.com/6307069/diff/1/source/blender/editors/interface...
source/blender/editors/interface/interface.c:3694: char
**uiButGetStrInfo(bContext *C, uiBut *but, int nbr, ...)
On 2012/06/10 13:56:02, ideasman42 wrote:
> This finction is OK - but dont really like how all its callers have to then
> access buf[n] - with hard coded constants.
> 
> This could be done differently... rather then use varargs
> 
> typedef struct UIStringInfo {
>     int type;
>     char *strinfo;
> } UIStringInfo;
> 
> 
> then pass varargs of these which each get filled in.
> 
> UIStringInfo label = {BUT_GET_LABEL, ""};
> 
> uiButGetStrInfo(....., label);
> 
> now use label.strinfo;

Done.

http://codereview.appspot.com/6307069/diff/1/source/blender/editors/interface...
File source/blender/editors/interface/interface_ops.c (right):

http://codereview.appspot.com/6307069/diff/1/source/blender/editors/interface...
source/blender/editors/interface/interface_ops.c:786: sprintf(path,
"%s%s/%s.po", root, uilng, uilng);
On 2012/06/10 13:56:02, ideasman42 wrote:
> should use BLI_join_dirfile

Done.

http://codereview.appspot.com/6307069/diff/1/source/blender/editors/interface...
source/blender/editors/interface/interface_ops.c:792: sprintf(path,
"%s%s/%s.po", root, t, t);
On 2012/06/10 13:56:02, ideasman42 wrote:
> should use BLI_join_dirfile

Done.

http://codereview.appspot.com/6307069/diff/1/source/blender/editors/interface...
source/blender/editors/interface/interface_ops.c:863: return
WM_operatortype_find(EDTSRC_I18N_OP_NAME, 0) ? TRUE : FALSE;
On 2012/06/10 14:11:51, ideasman42 wrote:
> suggest not to do this for the poll function, instead report an error when
> running the op. search an op on poll is a bit odd.
> 
> this way users also get told they need an addon.

Done.

[mont29, 2012-07-08 20:09:49]

Update patch.Together with addon linked on pasteall.org, it’s now rather
complete...