Validate.java use returncodes on error

[Xaratas]

xsd-validator/src/xsdvalidator/validate.java

To use this nice wrapper in a pre-commit hook for git the errorcodes in the inner try need to have a non zero exit code:

            try {
                validator.validate(source);
                System.out.println(mXMLFileName + " validates.");
            }   
            catch (SAXParseException ex) {
                System.out.println(mXMLFileName + " fails to validate because: \n");
                System.out.println(ex.toString());
                System.out.println();
                System.exit(1);
            }   
            catch (SAXException ex) {
                System.out.println(mXMLFileName + " fails to validate because: \n");
                System.out.println(ex.getMessage());
                System.out.println();
                System.exit(1);
            }   
            catch (IOException io) {
                System.err.println("Error reading XML source: " + mXMLFileName);
                System.err.println(io.getMessage());
                System.exit(2);
            }   
        } catch (SAXException sch) {
            System.err.println("Error reading XML Schema: " + mXSDFileName);
            System.exit(2);
        }  

[amouat]

Yeah, that's a bug. I seem to have done it in one place and not others. I'll try to get to this soon, but I'm snowed under at the minute :(

[amouat]

BTW, what was your logic regarding which error code to use? 1 is failed validation and 2 is a file error?

[Xaratas]

So I thought. For my case i yust needed something >0.

[Xaratas]

One more the last catch should be:

        } catch (SAXException sch) {
            System.err.println("Error reading XML Schema: " + sch.getMessage());
            System.exit(2);
        }  

[amouat]

Finally merged this. Apologies for taking so long.

[Xaratas]

Oh, i saw you have not catched the error message change of my last comment. But thats the interesting part, why its not valid. ;)

[amouat]

Sorry, fixed now.