Scripted Plugin: FindBrokenFiles

[Jenna]

OK, I've committed it to trunk...

BTW: Reminds me: I had an issue with this script the past time I used it: When removing files, some were left and you had to call the script several times to really remove all broken files. Is that still the case?

Yes it is.

The cause is that you loop forward through the files and remove them.

In general it's better to do this backwards if removing array members, because a removed index does not change the indexes following in the loop.

I fixed it in trunk and now it seems to work as expected.

[MortenMacFly]

I've tried it on the codeblocks-unix.cbp and it removed one file successfully

Note that this might not really be a missing file. some of the files are Windows only and are simple "disabled" in Unix.

[MortenMacFly]

In general it's better to do this backwards if removing array members, because a removed index does not change the indexes following in the loop.

I know, but it was really just a demo and I was to lazy to take care.

I fixed it in trunk and now it seems to work as expected.

Thanks!

[LETARTARE]

Good year for C::B.
@jens
r7650 in 'src/scripts/plugin_find_broken_files.script',
line of code:

entries.Add(_T("Plugins/0:-Find broken files in project"), 1);

creates an entry 'Plugins' in the international versions !
I propose instead

entries.Add(_("&Plugins") + _T("/0:-") + _("Find broken files in project"), 1) ;

which also allows the translation.

Other hand, all the scripts in this directory can be prepared for the translation, I can send
a zip of changed files and 'scripts.pot', that works at home.

[oBFusCATed]

As far as I can see you've not tested the change, have you?
I think so, because the correct line is:

Code Select Expand

entries.Add(_("P&lugins") + _T("/0:-") + _("Find broken files in project"), 1);

Please verify my version of the line is correct and I'll commit it.

[LETARTARE]

I tested and it works with my setup !
In r7650 (English) with '&Plugins', 'P&lugins', 'Plugins' it works !!
and French works as.

[oBFusCATed]

So what is the correct line? Your last post is quite confusing.

[LETARTARE]

The '&' is not taken into account.
Take 'Plugins' as 'Jens'.

[LETARTARE]

r7650 : In the source 'src\main.cpp:926'  found  _('P&lugins').

[oBFusCATed]

Yes, I know, that's why I've said that you've not tested your change, but if it doesn't matter I'll commit a version without the '&' character.

[LETARTARE]

@ofuscated
I am sorry but I tested all the possibilities on r7650, before posting:

'Plugins'

'&Plugins'

'P&lugins'

all work on my computer!

In addition, the menus, letters preceded by '&' are no different from others letters.