Review Request: Changes in processing playlist files

2012-11-26 Thread Tatjana Gornak
d saving through gui. Thanks, Tatjana Gornak ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request: Changes in processing playlist files

2012-11-26 Thread Tatjana Gornak
aylist::title(), but to get title you need to load playlist, thereore title() can not be a constant method. - Tatjana --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107473/#rev

Re: Review Request: Changes in processing playlist files

2012-12-29 Thread Tatjana Gornak
org/r/107473/diff/ Testing --- 1) All unit-tests were passed. 2) For all playlists I've also checked loading and saving through gui. Thanks, Tatjana Gornak ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request: Changes in processing playlist files

2013-01-03 Thread Tatjana Gornak
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107473/#review24408 ----------- On Dec. 29, 2012, 12:46 p.m., Tatjana Gornak wrote: > > --

Re: Review Request: Changes in processing playlist files

2013-01-11 Thread Tatjana Gornak
ttp://git.reviewboard.kde.org/r/107473/#review24408 ------- On Jan. 3, 2013, 12:34 p.m., Tatjana Gornak wrote: > > --- > This is an automatically generated e-mail

Re: Review Request 107473: Changes in processing playlist files

2013-01-15 Thread Tatjana Gornak
aylists/types/file/xspf/XSPFPlaylist.h ebf3235 src/core/playlists/Playlist.h 8fd1ffb Diff: http://git.reviewboard.kde.org/r/107473/diff/ Testing --- 1) All unit-tests were passed. 2) For all playlists I've also checked loading and sa

Re: Review Request 107473: Changes in processing playlist files

2013-02-02 Thread Tatjana Gornak
wboard.kde.org/r/107473/diff/ Testing --- 1) All unit-tests were passed. 2) For all playlists I've also checked loading and saving through gui. Thanks, Tatjana Gornak ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request 107473: Changes in processing playlist files

2013-02-03 Thread Tatjana Gornak
) All unit-tests were passed. 2) For all playlists I've also checked loading and saving through gui. Thanks, Tatjana Gornak ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request 107473: Changes in processing playlist files

2013-02-03 Thread Tatjana Gornak
pl/playlists/types/file/xspf/TestXSPFPlaylist.cpp f4643ca Diff: http://git.reviewboard.kde.org/r/107473/diff/ Testing --- 1) All unit-tests were passed. 2) For all playlists I've also checked loading and saving through gui. Thanks, Tatjana Gornak

Re: Review Request 107473: Changes in processing playlist files

2013-02-07 Thread Tatjana Gornak
/TestM3UPlaylist.cpp 0f19e1b tests/core-impl/playlists/types/file/xspf/TestXSPFPlaylist.cpp f4643ca Diff: http://git.reviewboard.kde.org/r/107473/diff/ Testing --- 1) All unit-tests were passed. 2) For all playlists I've also checked loading and saving throu

Re: Review Request 107473: Changes in processing playlist files

2013-02-12 Thread Tatjana Gornak
encoding detection job to QDomDocument - Tatjana --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107473/#review27042 -------

Re: Review Request 107473: Changes in processing playlist files

2013-03-02 Thread Tatjana Gornak
) All unit-tests were passed. 2) For all playlists I've also checked loading and saving through gui. Thanks, Tatjana Gornak ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request 107473: Changes in processing playlist files

2013-03-04 Thread Tatjana Gornak
rTrackLoad will affect all inherited from Playlist classes, so it will be harder to remove this code. If it is ok, then I'll add this optional argument to triggerTrackLoad. - Tatjana --- This is an automatically generate

Re: Review Request 107473: Changes in processing playlist files

2013-03-05 Thread Tatjana Gornak
ould insert newly loaded tracks. - Tatjana --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107473/#review28555 ------- On Ma

Re: Review Request 107473: Changes in processing playlist files

2013-03-05 Thread Tatjana Gornak
re processed ('return' above prevents it to execute if not all entities in m_tracks were processed). - Tatjana ------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107473/#review285

Re: Review Request 107473: Changes in processing playlist files

2013-03-10 Thread Tatjana Gornak
f/ Testing --- 1) All unit-tests were passed. 2) For all playlists I've also checked loading and saving through gui. Thanks, Tatjana Gornak ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request 107473: Changes in processing playlist files

