Does this look okay?: disable_print = disable_print || !IsPrinterAvailable();
Let me know if it looks okay and I'll commit the changes. I can't test it since I don't have VC++ but it looks right... -_-' Same change is needed for the BeOS section, too, it looks like. -Mark On Thu, 13 May 2004, Bill Kendrick wrote: > I just got a report that "noprint=yes" in tuxpaint.cfg is getting ignored > in Windows. > > Looking at 'tuxpaint.c' in CVS, I see in "reset_avail_tools()" that we do > this: > > #ifdef WIN32 > disable_print = !IsPrinterAvailable(); > #endif > > Whoops! That clobbers whatever setting was already in there! > I think we need a "printer_available" boolean variable, that gets checked > at the same time "disable_print" is checked, rather than clobbering the > "disable_print" setting. :^) > > > In other words... user says: "don't print". Tux Paint goes: "Is there a > printer? Yes! Turn on the print button!" > > Oops ;) > > > Anyone feel like patching? I'm away from my Linux dev box until this weekend. > > Thanks! > > > (Maybe thank "Adam Moore" in the Changelog, for pointing this bug out! ;) ) > > -bill! > _______________________________________________ > Tuxpaint-dev mailing list > [EMAIL PROTECTED] > http://tux4kids.net/mailman/listinfo/tuxpaint-dev > -- Mark K. Kim AIM: markus kimius Homepage: http://www.cbreak.org/ Xanga: http://www.xanga.com/vindaci Friendster: http://www.friendster.com/user.jsp?id=13046 PGP key fingerprint: 7324 BACA 53AD E504 A76E 5167 6822 94F0 F298 5DCE PGP key available on the homepage _______________________________________________ Tuxpaint-dev mailing list [EMAIL PROTECTED] http://tux4kids.net/mailman/listinfo/tuxpaint-dev
