add TextMessagePart
ClosedPublic

Authored by knauss on Nov 4 2015, 7:12 PM.

Details

Diff Detail

Repository
R43 KDE PIM
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
knauss updated this revision to Diff 1172.Nov 4 2015, 7:12 PM
knauss retitled this revision from to add TextMessagePart.
knauss added a reviewer: vkrause.
knauss added a project: KDE PIM.
vkrause added inline comments.Nov 7 2015, 9:49 AM
messageviewer/src/viewer/objecttreeparser.h
446

I assume that's also just temporary during refactoring? Otherwise we might want to look for a better name.

knauss added inline comments.Nov 7 2015, 11:12 AM
messageviewer/src/viewer/messagepart.h
28

unrelated change, need to move to other patch.

messageviewer/src/viewer/objecttreeparser.h
446

well there are only two places that refer to that function (defaultHandling and TextMessagePart). So it will be very fast to move that function out of otp.
My plan:

  • defaultHandling should use TextMessagePart, too. After that TextMessagePart is the only occurence of writeBodyStr2
  • because writeBodyStr2 is not more bundled with otp already in any case, the complete logic can move to TextMessagePart.

-> TextMessagePart doesn't need to be a friend class of otp anymore.

vkrause accepted this revision.Nov 12 2015, 10:37 AM
vkrause edited edge metadata.
This revision is now accepted and ready to land.Nov 12 2015, 10:37 AM
This revision was automatically updated to reflect the committed changes.