Difference between revisions of "User:Mlam19"
m |
m |
||
(11 intermediate revisions by the same user not shown) | |||
Line 9: | Line 9: | ||
|- | |- | ||
| style="color:#FFFFFF;background-color:#800000;font-weight:bold;text-align:center;width:40%;"|Semester | | 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;"| | + | | style="color:#000000;background-color:#FFFFFF;padding-left:0.4em;"|6 (Winter 2010) |
|- | |- | ||
| style="color:#FFFFFF;background-color:#800000;font-weight:bold;text-align:center;width:40%;"|Course | | 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;"|[[ | + | | style="color:#000000;background-color:#FFFFFF;padding-left:0.4em;"|[[DPS911]] |
|- | |- | ||
| colspan=2 style="color:#ffffff;background-color:#800000;font-weight:bold;text-align:center;"|System | | colspan=2 style="color:#ffffff;background-color:#800000;font-weight:bold;text-align:center;"|System | ||
Line 18: | Line 18: | ||
| style="color:#FFFFFF;background-color:#800000;font-weight:bold;text-align:center;width:40%;"|Operating Systems | | 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;"| | | style="color:#000000;background-color:#FFFFFF;padding-left:0.4em;"| | ||
− | *[http://www.debian.org/ Debian GNU/Linux] | + | *[http://www.debian.org/ Debian GNU/Linux (AMD64)] |
− | *Windows | + | *Windows 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 | | colspan=2 style="color:#ffffff;background-color:#800000;font-weight:bold;text-align:center;"|Languages | ||
Line 40: | Line 41: | ||
|- | |- | ||
| style="color:#FFFFFF;background-color:#800000;font-weight:bold;text-align:center;width:40%;"|Academic | | 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;"|[[ | + | | style="color:#000000;background-color:#FFFFFF;padding-left:0.4em;"|[[Porting pushStyle(), popStyle(), and boolean() 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:#FFFFFF;background-color:#800000;font-weight:bold;text-align:center;width:40%;"|Personal | ||
| style="color:#000000;background-color:#FFFFFF;padding-left:0.4em;"| | | style="color:#000000;background-color:#FFFFFF;padding-left:0.4em;"| | ||
− | *[http:// | + | *[http://www.reservedbits.com/projects/tt2/index.php ''Turbo Trek 2''] (Java game) |
*[http://starcraft.wikia.com/wiki/User:Meco ''StarCraft Wikia''] | *[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 | | colspan=2 style="color:#ffffff;background-color:#800000;font-weight:bold;text-align:center;"|Contact | ||
|- | |- | ||
Line 58: | Line 59: | ||
|} | |} | ||
Welcome to '''Matthew Lam'''<nowiki>'</nowiki>s user page! | 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] |
Latest revision as of 21:36, 15 February 2010
Matthew Lam | |
College | |
Program | BSD |
Semester | 6 (Winter 2010) |
Course | DPS911 |
System | |
Operating Systems |
|
Languages | |
Familiar |
|
Rudimentary |
|
Projects | |
Academic | Processing.js (Fall 2009/Winter 2010) |
Personal |
|
Contact | |
Blog |
Welcome to Matthew Lam's user page!
Contents
DPS909
Labs
Week | Work |
---|---|
1 | |
2 | |
3 | |
4 | |
5 | |
6 | |
7 | |
8 | |
9 |
Week 6 Thunderbird bug fix lab
Review comments
------- 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,
------- Comment #4 From Ehren Metcalfe 2009-10-17 12:23:58 [reply] ------- (From update of attachment 1403 [details]) Review passed... looks good to me.
Patch
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,
Week 9 XPCOM lab
Project
Other
- Views on Open Source, FSOSS 2009 report