https://bugzilla.xfce.org/show_bug.cgi?id=14956
--- Comment #4 from alexxcons <[email protected]> --- @Andre Miranda Real nice work, IMO that's a fair tradeoff ! Just tested it, works fine for me. Only some minor things I noticed during review: - indention of variable declarations in both new methods - 863: /* check if we don't have a directory here */ --> /* check if we have a directory here */ IMO would be nice to have in the 4.14 branch as well. -- You are receiving this mail because: You are the assignee for the bug. _______________________________________________ Xfce-bugs mailing list [email protected] https://mail.xfce.org/mailman/listinfo/xfce-bugs
