The default bug view has changed. See this FAQ.

Typos/Errors in gclicommands.properties: Comments always talk about 'step over'

RESOLVED FIXED in Firefox 18

Status

()

Firefox
Developer Tools: Console
--
trivial
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: aryx, Assigned: SADINENI RAVI CHANDRA)

Tracking

Trunk
Firefox 18
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug][mentor=aryx][fixed-in-fx-team])

Attachments

(1 attachment, 2 obsolete attachments)

Firefox Nightly 20120825

The comments here always talk about 'step over', but should be altered to match the string.

# LOCALIZATION NOTE (dbgStepOverDesc) A very short string used to describe the
# function of the dbg step over command.
dbgStepOverDesc=Executes the current statement and then stops at the next statement. If the current statement is a function call then the debugger executes the whole function, and it stops at the next statement after the function call

# LOCALIZATION NOTE (dbgStepInDesc) A very short string used to describe the
# function of the dbg step over command.
dbgStepInDesc=Executes the current statement and then stops at the next statement. If the current statement is a function call, then the debugger steps into that function, otherwise it stops at the next statement

# LOCALIZATION NOTE (dbgStepOutDesc) A very short string used to describe the
# function of the dbg step over command.
dbgStepOutDesc=Steps out of the current function and up one level if the function is nested. If in the main body, the script is executed to the end, or to the next breakpoint. The skipped statements are executed, but not stepped through

Updated

5 years ago
Whiteboard: [good first bug] → [good first bug][mentor=aryx]
Assignee: nobody → ravicat2013
Status: NEW → ASSIGNED
(Assignee)

Comment 1

5 years ago
Created attachment 655958 [details] [diff] [review]
patch: changed the comments accordingly
(Assignee)

Comment 2

5 years ago
Created attachment 655964 [details] [diff] [review]
patch: changed the comments accordingly
Attachment #655958 - Attachment is obsolete: true
(Assignee)

Updated

5 years ago
Attachment #655964 - Flags: review?(archaeopteryx)
Attachment #655964 - Flags: approval1.9.2.28?
Attachment #655964 - Flags: approval-mozilla-aurora?
Attachment #655964 - Flags: approval1.9.2.28?
Attachment #655964 - Flags: approval-mozilla-aurora?
Comment on attachment 655964 [details] [diff] [review]
patch: changed the comments accordingly

> # LOCALIZATION NOTE (dbgStepInDesc) A very short string used to describe the
>-# function of the dbg step over command.
>+# function of the dbg step In  command.
> dbgStepInDesc=Executes the current statement and then stops at the next statement. If the current statement is a function call, then the debugger steps into that function, otherwise it stops at the next statement

Please change the "In" to lower case and use only one whitespace behind it.


> # LOCALIZATION NOTE (dbgStepOutDesc) A very short string used to describe the
>-# function of the dbg step over command.
>+# function of the dbg step out  command.
> dbgStepOutDesc=Steps out of the current function and up one level if the function is nested. If in the main body, the script is executed to the end, or to the next breakpoint. The skipped statements are executed, but not stepped through

Please use only one whitespace behind the "out".

Furthermore, this doesn't fix something critical, so only request review without requesting uplifting to aurora or more mature branches.

Thank you.
Attachment #655964 - Flags: review?(archaeopteryx) → review-
(Assignee)

Comment 4

5 years ago
Created attachment 655979 [details] [diff] [review]
patch: changed the comments accordingly
Attachment #655964 - Attachment is obsolete: true
Attachment #655979 - Flags: review?(archaeopteryx)
(Assignee)

Comment 5

5 years ago
(In reply to Archaeopteryx [:aryx] from comment #3)
> Comment on attachment 655964 [details] [diff] [review]
> patch: changed the comments accordingly
> 
> > # LOCALIZATION NOTE (dbgStepInDesc) A very short string used to describe the
> >-# function of the dbg step over command.
> >+# function of the dbg step In  command.
> > dbgStepInDesc=Executes the current statement and then stops at the next statement. If the current statement is a function call, then the debugger steps into that function, otherwise it stops at the next statement
> 
> Please change the "In" to lower case and use only one whitespace behind it.
> 
> 
> > # LOCALIZATION NOTE (dbgStepOutDesc) A very short string used to describe the
> >-# function of the dbg step over command.
> >+# function of the dbg step out  command.
> > dbgStepOutDesc=Steps out of the current function and up one level if the function is nested. If in the main body, the script is executed to the end, or to the next breakpoint. The skipped statements are executed, but not stepped through
> 
> Please use only one whitespace behind the "out".
> 
> Furthermore, this doesn't fix something critical, so only request review
> without requesting uplifting to aurora or more mature branches.
> 
> Thank you.

i am sorry for that . i am new. so dont know which options to select.
i have made the necessary changes as said by you

thank you
Comment on attachment 655979 [details] [diff] [review]
patch: changed the comments accordingly

Thank you, no need to worry. The patch looks good, but has to be formally reviewed by the owner or a peer of the devtools/console team, so asking :jwalker for review (and doing/delegating the checkin).
Attachment #655979 - Flags: review?(archaeopteryx) → review?(jwalker)
Comment on attachment 655979 [details] [diff] [review]
patch: changed the comments accordingly

Review of attachment 655979 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks.
Will add to my queue of things to land
Attachment #655979 - Flags: review?(jwalker) → review+
https://tbpl.mozilla.org/?tree=Try&rev=95edd1e1af78
https://tbpl.mozilla.org/?tree=Fx-Team&rev=3636c402ea3d
Whiteboard: [good first bug][mentor=aryx] → [good first bug][mentor=aryx][fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/a14a4bf5e790
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 18
You need to log in before you can comment on or make changes to this bug.