Review Request: make parameters of KoTextEditor const refs if possible instead of passing by value

Friedrich W. H. Kossebau kossebau at kde.org
Thu Jan 3 18:59:15 GMT 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108131/
-----------------------------------------------------------

Review request for Calligra, Pierre Stirnweiss and C. Boemann.


Description
-------

Looked all day at those lines, until I had the need to do something about it :)

Okay? 

For the visitor nethods I was not sure if the object to visit should be passed as ref or const ref, as the methods did not really say if they are supposed to be for read-only or modifying visits. So chose ref, to not restrict.


Diffs
-----

  libs/kotext/KoTextEditor.h e9e5234 
  libs/kotext/KoTextEditor.cpp 18a1972 
  libs/kotext/KoTextEditor_format.cpp 7bd345b 
  libs/kotext/KoTextEditor_p.h 823e888 
  libs/kotext/KoTextEditor_undo.cpp 73600d6 
  libs/kotext/changetracker/KoChangeTracker.h 65ed91f 
  libs/kotext/changetracker/KoChangeTracker.cpp dda52c7 
  libs/kotext/commands/DeleteCommand.cpp 622062a 

Diff: http://git.reviewboard.kde.org/r/108131/diff/


Testing
-------

"make test" results as before.


Thanks,

Friedrich W. H. Kossebau

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20130103/a054e224/attachment.htm>


More information about the calligra-devel mailing list