Difference between revisions of "User:Mlam19"
m |
|||
Line 40: | Line 40: | ||
|- | |- | ||
| 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;"|[[Porting pushStyle(), popStyle(), and boolean() to Processing.js|''Processing.js'' (Fall 2009)]] | + | | 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 | ||
Line 46: | Line 46: | ||
*[http://ttfclusiop.hostrator.com/projects/tt2/index.php ''Turbo Trek 2''] (Java game) | *[http://ttfclusiop.hostrator.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 177: | Line 177: | ||
===Other=== | ===Other=== | ||
*[[User:Mlam19/Views on Open Source|''Views on Open Source'']], FSOSS 2009 report | *[[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] |
Revision as of 05:09, 14 January 2010
Matthew Lam | |
College | |
Program | BSD |
Semester | 5 (Fall 2009) |
Course | DPS909 |
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