Re: [yoshimi-user] suggested patch

  • From: Jeremy Jongepier <jeremy@xxxxxxxxxxxxxx>
  • To: yoshimi-user@xxxxxxxxxxxxxxxxxxxxx
  • Date: Mon, 06 Feb 2012 14:27:06 +0100

On 02/06/12 13:02, Brendan Jones wrote:

Hi Jeremy,

I took it upon myself to implement the window icon.

Basically I took your png and converted it to an xpm and added that to
the UI directory (have to change the declaration to static const). This
adds a new dependancy - libXpm.

Icons.fl is a class to hold a single instance of the pixmap, but it is
possible to add other icons/pixmaps to it if you wanted have different
icons for different windows.

There is probably a smarter way to do this by subclassing FL_Window and
its derivatives, but I'm not sure how the fluid editor would react -
there's fewer changes this way I think, and the UI hasn't changed much
since its inception.

Anyway, patch attached. Comments welcome!

regards,

Brendan

Hello Brendan,

Great, thanks! Can't get it to compile though. First I had to change the reference to Fl/x.h to Fl/x.H and then the compilitaion got stuck almost at the end:
[ 82%] Building CXX object CMakeFiles/yoshimi.dir/main.cpp.o
make[2]: *** No rule to make target `/usr/lib/libGL.so', needed by `yoshimi'. Stop.
make[1]: *** [CMakeFiles/yoshimi.dir/all] Error 2
make: *** [all] Error 2

I'm at work trying to set up mail relaying for one of our Nagios servers so I'm not very concentrated. I'll look into it tonight hopefully. This is with Ubuntu 11.10 64bits.

Best,

Jeremy


Other related posts: