the last comma in param list makes a syntax error

Drupal
.
formatPlural
(
1,
"Whitespace Call plural",
"Whitespace Call @count plural",
)
;

should be

"Whitespace Call @count plural"
Files: 
CommentFileSizeAuthor
#8 1334768-8.patch361 bytesfietserwin
PASSED: [[SimpleTest]]: [MySQL] 34,271 pass(es).
[ View ]
#2 trailing_comma-1334768-2.patch341 bytesls206
PASSED: [[SimpleTest]]: [MySQL] 36,940 pass(es).
[ View ]

Comments

Issue tags:+Novice

Yes it's true and it will fail on some browsers.
Can you make a patch?

StatusFileSize
new341 bytes
PASSED: [[SimpleTest]]: [MySQL] 36,940 pass(es).
[ View ]

My first patch, hope it does the job!

Status:Active» Needs review

You have to set it to needs review to trigger the testbot.
Let's see if your patch gets accepted.

Thnx for making it!

Status:Needs review» Reviewed & tested by the community

thanks

Ah yes, I missed that step. D'oh!

NP, hopefully the first of many ;)

Status:Reviewed & tested by the community» Fixed

D'oh! Thanks a lot for the fix!

Committed and pushed to 7.x. :)

Status:Fixed» Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Version:7.x-dev» 8.x-dev
Status:Closed (fixed)» Needs review
StatusFileSize
new361 bytes
PASSED: [[SimpleTest]]: [MySQL] 34,271 pass(es).
[ View ]

Error also appears in 8.x

Status:Needs review» Reviewed & tested by the community

Thi shas already been reviewed for D7, it is an extrmeley small and simnple change, no other changes in D8 are touching this (except the move to /core), so I guess this can be set to RTBC without additional reviews?!

true :)

Status:Reviewed & tested by the community» Fixed

Hmm not sure why this never got committed to 8.x in the first place, but committed/pushed to 8.x now.

Automatically closed -- issue fixed for 2 weeks with no activity.