Difference between revisions of "User:Mlam19"

From CDOT Wiki
Jump to: navigation, search
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
  • Java
  • HTML
  • CSS
Rudimentary
  • C
  • C++
  • C#
  • PHP
  • SQL
Projects
Academic Processing.js (Fall 2009/Winter 2010)
Personal
Contact
Email
Blog

Welcome to Matthew Lam's user page!

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

DPS909

Project