robstewart57/hsparql — pull request #2 by cordawyn (2012-09-10 09:17:36+00:00)
Title: Replacing home-brewn RDF types of Hsparql with those of RDF4H

This is a work-in-progress mostly - I could use some feedback on the changes. My concerns are:

1. RDF types are wrapped in Bound type (in BindingValue). This leads to the value being wrapped as a "Bound (LNode (PlainL "..."))" hamburger. Is this OK?

2. There are string conversions that should be avoided (I talking about String <-> ByteString <-> FastString stuff). If I understand it correctly, "String <-> ByteString" can be avoided by using OverloadedStrings and FastString is being taken care of (i.e. removed) already?

I'm also going to write the tests for the modules before this pull request is merged, so don't merge it yet ;-)