Unity

Merge lp://qastaging/~aacid/unity/initialize_using_nofilters_background_6.0 into lp://qastaging/unity/6.0

  1. initialize_using_nofilters_background_6.0
  2. Merge into 6.0
Proposed by Albert Astals Cid
Status: Rejected
Rejected by: Albert Astals Cid
Proposed branch: lp://qastaging/~aacid/unity/initialize_using_nofilters_background_6.0
Merge into: lp://qastaging/unity/6.0
Diff against target: 11 lines (+1/-0)
1 file modified
dash/PlacesGroup.cpp (+1/-0)
To merge this branch: bzr merge lp://qastaging/~aacid/unity/initialize_using_nofilters_background_6.0
Reviewer Review Type Date Requested Status
Gord Allott Pending
Review via email: mp+132297@code.qastaging.launchpad.net

Commit message

Initialize _using_nofilters_background to true

It is the correct value according to Gord

Description of the change

Initialize _using_nofilters_background to true

It is the correct value according to Gord

To post a comment you must log in.
Revision history for this message
Albert Astals Cid (aacid) wrote :

master fix already merged https://code.launchpad.net/~aacid/unity/initialize_using_nofilters_background/+merge/132298

Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

I'd think that since Unity can be considered part of infrastructure, and the fix isn't clearly fixing a high impact bug, it doesn't fall under SRUable commits. That could change if there was a clear user problem because of this bug.

Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

Very likely this can lead to a bug... So, we only have to find it. :)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches