This code works OK for me, can you test and/or post code that doesn't work? (attached)

#include <allegro.h>

int main()
{
  allegro_init();
  install_keyboard();
  set_gfx_mode(GFX_AUTODETECT_WINDOWED, 640, 480, 0,0);
  set_window_title("Matthew Leverton Ate My Hamster");
  readkey();
}
END_OF_MAIN()

Pete

Your poor hamster.

Marcello

When I had set_window_title() called before set_gfx_mode(), Windows and Linux windows showed the proper window title while Mac did not.

Personally, and if it's even possible, I prefer not to have an order requirement. Otherwise, a documentation note would be nice.

Not a big deal either way, I'm just glad it works thanks to Peter's help.

It's already been thought of, apart from there's a bug. It works like this:
install_allegro sets the default title (I hope no-one is expecting it to work if called before install_allegro?)

The implementation of set_window_title copies the supplied title into the variable osx_window_title, and does the OS call to actually set the title if the window exists.

In set_gfx_mode, when the window is created, the call
set_window_title(osx_window_title)
is made. This should set the title to whatever has been set previously (which is the default if set_window_title has never been called by the user). This is the desired behaviour. Unfortunately, _al_sane_strcpy fails if source == destination, which it is, in this case. It leaves osx_window_title as an empty string, which is probably what you are seeing.

Still following me?

So I propose an explicit check for this case,

Index: src/macosx/system.m
===================================================================
--- src/macosx/system.m (revision 7527)
+++ src/macosx/system.m (working copy)
@@ -563,7 +563,8 @@
 {
    char tmp[ALLEGRO_MESSAGE_SIZE];
    
-   _al_sane_strncpy(osx_window_title, title, ALLEGRO_MESSAGE_SIZE);
+   if (osx_window_title != title)
+       _al_sane_strncpy(osx_window_title, title, ALLEGRO_MESSAGE_SIZE);
    do_uconvert(title, U_CURRENT, tmp, U_UTF8, ALLEGRO_MESSAGE_SIZE);
 
    NSString *ns_title = [NSString stringWithUTF8String: tmp];

This is IMO the lowest impact solution, though probably not the neatest one.
(diff attached)

Pete

ps. AFAIK, the behaviour of strncpy is supposed to be undefined if source and destination overlap (link) so the behaviour of _al_sane_strncpy seems quite reasonable in this case.