Thanks, Zoe! This is the biggest TB 91 blocker right now. I appreciate you looking into it and fixing it. The fix seems to be the same code as Neil did for Owl in bug 1555633 at [the end of the same file](https://searchfox.org/comm-central/source/mailnews/base/src/nsNewMailnewsURI.cpp#113). Code NITs: * Either move the inline function into the `else` block, where it's used, or use the function `NewURI` also in the `if` block, like: `NewURI(); return rv;`
Bug 1663859 Comment 31 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
Thanks, Zoe! This is the biggest TB 91 blocker right now. I appreciate you looking into it and fixing it. The fix seems to be the same code as Neil did for Owl in bug 1555633 at [the end of the same file](https://searchfox.org/comm-central/source/mailnews/base/src/nsNewMailnewsURI.cpp#113). Minor code NITs (doesn't change functionality at all, just easier to read code): * Either move the inline function into the `else` block, where it's used, or use the function `NewURI` also in the `if` block, like: `NewURI(); return rv;`