Non-breaking space character (c2 a0) instead of 2 spaces at the end of wifi-sleep-description

RESOLVED FIXED in Firefox OS master

Status

Firefox OS
Gaia::Settings
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: Ton, Assigned: flod)

Tracking

unspecified
2.2 S14 (12june)

Firefox Tracking Flags

(b2g-master fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Even though it may not show up at first glance inside an editor or mxr, wifi-sleep-description currently ends with 2 spaces that in fact are a 20 c2 a0 (hex) sequence, where c2 a0 is known as a non-breaking space character (see e.g. http://stackoverflow.com/questions/2946164/streamwriter-problem-2-spaces-written-as-hex-20-c2-a0-instead-of-hex-20-20 and http://www.jetmore.org/john/blog/2012/03/multiple-spaces-after-period-in-wordpress/).

This showed up by coincidence when using a non UTF enabled diff tool and may be harmless, but rather reporting it to prevent possible trouble. It could probably be resolved by just removing the spaces and saving the file again.

Comment 1

3 years ago
Created attachment 8613492 [details] [review]
[gaia] flodolo:patch-1 > mozilla-b2g:master
(Assignee)

Updated

3 years ago
Assignee: nobody → francesco.lodolo
(Assignee)

Comment 2

3 years ago
Comment on attachment 8613492 [details] [review]
[gaia] flodolo:patch-1 > mozilla-b2g:master

Silly patch to fix weird characters probably introduced by a text editor.
Attachment #8613492 - Flags: review?(jaoo)
Comment on attachment 8613492 [details] [review]
[gaia] flodolo:patch-1 > mozilla-b2g:master

LGTM, thanks!
Attachment #8613492 - Flags: review?(jaoo) → review+
(Assignee)

Updated

3 years ago
Keywords: checkin-needed

Updated

3 years ago
Keywords: checkin-needed

Comment 4

3 years ago
http://docs.taskcluster.net/tools/task-graph-inspector/#fN8aJh8QQSm8tr0dBtzehw

The pull request failed to pass integration tests. It could not be landed, please try again.
(Assignee)

Updated

3 years ago
Keywords: checkin-needed

Updated

3 years ago
Keywords: checkin-needed

Comment 5

3 years ago
https://github.com/mozilla-b2g/gaia/pull/30348

Autolander could not land the pull request due to not having collaborator rights. This is possibly due to a tree closure. Please check the tree status and request checkin again once the tree is open.
(Assignee)

Updated

3 years ago
Keywords: checkin-needed

Updated

3 years ago
Keywords: checkin-needed
Comment hidden (obsolete)
Let's try this again. Sorry about that.
Keywords: checkin-needed

Updated

3 years ago
Keywords: checkin-needed
Comment hidden (obsolete)
Keywords: checkin-needed
Master: https://github.com/mozilla-b2g/gaia/commit/126e8f8b8c6dcbdfb50daa93c987ae1ddbe8fee0
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-b2g-master: --- → fixed
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 2.2 S14 (12june)
You need to log in before you can comment on or make changes to this bug.