Re: Review Request 125164: Finish killed KIO jobs

2015-11-03 Thread Aleix Pol Gonzalez

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

(Updated Nov. 3, 2015, 11:44 p.m.)


Status
--

This change has been discarded.


Review request for KDE Frameworks.


Repository: kio


Description
---

Scheduler::jobFinished cleans up the job but doesn't report it as such.


Diffs
-

  src/core/simplejob.cpp 3380cbe 
  autotests/jobtest.h 7641131 
  autotests/jobtest.cpp 7da92d5 

Diff: https://git.reviewboard.kde.org/r/125164/diff/


Testing
---

Added a test and made it pass.


Thanks,

Aleix Pol Gonzalez

___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Review Request 125164: Finish killed KIO jobs

2015-10-30 Thread Aleix Pol Gonzalez

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

(Updated Oct. 30, 2015, 12:46 p.m.)


Review request for KDE Frameworks.


Changes
---

Updated the patch not to try to kill jobs that already finished, as suggested 
by David.

Still doesn't solve the issue.

```
snapshot(2084)/(default) unknown: No file found for ".xml" , even though 
update-mime-info said it would exist.
ksnapshot(2084)/(default) unknown: Either it was just removed, or the directory 
doesn't have executable permission...
ksnapshot(2084)/(default) unknown: ("/home/apol/.local/share/mime", 
"/home/kde-devel/kde5/share/mime", "/usr/share/mime")
ksnapshot(2084)/(default) KIO::SimpleJob::doKill: killing a job that already 
finished KIO::ListJob(0xec3f90)
ksnapshot(2084)/(default) KIO::SimpleJobPrivate::simpleJobInit: Invalid URL: 
QUrl("kdevelop-build1.png")
ksnapshot(2084)/(default) KIO::SimpleJobPrivate::simpleJobInit: Invalid URL: 
QUrl("kdevelop-build1.bmp")
ksnapshot(2084)/(default) KIO::SimpleJobPrivate::simpleJobInit: Invalid URL: 
QUrl("kdevelop-build1.bmp")
ksnapshot(2084)/(default) KIO::SimpleJobPrivate::simpleJobInit: Invalid URL: 
QUrl("kdevelop-build1.png")
ksnapshot(2084)/(default) KDirModelPrivate::_k_slotNewItems: Items emitted in 
directory QUrl("file:///home/apol/Documents") but that directory isn't in 
KDirModel! Root directory: QUrl("file:///home/apol/imatges")
ksnapshot(2084)/(default) KDirModelPrivate::_k_slotNewItems: Item: 
QUrl("file:///home/apol/Documents/sudokuTitle.png")
ksnapshot(2084)/(default) KDirModelPrivate::_k_slotNewItems: Item: 
QUrl("file:///home/apol/Documents/firma.png")   

   
ksnapshot(2084)/(default) KDirModelPrivate::dump: Dumping contents of KDirModel 
KDirModel(0xa3c5f0) dirLister url: QUrl("file:///home/apol/imatges")

  
ksnapshot(2084)/(default) unknown: ASSERT: "result" in file 
/home/kde-devel/frameworks/kio/src/widgets/kdirmodel.cpp, line 433  

  



  
Program received signal SIGABRT, Aborted.   


  
0x722885f8 in raise () from /usr/lib/libc.so.6  


  
(gdb) where 


  
#0  0x722885f8 in raise () from /usr/lib/libc.so.6  


  
#1  0x72289a7a in abort () from /usr/lib/libc.so.6  


  
#2  0x72f2805e in qt_message_fatal (context=..., message=) at 
/home/kde-devel/frameworks/qt5/qtbase/src/corelib/global/qlogging.cpp:1610  

 
#3  QMessageLogger::fatal (this=this@entry=0x7fff9960, 
msg=msg@entry=0x731db7f0 "ASSERT: \"%s\" in file %s, line %d") at 
/home/kde-devel/frameworks/qt5/qtbase/src/corelib/global/qlogging.cpp:784   
 
#4  0x72f235de in qt_assert (assertion=, file=, line=) at 
/home/kde-devel/frameworks/qt5/qtbase/src/corelib/global/qglobal.cpp:3046   

   
Python Exception  cannot resolve overloaded method `data': 

Re: Review Request 125164: Finish killed KIO jobs

2015-10-29 Thread Aleix Pol Gonzalez

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


Reopened because now everyone has the crash I was complaining on the reviews 
there:
https://git.reviewboard.kde.org/r/125613/

Will have to look into how to fix properly, whatever that means.

- Aleix Pol Gonzalez


On Oct. 29, 2015, 1:35 p.m., Aleix Pol Gonzalez wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125164/
> ---
> 
> (Updated Oct. 29, 2015, 1:35 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: kio
> 
> 
> Description
> ---
> 
> Scheduler::jobFinished cleans up the job but doesn't report it as such.
> 
> 
> Diffs
> -
> 
>   src/core/simplejob.cpp 3380cbe 
>   autotests/jobtest.cpp c24bcea 
>   autotests/jobtest.h ef8c3e1 
> 
> Diff: https://git.reviewboard.kde.org/r/125164/diff/
> 
> 
> Testing
> ---
> 
> Added a test and made it pass.
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel


Re: Review Request 125164: Finish killed KIO jobs

2015-10-22 Thread Aleix Pol Gonzalez

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

(Updated Oct. 23, 2015, 2:48 a.m.)


Review request for KDE Frameworks.


Changes
---

When killing a job, report it as finished.


Summary (updated)
-

Finish killed KIO jobs


Repository: kio


Description (updated)
---

Scheduler::jobFinished cleans up the job but doesn't report it as such.


Diffs (updated)
-

  src/core/simplejob.cpp 3380cbe 
  autotests/jobtest.cpp c24bcea 
  autotests/jobtest.h ef8c3e1 

Diff: https://git.reviewboard.kde.org/r/125164/diff/


Testing
---

Added a test and made it pass.


Thanks,

Aleix Pol Gonzalez

___
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel