René 'Necoro' Neumann

Results 25 comments of René 'Necoro' Neumann

Stumbling over this issue: Yes, the parser holds internal state (cf. internal/shared/xmlbase). Question is: What is the purpose of using the same parser instance twice? Why not instantiate a new...

Hi @mmcdole, would you mind giving a feedback, whether this has any chance of getting included and/or what is needed to do so? (I also noted, that the coverage bot...

@mmcdole I use your library for the parsing part in an [RSS reader](https://github.com/Necoro/feed2imap-go). A user found the StackOverflow RSS feed to have that problem and opened [an issue with me](https://github.com/Necoro/feed2imap-go/issues/24#issuecomment-669448395)....

> [![Coverage Status](https://camo.githubusercontent.com/1e16dcf17a53574510d79dea6b021e6a95cb3cfc27c1a280b22057003e6f0fca/68747470733a2f2f636f766572616c6c732e696f2f6275696c64732f33393039313532352f6261646765)](https://coveralls.io/builds/39091525) > > Coverage increased (+0.5%) to 75.256% when pulling **[5a9d204](https://github.com/mmcdole/gofeed/commit/5a9d204f8125960177546c9f4f83461b24c79e86) on Necoro:fix_151** into **[5d81d4b](https://github.com/mmcdole/gofeed/commit/5d81d4b54b22ccc4b0f038a0130d342bc6b88312) on mmcdole:master**. Wow... I finally convinced him to run \o/

That one was tricky: `wget -O - https://www.dailytelegraph.com.au/feed` outputs the expected feed. But when looking at all the redirects: ``` --2021-06-15 00:12:20-- https://www.dailytelegraph.com.au/feed/ Loaded CA certificate '/etc/ssl/certs/ca-certificates.crt' Resolving www.dailytelegraph.com.au (www.dailytelegraph.com.au)......

Ok, special atom handling in RSS is already done for `link`, so I'll do the same for author. While at it, I'll incorporate `atom` and `atom03` into `canonicalNamespaces` to avoid...

What about ``` OptionalHeightFlag := HeightFlag OptionalHeightFlag.Required = False ```

I have to change my report: It works also in decorators. The bug I noticed was another one: it only works when you write 'key=value' and not when using 'key...

The problematic part is: ```go // TODO: this could break extensions (this only works when item is FLAGS) if fields, ok := value.([]interface{}); ok { for i, field := range...

change `imap.DeletedFlag` to `[]interface{}{imap.DeletedFlag}`