2013-03-10 Thread Tatjana Gornak
>= 0 ) > > loadingFinished( playlist ); // fool ourselves because nobody else > > would trigger this > > else > > subscribeTo( playlist ); // let loadingFinished() be called by the > > playlist > > > > I also think you can us

Re: Review Request 107473: Changes in processing playlist files

2013-03-12 Thread Tatjana Gornak
view28961 ------- On March 11, 2013, 1:57 a.m., Tatjana Gornak wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org

Re: Review Request 107473: Changes in processing playlist files

2013-03-14 Thread Tatjana Gornak
ests were passed. 2) For all playlists I've also checked loading and saving through gui. Thanks, Tatjana Gornak ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request 107473: Changes in processing playlist files

2013-03-16 Thread Tatjana Gornak
keLists.txt e2e0ce4 tests/core/playlists/TestPlaylistObserver.h PRE-CREATION tests/core/playlists/TestPlaylistObserver.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/107473/diff/ Testing --- 1) All unit-tests were passed. 2) For all playlists I've also checked loading and

Re: Review Request 107473: Changes in processing playlist files

2013-03-26 Thread Tatjana Gornak
p 5ea3a41 tests/core/playlists/CMakeLists.txt e2e0ce4 Diff: http://git.reviewboard.kde.org/r/107473/diff/ Testing --- 1) All unit-tests were passed. 2) For all playlists I've also checked loading and saving through gui. Thanks, Tatjana Gornak __

Review Request 109758: Asx playlist implementation.

2013-03-26 Thread Tatjana Gornak
/r/109758/diff/ Testing --- Loading and saving works Thanks, Tatjana Gornak ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request 107473: Changes in processing playlist files

2013-03-27 Thread Tatjana Gornak
rver.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/107473/diff/ Testing --- 1) All unit-tests were passed. 2) For all playlists I've also checked loading and saving through gui. Thanks, Tatjana Gornak ___ Amarok-devel mailin

Re: Review Request 107473: Changes in processing playlist files

2013-03-27 Thread Tatjana Gornak
But ok, protected will work as well. - Tatjana --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107473/#review29948 ------- On

Re: Review Request 107473: Changes in processing playlist files

2013-03-28 Thread Tatjana Gornak
This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107473/#review29971 --- On March 27, 2013, 7:35 p.m., Tatjana Gornak wrote: > > --

Re: Review Request 107473: Changes in processing playlist files

2013-03-29 Thread Tatjana Gornak
the track > > order is not preserved. Could you please have a look at it? Perhaps it is > > better to query all tracks at once after trackLoaded() is called instead of > > using trackAdded() method. > > Tatjana Gornak wrote: > Yes, sure I'll try to fix it. >

Re: Review Request 107473: Changes in processing playlist files

2013-04-01 Thread Tatjana Gornak
ia/uploaded/files/2013/03/27/0001-Cleanups-and-fixes-for-Tatjana-s-changes_1.patch Thanks, Tatjana Gornak ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Review Request 109879: Small cleanups in playlists

2013-04-05 Thread Tatjana Gornak
/ Testing --- Thanks, Tatjana Gornak ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request 109758: Asx playlist implementation.

2013-04-05 Thread Tatjana Gornak
/TestASXPlaylist.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/109758/diff/ Testing --- Loading and saving works Thanks, Tatjana Gornak ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request 109879: Small cleanups in playlists

2013-04-06 Thread Tatjana Gornak
://git.reviewboard.kde.org/r/109879/diff/ Testing --- Thanks, Tatjana Gornak ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request 109758: Asx playlist implementation.

2013-04-06 Thread Tatjana Gornak
/TestASXPlaylist.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/109758/diff/ Testing --- Loading and saving works Thanks, Tatjana Gornak ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request 109758: Asx playlist implementation.

2013-04-09 Thread Tatjana Gornak
/TestASXPlaylist.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/109758/diff/ Testing --- Loading and saving works Thanks, Tatjana Gornak ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request 109758: Asx playlist implementation.

2013-04-11 Thread Tatjana Gornak
: http://git.reviewboard.kde.org/r/109758/#review30945 ------- On April 9, 2013, 8:24 p.m., Tatjana Gornak wrote: > > --- > This is an automatically generated e-m

