Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Showing line no. of error in tag value by throwing expections. #100

Merged
merged 1 commit into from
Jul 18, 2017

Conversation

rtgdk
Copy link
Contributor

@rtgdk rtgdk commented Jul 18, 2017

Added try catch in TagToRDF for returning line no. of error by throwing exceptions when executing HandBuiltParser.
It still shows 2 exceptions when it is an invalid file. One for Tag Value Exception and one for RDF.
So when we pass an invalid file format or an invalid TagValue or invalid RDF file, it will combine 2 exception together. It needs to fixed.

…ceptions.

	* It still shows 2 exceptions when it is an invalid file. One for Tag Value Exception and one for RDF.

Signed-off-by: rtgdk <rohit.lodhartg@gmail.com>
@rtgdk
Copy link
Contributor Author

rtgdk commented Jul 18, 2017

@goneall Can we differentiate between an error of invalid file format and invalid tag value file when calling HandBuiltParser? We can check which type of error it is and throw only the required error.
Like if we have an invalid tag value file, there is no need to check for a RDF file which is what is happening currently in ConvertSPDXDocs.openRdfOrTagDoc .

@goneall goneall merged commit ebbb752 into spdx:master Jul 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants