Hadriel Kaplan
changed
bug 8357
| What |
Removed |
Added |
| Status |
RESOLVED
|
CONFIRMED
|
| Resolution |
FIXED
|
---
|
| Ever confirmed |
|
1
|
Comment # 3
on bug 8357
from Hadriel Kaplan
Sorry about the indentation getting screwed up in the patch, but that probably
caused you to think there was a duplicate return statement - there wasn't.
There really are 2 return statements: one if an error, one which returns 1.
The error case was after an 'else', but that 'else' did not make it into your
delivery. I should have wrapped the return statement in {} after the else to
make it obvious.
So, unfortunately this means the function as delivered into trunk will always
error, instead of only error'ing on an actual error.
:(
You are receiving this mail because:
- You are watching all bug changes.