Re: Review Request 109758: Asx playlist implementation.

2013-04-14 Thread Tatjana Gornak
/TestASXPlaylist.h PRE-CREATION tests/core-impl/playlists/types/file/asx/TestASXPlaylist.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/109758/diff/ Testing --- Loading and saving works Thanks, Tatjana Gornak ___ Amarok-devel mailing list Amarok

Re: Review Request 109758: Asx playlist implementation.

2013-04-14 Thread Tatjana Gornak
/TestASXPlaylist.h PRE-CREATION tests/core-impl/playlists/types/file/asx/TestASXPlaylist.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/109758/diff/ Testing --- Loading and saving works Thanks, Tatjana Gornak ___ Amarok-devel mailing list Amarok

Re: Review Request 109758: Asx playlist implementation.

2013-04-14 Thread Tatjana Gornak
-CREATION tests/core-impl/playlists/types/file/asx/TestASXPlaylist.cpp PRE-CREATION Diff: http://git.reviewboard.kde.org/r/109758/diff/ Testing --- Loading and saving works Thanks, Tatjana Gornak ___ Amarok-devel mailing list Amarok-devel

Review Request 110081: Launching amarok with --cdplay adds CD in playlist

2013-04-18 Thread Tatjana Gornak
Diffs - src/core-impl/collections/audiocd/AudioCdCollection.cpp 3dfa7c3 ChangeLog cc8b166 src/MainWindow.cpp 4273d8a Diff: http://git.reviewboard.kde.org/r/110081/diff/ Testing --- Launching amarok with --cdplay adds CD in playlist Thanks, Tatjan

Outdated MediaDeviceFramework

2013-04-24 Thread Tatjana Gornak
While working on my gsoc proposal I did some research on Collections and I am not sure that I understand why MediaDeviceFramework is outdated (it states in few places, but without mentioning any particular reason). Maybe there is some additional info somewhere and I just did not find it? I've look

Re: Review Request 110150: Feedback when transcoding during ripping an Audio CD fails

2013-04-28 Thread Tatjana Gornak
<http://git.reviewboard.kde.org/r/110150/#comment23626> Note: you can test if changes introduce whitespace errors with git diff --check - Tatjana Gornak On April 27, 2013, 2:17 p.m., Abhinandan Ramaprasath wrote: > > ---

GSoC proposal draft: Audio СD collection refactoring.

2013-04-29 Thread Tatjana Gornak
Hi all, I am looking forward to get some feedback on my proposal. Thanks in advance. =Proposal draft= ==Contact info== Name: Tatiana Gornak Email Address: t.gor...@gmail.com Freenode IRC Nick: melandory IM Service and Username: xmpp, err...@gmail.com Location (City, Country and/or Time Zone): Ka

Week 1 report. AudioCD collection refactoring.

2013-06-22 Thread Tatjana Gornak
Hi all, this is what was done during first week of GSoC: 1) New implementation of AudioCD collection was started: New AudioCDCollectionFactory implementation supplied with new version of AudioCDCollection. AudioCDCollectionFactory creates AudioCDCollection when Solid reports about new device

AudioCD. Week 2.

2013-07-04 Thread Tatjana Gornak
Hi all, I've summarized progress of last week in blog post: http://melandory.blogspot.de/2013/07/audiocd-week-2.html -- Sincerely yours, Tatiana Gornak ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-deve

Audiocd week3 report

2013-07-09 Thread Tatjana Gornak
Hi all, report for last week: http://melandory.blogspot.de/2013/07/audiocd-week-3.html -- Sincerely yours, Tatiana Gornak ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Review Request 114142: GSoC 2013: AudioCD

2013-11-26 Thread Tatjana Gornak
: http://git.reviewboard.kde.org/r/114142/diff/ Testing --- Thanks, Tatjana Gornak ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel

Re: Review Request 114142: GSoC 2013: AudioCD

2013-11-26 Thread Tatjana Gornak
/collections/CollectionLocation.cpp 209d6b4 Diff: http://git.reviewboard.kde.org/r/114142/diff/ Testing --- Thanks, Tatjana Gornak ___ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel