<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div apple-content-edited="true"><span class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px 0px; 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-align: auto; -khtml-text-decorations-in-effect: none; text-indent: 0px; -apple-text-size-adjust: auto; text-transform: none; orphans: 2; white-space: normal; widows: 2; word-spacing: 0px; "><div style="word-wrap: break-word; -khtml-nbsp-mode: space; -khtml-line-break: after-white-space; "><span class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px 0px; 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-align: auto; -khtml-text-decorations-in-effect: none; text-indent: 0px; -apple-text-size-adjust: auto; text-transform: none; orphans: 2; white-space: normal; widows: 2; word-spacing: 0px; "><span class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px 0px; 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-align: auto; -khtml-text-decorations-in-effect: none; text-indent: 0px; -apple-text-size-adjust: auto; text-transform: none; orphans: 2; white-space: normal; widows: 2; word-spacing: 0px; "><span class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px 0px; 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-align: auto; -khtml-text-decorations-in-effect: none; text-indent: 0px; -apple-text-size-adjust: auto; text-transform: none; orphans: 2; white-space: normal; widows: 2; word-spacing: 0px; "><span class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px 0px; 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-align: auto; -khtml-text-decorations-in-effect: none; text-indent: 0px; -apple-text-size-adjust: auto; text-transform: none; orphans: 2; white-space: normal; widows: 2; word-spacing: 0px; "><span class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px 0px; 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-align: auto; -khtml-text-decorations-in-effect: none; text-indent: 0px; -apple-text-size-adjust: auto; text-transform: none; orphans: 2; white-space: normal; widows: 2; word-spacing: 0px; "><span class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px 0px; 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-align: auto; -khtml-text-decorations-in-effect: none; text-indent: 0px; -apple-text-size-adjust: auto; text-transform: none; orphans: 2; white-space: normal; widows: 2; word-spacing: 0px; "><span class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px 0px; 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-align: auto; -khtml-text-decorations-in-effect: none; text-indent: 0px; -apple-text-size-adjust: auto; text-transform: none; orphans: 2; white-space: normal; widows: 2; word-spacing: 0px; "><span class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px 0px; 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-align: auto; -khtml-text-decorations-in-effect: none; text-indent: 0px; -apple-text-size-adjust: auto; text-transform: none; orphans: 2; white-space: normal; widows: 2; word-spacing: 0px; "><div>No, please don't do that!!!! I hate it when I get a huge file of diffs. The comment should be descriptive enough to let me know if I care about the diffs or not. As an emacs user it is trivial for me to browse diffs there if I *really* care. But e-mail is the *wrong* place for diffs!!</div></span></span></span></span></span></span></span></span></div></span> </div><br><div><div>On Feb 28, 2008, at 4:25 AM, Jay wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><span class="Apple-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; orphans: 2; text-align: auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0; "><div class="hmmessage" style="font-size: 10pt; font-family: Tahoma; ">Is it easy enough for the commit mails to contain links to view all of the diffs described?<br>That's be super duper nice.<br>I find browsing the cvsweb very inefficient.<br> <br> - Jay<br><br><br><blockquote><hr id="EC_stopSpelling">From:<span class="Apple-converted-space"> </span><a href="mailto:jayk123@hotmail.com">jayk123@hotmail.com</a><br>To:<span class="Apple-converted-space"> </span><a href="mailto:m3devel@elegosoft.com">m3devel@elegosoft.com</a><br>Subject: FW: put full paths to source files in debug info<br>Date: Thu, 28 Feb 2008 08:59:47 +0000<br><br>truncated again! and possibly misformated..<br><br><br><br><br><blockquote><hr id="EC_EC_stopSpelling">From:<span class="Apple-converted-space"> </span><a href="mailto:jayk123@hotmail.com">jayk123@hotmail.com</a><br>To:<span class="Apple-converted-space"> </span><a href="mailto:m3devel@elegosoft.com">m3devel@elegosoft.com</a><br>Subject: RE: put full paths to source files in debug info<br>Date: Thu, 28 Feb 2008 08:44:11 +0000<br><br>pps: "webinfo" (Reactor?) and assertion failures are affected by this also, but not fully.<br> <br>Before you would have gotten:<br> <br>***<br>*** runtime error:<br>*** <*ASSERT*> failed.<br>*** file "WaitProcessCygwin.m3", line 16<br>***<br> <br>but now you get:<br> <br>***<br>*** runtime error:<br>*** <*ASSERT*> failed.<br>*** file "..\src\thread\WIN32\WaitProcessCygwin.m3", line 16<br>***<br><br>I'd say the realpath call (or whatever the portable Modula-3 library call is) should be moved up to m3front to address these, and the parse.c change undone.<br> <br>As well, I believe resolving symlinks is happening too but that wasn't the point, so something instead of realpath might be preferable. Like, just see if the string starts ".\" or "..\", it will almost always start "..\", and if so, prepend current working directory and then workout the dots.<br> <br>It may also be reasonable to provide paths to the compiler to remove from the starts of paths, which would likely address the symlink issue, since they are more likely to be nearer the start of the path than the middle or end. As well as partly the privacy/size/same-across-machines issues.<br> <br>In any case, I think this easy small change is already very useful progress. Or does everyone else just fill up .gdbinit with dir commands? It seems to me that it shouldn't even that difficult, even if it isn't very difficult.<br> <br>Agreed?<br> <br> - Jay<br><br><hr id="EC_EC_EC_stopSpelling">From:<span class="Apple-converted-space"> </span><a href="mailto:jayk123@hotmail.com">jayk123@hotmail.com</a><br>To:<span class="Apple-converted-space"> </span><a href="mailto:m3devel@elegosoft.com">m3devel@elegosoft.com</a><br>Subject: re: put full paths to source files in debug info<br>Date: Thu, 28 Feb 2008 08:31:32 +0000<br><br>ps: does anyone care that binaries built from different cvs checkouts, but otherwise identical source, will no longer match, unless perhaps they are "stripped"?<br> <br>If so, or if any of the other issues bug people, or any other problem is brought up or discovered, this can be made a command line option. I will always turn it on. :)<br> <br> - Jay<br><br><hr id="EC_EC_EC_stopSpelling"><br>> Date: Thu, 28 Feb 2008 09:23:22 +0000<br>> To:<span class="Apple-converted-space"> </span><a href="mailto:m3commit@elegosoft.com">m3commit@elegosoft.com</a><br>> From:<span class="Apple-converted-space"> </span><a href="mailto:jkrell@elego.de">jkrell@elego.de</a><br>> Subject: [M3commit] CVS Update: cm3<br>><span class="Apple-converted-space"> </span><br>> CVSROOT: /usr/cvs<br>> Changes by: jkrell@birch. 08/02/28 09:23:22<br>><span class="Apple-converted-space"> </span><br>> Modified files:<br>> cm3/m3-sys/m3cc/gcc/gcc/m3cg/: parse.c<span class="Apple-converted-space"> </span><br>> cm3/m3-sys/m3front/src/misc/: Coverage.m3 Host.i3 Host.m3<span class="Apple-converted-space"> </span><br>> Scanner.m3<span class="Apple-converted-space"> </span><br>><span class="Apple-converted-space"> </span><br>> Log message:<br>> put full paths to source files in debug info<br>><span class="Apple-converted-space"> </span><br>> This has the minor downsides of:<br>> 1) grows the debug info (it is already huge; who is counting?)<br>> 2) reveals file system layout in debug info (privacy?)<br>> 3) does it inhibit debugging files from other people's machines or does gdb dir still work?<br>><span class="Apple-converted-space"> </span><br>> but definitely makes for a more pleasant debugging experience when<br>> debugging stuff you have built yourself.<br>><span class="Apple-converted-space"> </span><br>> The linear searching to see if a name has been allocated a number yet<br>> will obviously slow way down due to a large increase in common prefixes,<br>> but that should be a hash table anyway. Linear search is lame.<br>> (or a trie, but working from the ends of the strings, minus the last one or few<br>> characters, due to common prefixes as well as common suffixes)<br>><span class="Apple-converted-space"> </span><br>> Note that both m3front and m3cc changes are needed as m3front has paths<br>> relative to the current working directory or such.<br>><span class="Apple-converted-space"> </span><br>> For most packages, you can get by without the m3front change and just prepend<br>> "../src/" to the path in m3cc, but that doesn't work for hierarchical packages<br>> such as libm3 and m3core which I am debugging.<br></blockquote><br><hr>Need to know the score, the latest news, or you need your HotmailŪ-get your "fix".<span class="Apple-converted-space"> </span><a href="http://www.msnmobilefix.com/Default.aspx" target="_blank">Check it out.</a></blockquote><br><hr>Helping your favorite cause is as easy as instant messaging. You IM, we give.<span class="Apple-converted-space"> </span><a href="http://im.live.com/Messenger/IM/Home/?source=text_hotmail_join" target="_new">Learn more.</a></div></span></blockquote></div><br></body></html>