Difference between revisions of "User:Mlam19"

From CDOT Wiki
Jump to: navigation, search
m
 
(5 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;"|5 (Fall 2009)
+
| 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;"|[[DPS909]]
+
| 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 XP Professional (virtual machine)
+
*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;"|[[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
 
| style="color:#000000;background-color:#FFFFFF;padding-left:0.4em;"|
 
| style="color:#000000;background-color:#FFFFFF;padding-left:0.4em;"|
*[http://ttfclusiop.hostrator.com/projects/tt2/index.php ''Turbo Trek 2''] (Java game)
+
*[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 89: Line 90:
 
|
 
|
 
*[http://wobblyretroaction.blogspot.com/2009/10/fixing-bug-in-harmless-test-environment.html Thunderbird bug lab]
 
*[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]
 
|}
 
|}
  
Line 156: Line 167:
 
   {
 
   {
 
     if (ItMatchesDelimited(aInString, aInLength,</nowiki></pre>
 
     if (ItMatchesDelimited(aInString, aInLength,</nowiki></pre>
 +
 +
====Week 9 XPCOM lab====
 +
*[http://matrix.senecac.on.ca/~mlam19/dps909/firstxpcom.xpi Extension]
  
 
===Project===
 
===Project===
Line 163: 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]

Latest revision as of 22:36, 15 February 2010

Matthew Lam
College
Program BSD
Semester 6 (Winter 2010)
Course DPS911
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