<div dir="ltr">Thank you both for taking a look -- I'm going to move the discussion to the GitLab issue and merge request for better visibility, but to answer your question Andre, unfortunately I don't know of any mock burners :( I just use actual burners.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Oct 24, 2022 at 5:39 AM <<a href="mailto:andre@andreldm.com">andre@andreldm.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
  
    
  
  <div>
<div>Hi all,<br></div><div dir="auto">I was wondering the same, I can do code review (it's in my todolist, just don't wait for me to merge), but without an actual burner it's hard to actually test the app. Do you know any mock burner?<br></div><div><br></div><div>Cheers,<br></div><div>Andre Miranda<br></div><div><br></div><div>Oct 24, 2022, 11:31 by <a href="mailto:trash.paradise@protonmail.com" target="_blank">trash.paradise@protonmail.com</a>:<br></div><blockquote style="border-left:1px solid rgb(147,163,184);padding-left:10px;margin-left:5px"><div>Hi Hunter,<br></div><div><br></div><div>I took a quick look at your MR and it looks good, however I can't say much more as I don't have a burner to test and don't know the Xfburn code.<br></div><div><br></div><div>It builds and installs fine for me, however it then coredumps on launch. But it doesn't seem to be related to this MR, it's also the case with git master, I'll open a new issue about that.<br></div><div><br></div><div>If no one can do a real code review in the near future, which is likely, I think you should go ahead and merge if it's right for you. You can always complete/fix later.<br></div><div><br></div><div>Cheers,<br></div><div>Gaël<br></div><div><br></div><div>------- Original Message -------<br></div><div>On Saturday, October 22nd, 2022 at 3:16 AM, Hunter Turcin <<a href="mailto:huntertur@gmail.com" target="_blank">huntertur@gmail.com</a>> wrote:<br></div><div><br></div><blockquote><div>Hello again :)<br></div><div><br></div><div>I would like to thank you folks again for letting me volunteer to take<br></div><div>on the maintainership of Xfburn. Over the past few days, I've been<br></div><div>working on reviewing the long-pending merge requests and fixing some of<br></div><div>the long-standing bugs, inconsistencies, and crashes in the<br></div><div>application. I also seem to keep finding more of these bugs along the<br></div><div>way...<br></div><div><br></div><div>During an investigation of one such issue (a GTK assertion failure that<br></div><div>occurs when plugging in a burner), I discovered that the bones for<br></div><div>device hotplugging support were present in Xfburn and that those bones<br></div><div>just needed some massaging in order to let the application be able to<br></div><div>respond dynamically to additions and removals of burners instead of<br></div><div>only being able to properly respond to disc change events<br></div><div>(insertion/removal). In my testing so far, this has led to a more<br></div><div>robust Xfburn that is better equipped to handle the user forgetting to<br></div><div>plug in a burner (or forgetting to swap out the wrong burner for the<br></div><div>right burner) without needing to restart the application.<br></div><div><br></div><div>I have more details in the pending merge request (linked below). Please<br></div><div>take a look at reviewing it if anyone has the time (or the knowledge);<br></div><div>while everything seems to work properly in my testing, it would be good<br></div><div>to have some extra pairs of eyes seeing if there's anything missing or<br></div><div>any glaring violations of maintainability that might have been<br></div><div>introduced.<br></div><div><br></div><div><a href="https://gitlab.xfce.org/apps/xfburn/-/merge_requests/23" target="_blank">https://gitlab.xfce.org/apps/xfburn/-/merge_requests/23</a><br></div><div><br></div><div>Additionally, am I correct in my assumption that HAVE_GUDEV is only<br></div><div>ever going to be defined on a Linux system?<br></div><div><br></div><div>Thanks again,<br></div><div>Hunter Turcin<br></div><div>_______________________________________________<br></div><div>Xfce4-dev mailing list<br></div><div><a href="mailto:Xfce4-dev@xfce.org" target="_blank">Xfce4-dev@xfce.org</a><br></div><div><a href="https://mail.xfce.org/mailman/listinfo/xfce4-dev" target="_blank">https://mail.xfce.org/mailman/listinfo/xfce4-dev</a><br></div></blockquote><div>_______________________________________________<br></div><div>Xfce4-dev mailing list<br></div><div><a href="mailto:Xfce4-dev@xfce.org" target="_blank">Xfce4-dev@xfce.org</a><br></div><div><a href="https://mail.xfce.org/mailman/listinfo/xfce4-dev" target="_blank">https://mail.xfce.org/mailman/listinfo/xfce4-dev</a><br></div></blockquote><div dir="auto"><br></div>  </div>

_______________________________________________<br>
Xfce4-dev mailing list<br>
<a href="mailto:Xfce4-dev@xfce.org" target="_blank">Xfce4-dev@xfce.org</a><br>
<a href="https://mail.xfce.org/mailman/listinfo/xfce4-dev" rel="noreferrer" target="_blank">https://mail.xfce.org/mailman/listinfo/xfce4-dev</a></blockquote></div>