| 1 |
On 31-08-2011 11:45:27 +0200, Burcin Erocal wrote: |
| 2 |
> I didn't have time to test the patch myself, since we already have a |
| 3 |
> report of it not working, here is a justification for my patch... |
| 4 |
> |
| 5 |
> AFAICT, the original error message indicates that more arguments are |
| 6 |
> provided to % than there are place holders in the string. The + |
| 7 |
> operator has lower precedence than %, so the string is not concatenated |
| 8 |
> with the part on the previous line. Note that the string in the error |
| 9 |
> message has 3 place holders only, the fourth should come from the line |
| 10 |
> above. I added parenthesis around the string addition to fix the |
| 11 |
> precedence problem. |
| 12 |
|
| 13 |
Aha. That explains. Now I understand why the fix fixes it as well. |
| 14 |
I'll apply it ASAP. Thanks a lot! |
| 15 |
|
| 16 |
|
| 17 |
-- |
| 18 |
Fabian Groffen |
| 19 |
Gentoo on a different level |
Archives