Improve the loop in the `LiteralProxies` class constructor

RDFLib / OWL-RL

A simple implementation of the OWL2 RL Profile on top of RDFLib: it expands the graph with all possible triples that OWL RL defines. It can be used together with RDFLib to expand an RDFLib Graph object, or as a stand alone service with its own serialization.

http://www.ivan-herman.net/Misc/2008/owlrl/

Other

140 stars 30 forks source link

Improve the loop in the `LiteralProxies` class constructor #9

Closed wrobell closed 5 years ago

wrobell commented 5 years ago

Build the to_be_removed variable upfront to contain all the triples with literal object, then process it. This avoids incremental building of the set and makes code less nested.

ashleysommer commented 5 years ago

Nice simple optimisation. I like it. Looks good to me.