774603 - Check gclicommands.properties for copy paste errors & remove jsbBraceStyleDesc

Status: RESOLVED INVALID

(DevTools :: General, defect)

Description

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

See Francesco's second point below ... we should either change the string or hard code it.

(In reply to Francesco Lodolo [:flod] from comment #16)
> > The skipped statements are executed, but not stepped through
> Could you explain the meaning of this string?
> 

No idea, it is not part of this patch

> > jsbBraceStyleDesc=Collapse, expand, end-expand, expand-strict
> Is this supposed to stay non localized? They look like parameter values to me
> 

They should not be localised, the string should be removed.

> Side note: copy&paste of l10n comments (clearly wrong, e.g. "A very short
> string used to describe thefunction of the dbg step over command.") is kind
> of worse than having no comments at all.

The word dbg is not contained in this patch.

Comment 1

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

We should go over all of the gcli strings and ensure that there are no copy / paste errors.

Comment 2

[Nicolas Chevobbe [:nchevobbe]]

GCLI does not exist anymore