getting-started-files/script.rst
changeset 345 d82151cc11f5
parent 245 3ed6ef2ea91f
child 379 4b3c0d8fffe2
--- a/getting-started-files/script.rst	Thu Oct 21 14:34:15 2010 +0530
+++ b/getting-started-files/script.rst	Thu Oct 21 18:17:28 2010 +0530
@@ -15,7 +15,7 @@
 .. #. getting started with for
      
 .. Author              : Puneeth
-   Internal Reviewer   : 
+   Internal Reviewer   : Anoop Jacob Thomas<anoop@fossee.in>
    External Reviewer   :
    Checklist OK?       : <put date stamp here, if OK> [2010-10-05]
 
@@ -77,6 +77,8 @@
 
 Following is an exercise that you must do. 
 
+.. #[[Anoop:add context switch to next slide - questions]]
+
 %%1%% Split the variable into a list, ``pend_list``, of the lines in
 the file. Hint, use the tab command to see what methods the string
 variable has.
@@ -87,6 +89,11 @@
 .. where we mention that strings have methods for manipulation. hint:
 .. use splitlines()]
 
+.. #[[Anoop: let us have it here, let us consider this as a
+   refresher]]
+
+.. #[[Anoop:add context switch to next slide - solution]]
+
 ::
 
   pend_list = pend.splitlines()
@@ -99,6 +106,10 @@
 
 .. #[punch: should we mention file-pointer?]
 
+.. #[[Anoop: I think we can say that ``f`` is a file pointer which
+   points to the next line/data to be read from the file. We could
+   skip details.]]
+
 Let us close the file opened into f.
 ::
 
@@ -139,6 +150,9 @@
 keyword. We could have used any other variable name, but ``line``
 seems meaningful enough.
 
+.. #[[Anoop: using dummy variable doesn't seem correct, can say line
+   is a variable]]
+
 Instead of just printing the lines, let us append them to a list,
 ``line_list``. We first initialize an empty list, ``line_list``. 
 ::
@@ -167,6 +181,15 @@
 contain the newline characters, because the string ``pend`` was
 split on the newline characters. 
 
+.. #[[Anoop: I think we need to tell them that each line can be
+   stripped and appended to list to avoid the problem of newline
+   characters.]]
+
+.. #[[Anoop: I think the code that are required to be typed can be
+   added to the slide.]]
+
+.. #[[Anoop: Context switches are to be added.]]
+
 {{{ show the summary slide }}}
 
 That brings us to the end of this tutorial. In this tutorial we