Review Request 127208: Remove Q3ValueList usage from RecipeDB::importUSDADatabase()

R.Harish Navnit harishnavnit at gmail.com
Mon Mar 28 10:04:19 UTC 2016



> On Feb. 29, 2016, 1:04 p.m., José Manuel  Santamaría Lema wrote:
> > Indeed QList seems appropiate to replace Q3ValueList, tested the patch with SQLite and works.
> > 
> > Thanks for the patch! Keep them coming ;)
> 
> R.Harish  Navnit wrote:
>     It seems like I lack privileges to push to this repo. Can you do it on my behalf ?
> 
> Albert Astals Cid wrote:
>     You should not lack the privileges since KDE commit policy is repo-wide.
>     
>     Which error do you get?
> 
> R.Harish  Navnit wrote:
>     This one : https://paste.kde.org/pswqqszwu
> 
> Albert Astals Cid wrote:
>     What does 
>       cat .git/config  | grep url
>     return when on the krecipes folder?

it returns an http url and a git:// url pointing to anongit.kde.org/krecipes. https://paste.kde.org/pkjfxxd3d


- R.Harish


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


On Feb. 28, 2016, 7:09 p.m., R.Harish  Navnit wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127208/
> -----------------------------------------------------------
> 
> (Updated Feb. 28, 2016, 7:09 p.m.)
> 
> 
> Review request for Krecipes and José Manuel  Santamaría Lema.
> 
> 
> Repository: krecipes
> 
> 
> Description
> -------
> 
> Replace the Q3ValueList occurences in RecipeDB::importUSDADatabase() with QList.
> QList preferred over QLinkedList since it's faster and convenient.
> 
> 
> Diffs
> -----
> 
>   src/backends/recipedb.cpp aa38c6b4b602544cfbfe3b7db73a8b9c59abe518 
> 
> Diff: https://git.reviewboard.kde.org/r/127208/diff/
> 
> 
> Testing
> -------
> 
> Successfully built and re-ran the setup wizard. Imports work fine.
> 
> 
> Thanks,
> 
> R.Harish  Navnit
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/krecipes-devel/attachments/20160328/3302393b/attachment-0001.html>


More information about the Krecipes-devel mailing list