Closed
Bug 1374427
Opened 9 years ago
Closed 9 years ago
Remove redundant function names in satchel js-doc
Categories
(Toolkit :: Form Manager, enhancement)
Toolkit
Form Manager
Tracking
()
RESOLVED
FIXED
mozilla56
| Tracking | Status | |
|---|---|---|
| firefox56 | --- | fixed |
People
(Reporter: jonathanGB, Assigned: jonathanGB, Mentored)
Details
Attachments
(1 file)
Some `js-doc` of functions declare the function names, which doesn't offer useful information. This patch will remove such cases in the satchel project.
| Comment hidden (mozreview-request) |
|
||
Comment 2•9 years ago
|
||
| mozreview-review | ||
Comment on attachment 8879309 [details]
Bug 1374427 - remove redundant function names in js-doc.
https://reviewboard.mozilla.org/r/150582/#review155834
::: toolkit/components/satchel/FormHistory.jsm:456
(Diff revision 1)
> /**
> - * dbInit
> *
Please remove the empty lines too
Attachment #8879309 -
Flags: review?(MattN+bmo)
| Comment hidden (mozreview-request) |
|
|
||
Comment 4•9 years ago
|
||
| mozreview-review | ||
Comment on attachment 8879309 [details]
Bug 1374427 - remove redundant function names in js-doc.
https://reviewboard.mozilla.org/r/150582/#review155966
Thanks
Attachment #8879309 -
Flags: review?(MattN+bmo) → review+
|
Assignee | |
Updated•9 years ago
|
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/0000dd6746c4
remove redundant function names in js-doc. r=MattN
Keywords: checkin-needed
|
||
Comment 6•9 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in
before you can comment on or make changes to this bug.
Description
•