Closed
Bug 301964
Opened 19 years ago
Closed 19 years ago
Add IETF Atom 1.0 support to Thunderbird
Categories
(MailNews Core :: Feed Reader, enhancement)
MailNews Core
Feed Reader
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sayrer, Assigned: mscott)
Details
Attachments
(2 files, 1 obsolete file)
920 bytes,
text/plain
|
Details | |
14.91 KB,
patch
|
mscott
:
review+
|
Details | Diff | Splinter Review |
Atom is finalized, and we have a namespace to use.
Reporter | ||
Comment 1•19 years ago
|
||
Attachment #190346 -
Flags: review?(mscott)
Reporter | ||
Comment 2•19 years ago
|
||
Assignee | ||
Comment 3•19 years ago
|
||
Comment on attachment 190346 [details] [diff] [review] Add Atom 1.0 support to Thunderbird this worked well for me. Great job as always. I had one question about: serializeTextConstruct: function(textElement) are we doing the right thing with the content variable here? it looks like we initialize it to null, then we change it to an empty string, then at the end of the routine we flip it back to null. Can that be optimized a big more? i.e. start with an empty string and then return content directly instead of converting it back to null. I made a couple very minor white space changes including: getting rid of the braces around single line if/else clauses.
Reporter | ||
Comment 4•19 years ago
|
||
(In reply to comment #3) > Can that be optimized a big more? i.e. start with an empty string and then > return content directly instead of converting it back to null. I've attached something a bit better, but returning null is the correct behavior if we don't find anything in the content element (the storage code depends on it). > I made a couple very minor white space changes including: getting rid of the > braces around single line if/else clauses. Always forget that. Fixed.
Reporter | ||
Comment 5•19 years ago
|
||
Attachment #190346 -
Attachment is obsolete: true
Reporter | ||
Updated•19 years ago
|
Attachment #190346 -
Flags: review?(mscott)
Reporter | ||
Updated•19 years ago
|
Attachment #190499 -
Flags: review?(mscott)
Assignee | ||
Comment 6•19 years ago
|
||
Comment on attachment 190499 [details] [diff] [review] serializeTextConstruct fixed I'm going to change: + if (content == "") + return null; + else + return content; to // we need to return null and not an empty string return content ? content : null; before I check in. Everything else looks good.
Attachment #190499 -
Flags: review?(mscott) → review+
Assignee | ||
Comment 7•19 years ago
|
||
fixed. thanks again for another great patch Robert. Keep 'em coming :)
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird1.1
Component: RSS → Feed Reader
Product: Thunderbird → MailNews Core
Target Milestone: Thunderbird1.1 → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•