Vote Details for "XBM" by alan_k

» Details
  • Voter: Alan Knowles 
  • Vote: +1 (conditional)
  • Reviews: Cursory source review
» Comment
few minor CS fixes
* your constants should be prefixed IMAGE_XBM_ not just XBM
* a few short if statements (always use the { }), and the alightment of multiline conditions is a little odd (add more, rather than less indentation)
* @include can be a point of much fustration. - it's usually a programming error / eg. not installed), rather than a runtime error...