Alon Bar-Lev has posted comments on this change.
Change subject: core: filetransaction: Always add a newline
......................................................................
Patch Set 2:
(1 comment)
....................................................
File src/otopi/filetransaction.py
Line 146: if self._content:
Line 147: self._content += '\n'
Line 148: else:
Line 149: self._content = str(content)
Line 150: if not self._content.endswith('\n'):
why not move this to lower indent as common code? removing above?
This way we can also support array with newline embed.
Line 151: self._content += '\n'
Line 152:
Line 153: self._mode = mode
Line 154: self._dmode = dmode
--
To view, visit http://gerrit.ovirt.org/18690
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I97f0feef1c003250c9f7453245f68398c64051cd
Gerrit-PatchSet: 2
Gerrit-Project: otopi
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Yedidyah Bar David <[email protected]>
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
