Opened 10 years ago

Closed 10 years ago

#11265 closed defect (fixed)

"\dots" in docstring is not converted to "..." properly.

Reported by: klee Owned by: mvngu
Priority: minor Milestone: sage-4.7.2
Component: misc Keywords:
Cc: jhpalmieri Merged in: sage-4.7.2.alpha1
Authors: Kwankyu Lee Reviewers: Ivo Hedtke
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

I looked into the detex function. To me, it seems a little buggy.

  1. "\ldots", "\cdots" are both converted to "...". Hence there is no reason not to convert "\dots" to "..."
  1. There is a bug. For example "souldots" is converted to "sou...". That is, if there is a word which contains "ldots" or "cdots", it is shown broken in command line.

With the change for 'dots', I want to watch out not to convert "The dots and boxes game has been thoroughly studied by Elwyn Berlekamp in his book" to something nonsensical.

Attachments (1)

trac11265.patch (2.7 KB) - added by klee 10 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 10 years ago by jhpalmieri

  • Cc jhpalmieri added

Changed 10 years ago by klee

comment:2 Changed 10 years ago by klee

  • Authors set to Kwankyu Lee
  • Milestone changed from sage-4.7.1 to sage-4.7
  • Status changed from new to needs_review

comment:3 Changed 10 years ago by robertwb

Tests?

comment:4 Changed 10 years ago by hedtke

  • Reviewers set to hedtke
  • Status changed from needs_review to positive_review
  • OK: works in console, notebook, html ref
  • OK: docbuild html ref: no problems
  • OK: docbuild pdf ref: no problems

comment:5 Changed 10 years ago by jdemeyer

  • Component changed from documentation to misc
  • Milestone changed from sage-4.7.1 to sage-4.7.2
  • Reviewers changed from hedtke to Ivo Hedtke

comment:6 Changed 10 years ago by jdemeyer

  • Merged in set to sage-4.7.2.alpha1
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.