Review Request 121175: KCodecs: Import KEmailAddress class from email address parsing and validation from KPimUtils

David Faure faure at kde.org
Mon Dec 1 14:38:33 UTC 2014



> On Nov. 23, 2014, 4:26 p.m., David Faure wrote:
> > Problem: I don't think anyone looking for this functionality would think of looking in KCodecs. This is not related to codecs at all.
> > 
> > kcoreaddons/src/lib/text/ has more similar stuff, like KStringHandler::tagUrls.
> > 
> > Thinking about it, we could have renamed KCodecs to KStringUtils and moved the kcoreaddons text subdir there, but it's too late for that.
> > So my suggestion would be to add KEmailAddress to kcoreaddons/src/lib/text.
> 
> Daniel Vrátil wrote:
>     There's a small problem with that, because KEmailAddress depends on the MIME stuff I imported to KCodecs. So either it has to stay in KCodecs, or it has to go to some Tier 2/Tier 3 framework, but there does not seem to be any framework that would fit for this. The only alternative to KCodecs that I can think of is KTextEditor, but I hoped to get the class to a more generic framework.
> 
> Daniel Vrátil wrote:
>     Ping? This blocks the kill-kpimutils plan, which in turn blocks kdepimlibs split. Could we please try to figure out what to do with this asap?

I see. OK for kcodecs then. Makes it slightly more general purpose than just codecs, but I guess that's fine.


- David


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121175/#review70816
-----------------------------------------------------------


On Nov. 19, 2014, 1:23 p.m., Daniel Vrátil wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121175/
> -----------------------------------------------------------
> 
> (Updated Nov. 19, 2014, 1:23 p.m.)
> 
> 
> Review request for KDE Frameworks and David Faure.
> 
> 
> Repository: kcodecs
> 
> 
> Description
> -------
> 
> This class can be very useful to many applications that need to deal with email
> address validation, and so it should not be hidden somewhere in PIM, but
> should be available in some more visibile framework. Also I need to kill the
> KPimUtils framework, and we don't really have any better-fitting framework
> available at this moment :)
> 
> 
> Diffs
> -----
> 
>   autotests/CMakeLists.txt 825e9e8 
>   autotests/kemailaddresstest.h PRE-CREATION 
>   autotests/kemailaddresstest.cpp PRE-CREATION 
>   src/CMakeLists.txt 8b5c4a0 
>   src/kemailaddress.h PRE-CREATION 
>   src/kemailaddress.cpp PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/121175/diff/
> 
> 
> Testing
> -------
> 
> Test passes.
> 
> 
> Thanks,
> 
> Daniel Vrátil
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20141201/a07784fd/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list