1160721 - Remove "double spacing" in SSLerrs.h and SECerrs.h strings

Status: RESOLVED FIXED

(NSS :: Libraries, defect, P5)

Description

[:Cykesiopka]

Bug 1153446 was filed on unnecessary "double spacing" in nsserrors.properties entries. It looks these double spaces were just copied over from the hard coded en-US strings in SSLerrs.h and SECerrs.h.

It's not clear to me if the double spacing in these two files is intentional, or if they are part of API and hence can't be changed. Feel free to resolve this bug as WONTFIX/INVALID etc as necessary.

Comment 1

[Manish [:manishkk]]

Attachment: Patch_BugLatest

Please review.

Thanks

Comment 2

[Eric Rescorla (:ekr)]

Comment on attachment 9027358 [details]
Patch_BugLatest

1. You need to do this CL as a change against NSS, not a change against Firefox. https://developer.mozilla.org/en-US/docs/Mozilla/Projects/NSS
2. CLs should be submitted using Phabricator, not Splinter.
3. You've also broken the indentation, which needs to line up with the parentheses, not be 2 spaces in. Run |./mach clang-format| before submitting.

Comment 3

[Manish [:manishkk]]

Attachment: Bug 1160721 Remove double spacing in SSLerrs.h and SECerrs.h strings r?ekr

Comment 4

[Manish [:manishkk]]

Please Ignore the patch! 
It was phabricator test.

Comment 5

[Manish [:manishkk]]

Attachment: Bug 1160721 Remove double spacing in SSLerrs.h and SECerrs.h strings

Comment 6

[J.C. Jones [:jcj] (he/him)]

Thanks, Manish.

https://hg.mozilla.org/projects/nss/rev/80d449d8ad1dbbc91ddfbca4547d564a6d380315