fvpolpeta / devide

Automatically exported from code.google.com/p/devide
BSD 3-Clause "New" or "Revised" License
2 stars 1 forks source link

CodeRunner (and other shells) should better sanitise / visualise source code issues #31

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
With version 8.2, mixing tabs and spaces in a python file, then loading and
executing that in the CodeRunner silently doesn't work.  Also lines
consisting of just whitespace are problematic.

This is an interpreter problem, but the CodeRunner should probably check
for these types of errors before submitting.

1. Create some test cases (and tests!)
2. Make logic for sanitising code (more than already happens).

Original issue reported on code.google.com by cpbotha on 11 Mar 2008 at 4:08

GoogleCodeExporter commented 9 years ago
Emiel contributed the following reproducer:

Hey Charl,

ik heb een voorwaarde gevonden voor het uitvoeren van scripts in de coderunner
scratch/setup/execute (waarvan we al een paar keer wijgering ervaren hebben):

er moet minstens 1 lege regel of een regel code staan na de laatste indention 
van het
script. Een commented regel telt niet mee.

Voorbeeld:

>for i in range(5):
>  print i
>#

werkt niet, maar

>for i in range(5):
>    print i
>print 'einde'

of

>for i in range(5):
>    print i
>

gaat wel goed.

ik kan me niet herinneren of dit ook fout was in mijn code, dus misschien zijn 
er nog
steeds andere uitzonderingen. Maar het is in ieder geval een begin :)

Groetjes,
Emiel

Original comment by cpbotha on 14 May 2008 at 2:20

GoogleCodeExporter commented 9 years ago
Fixed in r2994.

Original comment by cpbotha on 7 Jun 2008 at 10:27