#18357: Unicode Art
-------------------------------------+-------------------------------------
       Reporter:  vbraun             |        Owner:
           Type:  enhancement        |       Status:  needs_info
       Priority:  major              |    Milestone:  sage-6.8
      Component:  user interface     |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Volker Braun       |    Reviewers:  Vincent Delecroix
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/vdelecroix/18357                 |  161c0daca2f4a4752725225dfb9238ffc9692ddf
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by vbraun):

 Aah the evils of trailing whitespace. Of course we shouldn't make any
 improvements to Sage until we have conquered that one.

 1. Every object should inherit from `SageObject`. Doesn't really matter
 for stuff that is not exposed but why not?

 2. I just refactored the code, I didn't write it.

 3. There is of course always more than one way to do it but they should
 have different docstrings. Separate functions is the cleanest way to
 implement that.

 5. Of course the bike shed can also be painted pink. Benefit? Feel free to
 change it if you must.

 6. So? Its a good case to show that the code doesn't just apply to
 parenthesis

 7. Afaik yes, e.g. http://en.wikipedia.org/wiki/Bracket

--
Ticket URL: <http://trac.sagemath.org/ticket/18357#comment:28>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to