<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 10pt;
font-family:Verdana
}
--></style>
</head>
<body class='hmmessage'>
Yes it was crashing on Windows.<br>It wasn't really "Windows-specific", it was specifically in the code Randy added.<br>The two files are largely the same and they are both portable.<br>It's just a matter of what pictures get drawn using the same underlying Trestle graphics functions.<br>That is, the code is portable either way, it's just a behaviorial/visual decision that is made.<br>You know, you could perfectly well allow:<br><br> mentor -gui-style=windows <br> mentor -gui-style=trestle <br><br>You know, this isn't low level code, it is "middle".<br><br> - Jay<br><br><br><hr id="stopSpelling">From: hosking@cs.purdue.edu<br>Date: Mon, 14 Dec 2009 10:10:37 -0500<br>To: jay.krell@cornell.edu<br>CC: m3devel@elegosoft.com<br>Subject: Re: [M3devel] FW: Win32/ScrollerVBTClass.m3<br><br>
<base><div><span class="ecxApple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;"><span class="ecxApple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;"><div style="word-wrap: break-word;"><span class="ecxApple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;"><span class="ecxApple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;"><span class="ecxApple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;"><span class="ecxApple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;"><span class="ecxApple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;"><span class="ecxApple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;"><span class="ecxApple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;"><span class="ecxApple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;"><div><span class="ecxApple-style-span" style="font-size: medium;"><font class="ecxApple-style-span" color="#0000ff" face="'Gill Sans'">Do you mean mentor crashing on Windows? I would not like us to introduce new bugs to X11-based Trestle by merging Windows with X11 Trestle.</font></span></div></span></span></span></span></span></span></span></span></div></span></span>
</div>
<br><div><div>On 13 Dec 2009, at 20:43, Jay K wrote:</div><br class="ecxApple-interchange-newline"><blockquote><span class="ecxApple-style-span" style="border-collapse: separate; font-family: Helvetica; font-size: medium; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;"><div class="ecxhmmessage" style="font-size: 10pt; font-family: Verdana;">Randy's changes make the Windows GUI look more like<br>Windows. My changes make mentor not crash starting<br>up just by checking for NIL and taking a lock.<br>I don't understand Trestle's locking and the comments<br>explaining it were too long for me to understand,<br>so I'm just guessing.<br> <br>Given how ugly and non-uniform X Windows GUIs tend to be,<br>Trestle being a good example -- it has its own<br>unique ugly look, changing is not a bad idea,<br>but not done here.<br> <br>- Jay<br><br> <br><hr id="ecxstopSpelling">From:<span class="ecxApple-converted-space"> </span><a href="mailto:hosking@cs.purdue.edu">hosking@cs.purdue.edu</a><br>Date: Sun, 13 Dec 2009 13:30:18 -0500<br>To:<span class="ecxApple-converted-space"> </span><a href="mailto:rcoleburn@scires.com">rcoleburn@scires.com</a><br>CC:<span class="ecxApple-converted-space"> </span><a href="mailto:m3devel@elegosoft.com">m3devel@elegosoft.com</a>;<span class="ecxApple-converted-space"> </span><a href="mailto:jay.krell@cornell.edu">jay.krell@cornell.edu</a><br>Subject: Re: [M3devel] FW: Win32/ScrollerVBTClass.m3<br><br>Is that what these changes do? Make POSIX GUI like Windows. Let's please not do that gratuitously.<div><div><span class="ecxecxApple-style-span" style="text-transform: none; text-indent: 0px; border-collapse: separate; font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; font-size: 12px; line-height: normal; font-size-adjust: none; font-stretch: normal; -x-system-font: none; white-space: normal; letter-spacing: normal; color: rgb(0, 0, 0); word-spacing: 0px;"><span class="ecxecxApple-style-span" style="text-transform: none; text-indent: 0px; border-collapse: separate; font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; font-size: 12px; line-height: normal; font-size-adjust: none; font-stretch: normal; -x-system-font: none; white-space: normal; letter-spacing: normal; color: rgb(0, 0, 0); word-spacing: 0px;"><div style="word-wrap: break-word;"><span class="ecxecxApple-style-span" style="text-transform: none; text-indent: 0px; border-collapse: separate; font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; font-size: 12px; line-height: normal; font-size-adjust: none; font-stretch: normal; -x-system-font: none; white-space: normal; letter-spacing: normal; color: rgb(0, 0, 0); word-spacing: 0px;"><span class="ecxecxApple-style-span" style="text-transform: none; text-indent: 0px; border-collapse: separate; font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; font-size: 12px; line-height: normal; font-size-adjust: none; font-stretch: normal; -x-system-font: none; white-space: normal; letter-spacing: normal; color: rgb(0, 0, 0); word-spacing: 0px;"><span class="ecxecxApple-style-span" style="text-transform: none; text-indent: 0px; border-collapse: separate; font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; font-size: 12px; line-height: normal; font-size-adjust: none; font-stretch: normal; -x-system-font: none; white-space: normal; letter-spacing: normal; color: rgb(0, 0, 0); word-spacing: 0px;"><span class="ecxecxApple-style-span" style="text-transform: none; text-indent: 0px; border-collapse: separate; font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; font-size: 12px; line-height: normal; font-size-adjust: none; font-stretch: normal; -x-system-font: none; white-space: normal; letter-spacing: normal; color: rgb(0, 0, 0); word-spacing: 0px;"><span class="ecxecxApple-style-span" style="text-transform: none; text-indent: 0px; border-collapse: separate; font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; font-size: 12px; line-height: normal; font-size-adjust: none; font-stretch: normal; -x-system-font: none; white-space: normal; letter-spacing: normal; color: rgb(0, 0, 0); word-spacing: 0px;"><span class="ecxecxApple-style-span" style="text-transform: none; text-indent: 0px; border-collapse: separate; font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; font-size: 12px; line-height: normal; font-size-adjust: none; font-stretch: normal; -x-system-font: none; white-space: normal; letter-spacing: normal; color: rgb(0, 0, 0); word-spacing: 0px;"><span class="ecxecxApple-style-span" style="text-transform: none; text-indent: 0px; border-collapse: separate; font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; font-size: 12px; line-height: normal; font-size-adjust: none; font-stretch: normal; -x-system-font: none; white-space: normal; letter-spacing: normal; color: rgb(0, 0, 0); word-spacing: 0px;"><span class="ecxecxApple-style-span" style="text-transform: none; text-indent: 0px; border-collapse: separate; font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; font-size: 12px; line-height: normal; font-size-adjust: none; font-stretch: normal; -x-system-font: none; white-space: normal; letter-spacing: normal; color: rgb(0, 0, 0); word-spacing: 0px;"><div><span class="ecxecxApple-style-span" style="font-size: medium;"><font class="ecxecxApple-style-span" color="#0000ff" face="'Gill Sans'"><br></font></span></div></span></span></span></span></span></span></span></span></div></span></span></div><br><div><div>On 13 Dec 2009, at 00:02, Randy Coleburn wrote:</div><br class="ecxecxApple-interchange-newline"><blockquote><span class="ecxecxApple-style-span" style="text-transform: none; text-indent: 0px; border-collapse: separate; font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; font-size: medium; line-height: normal; font-size-adjust: none; font-stretch: normal; -x-system-font: none; white-space: normal; letter-spacing: normal; word-spacing: 0px;"><div class="ecxecxhmmessage" style="font-family: Verdana; font-size: 10pt;"><div>Jay:</div><div> </div><div>Back in 2008 when I made these changes I did go to the current source tree and factor all this work into the then current tree. It was not a file replace situation, but rather very careful work trying to merge the changes back into the current tree. The POSIX implementation did not change, because the modifications were intended for Windows only. I think it would be wrong to try and make the GUI on Unix look like Windows.</div><div> </div><div>Regards,</div><div>Randy<br><br>>>> Jay K <<a href="mailto:jay.krell@cornell.edu">jay.krell@cornell.edu</a>> 12/12/2009 10:41 PM >>><br>2008-05-27 02:29 rcoleburn<br><br> * m3-ui/vbtkit/src/: lego/m3makefile, vbtkitresources/ScrollBg,<br> vbtkitresources/close.ppm, vbtkitresources/down.ppm,<br> vbtkitresources/grow.ppm, vbtkitresources/left.ppm,<br> vbtkitresources/m3makefile, vbtkitresources/right.ppm,<br> vbtkitresources/up.ppm, lego/POSIX/ScrollerVBTClass.m3,<br> lego/POSIX/ZChassisVBT.m3, lego/POSIX/m3makefile,<br> vbtkitutils/m3makefile, lego/WIN32/ScrollerVBTClass.m3,<br> lego/WIN32/ZChassisVBT.m3, lego/WIN32/m3makefile,<br> vbtkitutils/POSIX/VBTKitEnv.i3, vbtkitutils/POSIX/m3makefile,<br> vbtkitutils/WIN32/VBTKitEnv.i3, vbtkitutils/WIN32/m3makefile:<br><br> Incorporate changes made to vbtkit by Critical Mass circa September 1997 that make the GUI appearance more like that of Windows. The m3makefiles have been adjusted so that these changes affect only Windows (WIN32) platforms. These changes were part of work sponsored by Randy Coleburn at Scientific Research Corporation (SRC). The main files affected are VBTKitEnv.i3, ScrollerVBTClass.m3, and ZChassisVBT.m3, plus some new resource files have been added for WIN32 only.<br><br><br>Randy, are files from 1997 really correct to incorporate into the tree 11 years later? Or maybe they are rather out of date?<br>Can the work be redone against the 2008 versions? And factor out the common parts?<br> <br>I guess I should go and diff these against the 3.6 or 4.1 versions?<br>And then apply those diffs to the Posix file to get the Win32 version?<br>(Or again, try to share what is the same?)<br> <br> <br> - Jay<br> <br>> Date: Sun, 13 Dec 2009 04:37:34 +0000<br>> To:<span class="ecxecxApple-converted-space"> </span><a href="mailto:m3commit@elegosoft.com">m3commit@elegosoft.com</a><br>> From:<span class="ecxecxApple-converted-space"> </span><a href="mailto:jkrell@elego.de">jkrell@elego.de</a><br>> Subject: [M3commit] CVS Update: cm3<br>><span class="ecxecxApple-converted-space"> </span><br>> CVSROOT: /usr/cvs<br>> Changes by: jkrell@birch. 09/12/13 04:37:34<br>><span class="ecxecxApple-converted-space"> </span><br>> Modified files:<br>> cm3/m3-ui/vbtkit/src/lego/: Tag: release_branch_cm3_5_8<span class="ecxecxApple-converted-space"> </span><br>> m3makefile<span class="ecxecxApple-converted-space"> </span><br>> cm3/m3-ui/vbtkit/src/lego/WIN32/: Tag: release_branch_cm3_5_8<span class="ecxecxApple-converted-space"> </span><br>> ScrollerVBTClass.m3<span class="ecxecxApple-converted-space"> </span><br>><span class="ecxecxApple-converted-space"> </span><br>> Log message:<br>> I forgot to mention: I'm really just guessing here. It seems to work.<br>><span class="ecxecxApple-converted-space"> </span><br></div></div></span></blockquote></div><br></div></div></span><br class="ecxApple-interchange-newline"></blockquote></div><br> </body>
</html>