Merge lp:~ohad-kammar/qbzr/configurable-colours into lp:~qbzr-dev/qbzr/trunk
Proposed by
Ohad Kammar
Status: | Merged |
---|---|
Merged at revision: | not available |
Proposed branch: | lp:~ohad-kammar/qbzr/configurable-colours |
Merge into: | lp:~qbzr-dev/qbzr/trunk |
To merge this branch: | bzr merge lp:~ohad-kammar/qbzr/configurable-colours |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Gary van der Merwe | Approve | ||
Review via email: mp+1278@code.launchpad.net |
To post a comment you must log in.
This merge depends on the following merge (and indeed branched from it): /code.edge. launchpad. net/~ohad- kammar/ qbzr/config- bugfix/ +merge/ 1277
https:/
This is a fix to bug #280041: /bugs.edge. launchpad. net/qbzr/ +bug/280041
https:/
Problems with this fix:
1. I am not familiar with this project's code, so you may find the fix's implementation inconsistent with the usual programming practices of the team.
2. I didn't know how to handle errors correctly, so I implemented the mechanism for detecting them, but it doesn't do anything apart from ignoring an erroneous input and using the default instead.