Code review comment for lp:~persia/casper/trunk

Revision history for this message
Colin Watson (cjwatson) wrote :

> Fixed the closing quote. Replaced \t\t with real tabs. Updated against
> current trunk.

Thanks, but now I notice that there's a missing "; then" at the end of each if line as well. I'm a bit concerned that syntactic errors mean that this hasn't been tested - perhaps you could build a test image with this and run it through before we merge, to make sure that it really does the right thing?

« Back to merge proposal