1
edit
Changes
m
no edit summary
|-
| style="color:#FFFFFF;background-color:#800000;font-weight:bold;text-align:center;width:40%;"|Semester
| style="color:#000000;background-color:#FFFFFF;padding-left:0.4em;"|5 6 (Fall 2009Winter 2010)
|-
| style="color:#FFFFFF;background-color:#800000;font-weight:bold;text-align:center;width:40%;"|Course
| style="color:#000000;background-color:#FFFFFF;padding-left:0.4em;"|[[DPS909DPS911]]
|-
| colspan=2 style="color:#ffffff;background-color:#800000;font-weight:bold;text-align:center;"|System
| style="color:#FFFFFF;background-color:#800000;font-weight:bold;text-align:center;width:40%;"|Operating Systems
| style="color:#000000;background-color:#FFFFFF;padding-left:0.4em;"|
*[http://www.debian.org/ Debian GNU/Linux(AMD64)]*Windows XP Professional Vista (32-bit) (virtual machine)*Windows 7 (64-bit) (virtual machine)
|-
| colspan=2 style="color:#ffffff;background-color:#800000;font-weight:bold;text-align:center;"|Languages
|-
| style="color:#FFFFFF;background-color:#800000;font-weight:bold;text-align:center;width:40%;"|Academic
| style="color:#000000;background-color:#FFFFFF;padding-left:0.4em;"|[[Processing.js Porting pushStyle() , popStyle(), and popStyleboolean()to Processing.js|''Processing.js'' (Fall 2009/Winter 2010)]]
|-
| style="color:#FFFFFF;background-color:#800000;font-weight:bold;text-align:center;width:40%;"|Personal
| style="color:#000000;background-color:#FFFFFF;padding-left:0.4em;"|
*[http://ttfclusiopwww.hostratorreservedbits.com/projects/tt2/index.php ''Turbo Trek 2''] (Java game)
*[http://starcraft.wikia.com/wiki/User:Meco ''StarCraft Wikia'']
|-http://zenit.senecac.on.ca/wiki/index.php/Processing.js
| colspan=2 style="color:#ffffff;background-color:#800000;font-weight:bold;text-align:center;"|Contact
|-
|}
Welcome to '''Matthew Lam'''<nowiki>'</nowiki>s user page!
==DPS909==
===Labs===
{| border="1" cellpadding="5"
|-
!Week
!Work
|-
|1
|
*[http://wobblyretroaction.blogspot.com/2009/09/milestone-of-sorts-first-blog-post-ever.html Introduction blog]
|-
|2
|
*[http://wobblyretroaction.blogspot.com/2009/09/disembodied-voices-and-talking-heads.html Dial-in response blog]
*[http://zenit.senecac.on.ca/wiki/index.php?title=User%3AEgmetcalfe&diff=27166&oldid=27058 IRC lab]
|-
|3
|
*[http://wobblyretroaction.blogspot.com/2009/09/building-minefield-or-how-i-put-some.html Building Firefox lab]
|-
|4
|
*[http://wobblyretroaction.blogspot.com/2009/09/my-kingdom-for-patch.html Patching lab]
|-
|5
|
|-
|6
|
*[http://wobblyretroaction.blogspot.com/2009/10/fixing-bug-in-harmless-test-environment.html Thunderbird bug lab]
|-
|7
|
|-
|8
|
|-
|9
|
*[http://wobblyretroaction.blogspot.com/2009/11/firefox-addon-and-xpcom.html XPCOM lab]
|}
====Week 6 Thunderbird bug fix lab====
=====Review comments=====
<pre><nowiki>------- Comment #2 From Ehren Metcalfe 2009-10-17 12:08:25 [reply] -------
(From update of attachment 1402 [details])
diff -r e90895696bde netwerk/streamconv/converters/mozTXTToHTMLConv.cpp
>--- a/netwerk/streamconv/converters/mozTXTToHTMLConv.cpp Tue Oct 13 16:51:12 2009 -0400
>+++ b/netwerk/streamconv/converters/mozTXTToHTMLConv.cpp Sat Oct 17 13:08:43 2009 -0400
>@@ -191,17 +191,20 @@ mozTXTToHTMLConv::CompleteAbbreviatedURL
> if (pos >= aInLength)
> return;
>
> if (aInString[pos] == '@')
> {
> // only pre-pend a mailto url if the string contains a .domain in it..
> //i.e. we want to linkify johndoe@foo.com but not "let's meet @8pm"
> nsDependentString inString(aInString, aInLength);
>- if (inString.FindChar('.', pos) != kNotFound) // if we have a '.' after the @ sign....
>+ // if we have a '.' after the @ sign,
>+ // and there are not '..' after the @ sign
>+ if (inString.FindChar('.', pos) != kNotFound &&
Align second line of if expression with the first (8 spaces vs 12)
>+ inString.Find("..", pos) == kNotFound)
> {
> aOutString.AssignLiteral("mailto:");
> aOutString += aInString;
> }
> }
> else if (aInString[pos] == '.')
> {
> if (ItMatchesDelimited(aInString, aInLength,</nowiki></pre>
<pre><nowiki>------- Comment #4 From Ehren Metcalfe 2009-10-17 12:23:58 [reply] -------
(From update of attachment 1403 [details])
Review passed... looks good to me.</nowiki></pre>
=====Patch=====
<pre><nowiki>diff -r e90895696bde netwerk/streamconv/converters/mozTXTToHTMLConv.cpp
--- a/netwerk/streamconv/converters/mozTXTToHTMLConv.cpp Tue Oct 13 16:51:12 2009 -0400
+++ b/netwerk/streamconv/converters/mozTXTToHTMLConv.cpp Sat Oct 17 15:11:20 2009 -0400
@@ -191,17 +191,20 @@ mozTXTToHTMLConv::CompleteAbbreviatedURL
if (pos >= aInLength)
return;
if (aInString[pos] == '@')
{
// only pre-pend a mailto url if the string contains a .domain in it..
//i.e. we want to linkify johndoe@foo.com but not "let's meet @8pm"
nsDependentString inString(aInString, aInLength);
- if (inString.FindChar('.', pos) != kNotFound) // if we have a '.' after the @ sign....
+ // if we have a '.' after the @ sign,
+ // and there are not '..' after the @ sign
+ if (inString.FindChar('.', pos) != kNotFound &&
+ inString.Find("..", pos) == kNotFound)
{
aOutString.AssignLiteral("mailto:");
aOutString += aInString;
}
}
else if (aInString[pos] == '.')
{
if (ItMatchesDelimited(aInString, aInLength,</nowiki></pre>
====Week 9 XPCOM lab====
*[http://matrix.senecac.on.ca/~mlam19/dps909/firstxpcom.xpi Extension]
===Project===
*[[Porting_pushStyle(),_popStyle(),_and_boolean()_to_Processing.js|Project page]]
*[http://wobblyretroaction.blogspot.com/2009/09/processingjs-plan-to-push-and-pop.html Initial project plan blog]
*[http://wobblyretroaction.blogspot.com/2009/10/boolean-for-processingjs.html Project expansion blog]
===Other===
*[[User:Mlam19/Views on Open Source|''Views on Open Source'']], FSOSS 2009 report
==DPS909==
===Project===
*[[Porting_pushStyle(),_popStyle(),_and_boolean()_to_Processing.js|Project page]]
*[http://wobblyretroaction.blogspot.com/2010/01/2010-dps911-and-more-processingjs.html Project continuation blog]