Remove useless read method in SExtractor reader.

- No need to store the output in `self.table` and return it.
- `self.cols` already exist in the parent method.
Default reader's outputter is already instantiated in reader's init.
Avoid computing the number of cols at each iteration.
Merge pull request #2947 from saimn/ascii-small-things

Small enhancements for io.ascii