Closed Bug 774603 Opened 10 years ago Closed 3 years ago

Check gclicommands.properties for copy paste errors & remove jsbBraceStyleDesc

Categories

(DevTools :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: miker, Unassigned)

References

Details

(Whiteboard: [gclicommands])

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.
We should go over all of the gcli strings and ensure that there are no copy / paste errors.
Summary: Remove jsbBraceStyleDesc from jsb → Check gclicommands.properties for copy paste errors & remove jsbBraceStyleDesc
Product: Firefox → DevTools

GCLI does not exist anymore

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.