search

Cumulus / Syndic

RSS and Atom feed parsing
MIT License
34 stars 13 forks source link

relax grammar? #61

Open pwbs opened 9 years ago

pwbs commented 9 years ago

For imported feed, it seems to me that it would be relevant to have a mode that allows "slightly broken" feeds.

I made a quick patch for my own use because I sort of have to process an invalid feed.

Is anyone interested in taking this direction? (I would not advise to accept broken feeds by default, but only when explicitly asked for)

Chris00 commented 9 years ago

A relaxed mode would certainly be useful. IMHO, it should not take the path of your patch though — functorizing the whole module — but rather using an optional argument to the parsing functions.

dinosaure commented 9 years ago

Hmmmhmm, yeah It's a good way, go to possibly deeper. And, like @Chris00, I prefer an optionnal argument.

pwbs commented 8 years ago

I wouldn't mind using an optional argument. :)

dinosaure commented 8 years ago

@chambart explain one solution (without GADT) to relax the compute of flow and I have a PoC of this idea. All of fields of records with possible fail is registered in relax record. And we compute theses fields with relax. It's general and sounds good, so what do you think of that ?

dinosaure commented 8 years ago

See #66