# HG changeset patch
# User amit
# Date 1288089290 -19800
# Node ID d14bc84feca16e45fb2c66cef01c79f52c1529cd
# Parent  9ced58c5c3b6e86e3956eccbf898fd5e3b277666
Reviewed manipulating strings

diff -r 9ced58c5c3b6 -r d14bc84feca1 manipulating-strings/script.rst
--- a/manipulating-strings/script.rst	Tue Oct 26 15:44:09 2010 +0530
+++ b/manipulating-strings/script.rst	Tue Oct 26 16:04:50 2010 +0530
@@ -17,7 +17,7 @@
 ..   #. basic datatypes
      
 .. Author              : Puneeth 
-   Internal Reviewer   : 
+   Internal Reviewer   : Amit 
    External Reviewer   :
    Checklist OK?       : <put date stamp here, if OK> [2010-10-05]
 
@@ -36,6 +36,7 @@
 
 .. #[punch: reversed returns an iterator. should we still teach it?]
 
+
 We have an ``ipython`` shell open, in which we are going to work,
 through out this session. 
 
@@ -59,7 +60,7 @@
 
 
 ``s`` could be in any of the forms --- sat, saturday, Sat, Saturday,
-SAT, SATURDAY. We shall now be solving the problem only for the forms,
+SAT, SATURDAY. For now, shall now be solving the problem only for the forms,
 sat and saturday. We shall solve it for the other forms, at the end of
 the tutorial. 
 
@@ -69,6 +70,7 @@
 exists in the variable ``week``. 
 
 As, with any of the string data-types, strings can be sliced into
+.. #[Amit: Sequence data type???]
 sub-strings. To get the first three characters of s, we say, 
 
 ::
@@ -82,7 +84,7 @@
 using ``s[-1]``.  
 
 Following is an exercise that you must do. 
-
+.. #[Amit: I don't know I am not sure about the sentence formation.]
 %%1%% Obtain the sub-string excluding the first and last characters
 from the string s. 
 
@@ -127,7 +129,7 @@
 ::
 
     s[::-1]
-
+.. #[amit: I think using reversed in not required after this]
 Now, to check if the string is ``s`` is palindromic, we say
 ::
 
@@ -157,6 +159,9 @@
 Note that these methods, do not change the original string, but return
 a new string.
 
+.. #[amit: I wish we could include this right when s.upper() is used so 
+.. that it is clear]
+
 Following is an exercise that you must do. 
 
 %%2%% Check if ``s`` is a valid name of a day of the week. Change the
@@ -171,6 +176,8 @@
 
     s.lower()[:3] in week
 
+.. #[amit: May be a sentence or two about what our original problem was and 
+.. how this helps in solving it. One can loose the flow.]
 We just convert any input string to lower case and then check if it is
 present in the list ``week``. 
 
diff -r 9ced58c5c3b6 -r d14bc84feca1 progress.org
--- a/progress.org	Tue Oct 26 15:44:09 2010 +0530
+++ b/progress.org	Tue Oct 26 16:04:50 2010 +0530
@@ -40,7 +40,7 @@
 | 6.5 LO: | Assessment                             |     3 | Anoop    |                                       |           |
 |---------+----------------------------------------+-------+----------+---------------------------------------+-----------|
 | 7.1 LO: | manipulating lists                     |     3 | Madhu    |                                       |           |
-| 7.2 LO: | manipulating strings                   |     2 | Punch    | Pending                               |           |
+| 7.2 LO: | manipulating strings                   |     2 | Punch    | Amit                                  |           |
 | 7.3 LO: | getting started with tuples            |     2 | Nishanth |                                       |           |
 | 7.4 LO: | dictionaries                           |     2 | Anoop    | Pending                               |           |
 | 7.5 LO: | sets                                   |     2 | Nishanth |                                       |           |