Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29775597: Noissue - Use default parameters (Closed)

Created:
May 9, 2018, 3:54 p.m. by Manish Jethani
Modified:
May 11, 2018, 4 a.m.
Reviewers:
kzar
CC:
sergei
Base URL:
https://hg.adblockplus.org/adblockpluscore/
Visibility:
Public.

Description

Noissue - Use default parameters

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -3 lines) Patch
M lib/elemHide.js View 1 chunk +1 line, -3 lines 0 comments Download

Messages

Total messages: 3
Manish Jethani
May 9, 2018, 3:54 p.m. (2018-05-09 15:54:52 UTC) #1
Manish Jethani
Patch Set 1 https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Functions/Default_parameters This change is about using default parameters wherever it makes sense. ...
May 9, 2018, 4:03 p.m. (2018-05-09 16:03:52 UTC) #2
kzar
May 10, 2018, 11:02 a.m. (2018-05-10 11:02:44 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld