Aren't some of the RetroArch options somewhat irrelevant?

Libretro Lakka
#1

Aren’t some of the RetroArch options somewhat irrelevant? For example, those relating to Windowed vs Fullscreen?

There are also some things that lead to a crash that could probably be removed / changed. For example, in a PS1 multi-disc game (using the M3U method, rather than the “Append Disc” method or the rename-your-save-file method), if you change the disc without first cycling the disc tray status, it just crashes (this is based on Mednafen; no idea whether PCSX-Rearmed would react the same way). So would it not make sense to either automatically cycle when changing the disc, or to remove the option to change it until the disc is ejected? (If it didn’t crash, it could be argued that it’s worth having the option to emulate switching the disc without the PS1 noticing you doing so; but as it is, it’s pointless as anything other than a way to crash Lakka.)

I do get that you’ve probably just plopped RetroArch in, and for now focused more on making sure it works than making sure it’s optimally modified (which is fair enough), but it’s just something that I thought was a bit weird.

EDIT: One more. Is there any way to disable the automatic alphabetic sorting? For example, I don’t want “Final Fantasy IX” to come before “Final Fantasy VII” in the list, I realise this means needing to manually adjust the order in the playlist, but that’s fine; I just basically want Lakka to display them in the playlist order, rather than alphabetical order.

If these aren’t possible, could someone point me towards where to find the source so I can attempt to make these changes myself, as well as a quick guide to compiling it (doesn’t have to be too detailed, I’m usually able to work the finer details out once I get pointed in the right general direction)? I’d be interested in compiling for both x64 and Odroid-XU4.

EDIT: Removed some questions that I found the answers to elsewhere.

#2

Yes you’re right.

We should disable some options on Lakka. And fix that PS1 bug if possible.

Our sources are on github, under the libretro organization.

#3

Alright, thank you. I’ll take a look at that, see if I can get it to compile and make the few tweaks.

I take it there’s no option to disable the automatic sorting, then? I guess that’ll be one of the first tasks for me to try, then… :stuck_out_tongue:

#4

No options for that yet.

I got the behaviour you like with that patch:

diff --git a/menu/menu_displaylist.c b/menu/menu_displaylist.c index 14a0a30…62bba32 100644 — a/menu/menu_displaylist.c +++ b/menu/menu_displaylist.c @@ -1704,7 +1704,7 @@ static int menu_displaylist_parse_horizontal_list(menu_displaylist_info_t *info)

menu_driver_ctl(RARCH_MENU_CTL_PLAYLIST_GET, &playlist);
  • content_playlist_qsort(playlist, menu_displaylist_sort_playlist);

  • //content_playlist_qsort(playlist, menu_displaylist_sort_playlist);

    if (!strcmp(lpl_basename, “content_history”)) is_historylist = true; @@ -2771,7 +2771,7 @@ int menu_displaylist_push_list(menu_displaylist_info_t *info, unsigned type) case DISPLAYLIST_HORIZONTAL: ret = menu_displaylist_parse_horizontal_list(info);

  •     info->need_sort    = true;

  •     info->need_sort    = false;
    info->need_refresh = true;
    info->need_push    = true;
    break;

@@ -2960,13 +2960,13 @@ int menu_displaylist_push_list(menu_displaylist_info_t *info, unsigned type)

         menu_driver_ctl(RARCH_MENU_CTL_PLAYLIST_GET, &playlist);

  •        content_playlist_qsort(playlist, menu_displaylist_sort_playlist);

  •        //content_playlist_qsort(playlist, menu_displaylist_sort_playlist);

       ret = menu_displaylist_parse_playlist(info, playlist, path_playlist, false);

       if (ret == 0)
       {

  •           info->need_sort    = true;

  •           info->need_sort    = false;
          info->need_refresh = true;
          info->need_push    = true;
       }

@@ -2992,6 +2992,7 @@ int menu_displaylist_push_list(menu_displaylist_info_t *info, unsigned type)

         if (ret == 0)
         {

  •           info->need_sort    = false;
          info->need_refresh = true;
          info->need_push    = true;
       }
#5

Is that GitHub code of some kind? Sadly to say I have zero experience working with Git, beyond clicking the “Download All” link and managing code in more traditional ways… but at any rate, thanks for the help. It’s something I should probably try to learn. :slight_smile:

#6

It’s called a diff, or a patch. It is not a github specific format. But github and git are using these extensively.

With colors, it should be more readable http://dpaste.com/2G8F8J0

It is nothing more than a listing of what lines you have to change in the source code to apply the change you want.

#7

Hm… I’ll worry about the diff stuff later… but for now, can you point me towards, if I were to download the ZIP, what environment / compiler I would need to compile it? Especially interested in compiling for Odroid-XU4, but wouldn’t hurt to be able to compile for x64 PC too.

I have access to both Windows 10 and Ubuntu 15.04; the Windows PC is much more powerful so would be preferable if it could be done on there.