Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"except:" count in sage too high #9447

Closed
sagetrac-wjlaffin mannequin opened this issue Jul 7, 2010 · 2 comments
Closed

"except:" count in sage too high #9447

sagetrac-wjlaffin mannequin opened this issue Jul 7, 2010 · 2 comments

Comments

@sagetrac-wjlaffin
Copy link
Mannequin

sagetrac-wjlaffin mannequin commented Jul 7, 2010

sage: len(search_src("except:",interact=False).splitlines())
312

This number should be much lower.

Component: misc

Reviewer: Andrey Novoseltsev

Issue created by migration from https://trac.sagemath.org/ticket/9447

@sagetrac-wjlaffin sagetrac-wjlaffin mannequin assigned aghitza and jasongrout and unassigned aghitza Jul 7, 2010
@sagetrac-wjlaffin sagetrac-wjlaffin mannequin removed the c: algebra label Jul 7, 2010
@tscrim
Copy link
Collaborator

tscrim commented Apr 1, 2013

comment:2

These all seem to be gone in 5.9.beta1:

travis@travis-virtualbox:~/sage-5.9.beta1/devel/sage-main/sage$ grep -R "except:" .
./numerical/backends/glpk_backend.cpp:            /*except:*/ {
./structure/list_clone_timings_cy.c:        /*except:*/ {
./plot/plot3d/bugs.txt:        except:       # TODO -- this would catch control-C,etc. -- FIX THIS TO CATCH WHAT IS RAISED!!!!
./libs/singular/function.cpp:        /*except:*/ {

sage: len(search_src("except:",interact=False).splitlines())
0

@novoselt
Copy link
Member

novoselt commented Apr 2, 2013

Reviewer: Andrey Novoseltsev

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants