> On Juli 23, 2014, 9:13 vorm., Vishesh Handa wrote: > > src/file/lib/autotests/filemodifyjobtest.cpp, line 78 > > <https://git.reviewboard.kde.org/r/119415/diff/2/?file=292078#file292078line78> > > > > Hi. You're on the right track, but this isn't the exact way to fix the > > test. > > > > Lets take a look at what was happening before? > > * baloo_getxattr returns -1 if the value does not exist and a positive > > value indicating the length if it does exist. > > * Previously, the "user.xdg.tags" xatrr was always being set, even if > > it did not have any value. > > * What value should you be checking for now?
I'm not sure if i understand you. The test check only the old way and only the positiv way (writing a none empty tag). I think we need a new testcase in which explicit writing a empty tag. I recommend a new testmethod. - Felix ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119415/#review62926 ----------------------------------------------------------- On Juli 23, 2014, 8:39 vorm., Felix Eisele wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119415/ > ----------------------------------------------------------- > > (Updated Juli 23, 2014, 8:39 vorm.) > > > Review request for Baloo and Vishesh Handa. > > > Bugs: 333478 > http://bugs.kde.org/show_bug.cgi?id=333478 > > > Repository: baloo > > > Description > ------- > > Pacth to delete xattr from untagged file > > > Diffs > ----- > > src/file/lib/autotests/filemodifyjobtest.cpp ebfc9d9 > src/file/lib/baloo_xattr_p.h 87dc55f > src/file/lib/filecustommetadata.cpp 664b13f > > Diff: https://git.reviewboard.kde.org/r/119415/diff/ > > > Testing > ------- > > > Thanks, > > Felix Eisele > >
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
