Fwd: sage | Fix some compiler warnings, mostly use size_t for indexing (!46)

16 views
Skip to first unread message

Dima Pasechnik

unread,
Jul 29, 2020, 5:29:33 AM7/29/20
to sage-devel

please have a look:

---------- Forwarded message ---------
From: Ivan Komarov <git...@mg.gitlab.com>
Date: Sun, Jul 26, 2020 at 4:42 AM
Subject: Re: sage | Fix some compiler warnings, mostly use size_t for indexing (!46)
To: <dimpase...@gmail.com>


Ivan Komarov started a new discussion on src/sage/schemes/hyperelliptic_curves/hypellfrob/hypellfrob.cpp:

161 161
       zz_p::init(to_long(modulus));
162 162
 
163 163
       // Convert input data to single-precision format
164
-      int dim = M0.NumRows();

unused


Reply to this email directly or view it on GitLab.
You're receiving this email because of your account on gitlab.com. If you'd like to receive fewer emails, you can unsubscribe from this thread or adjust your notification settings.

Dima Pasechnik

unread,
Jul 29, 2020, 5:30:07 AM7/29/20
to sage-devel
please have a look

---------- Forwarded message ---------
From: Ivan Komarov <git...@mg.gitlab.com>
Date: Sun, Jul 26, 2020 at 4:33 AM
Subject: sage | Fix some compiler warnings, mostly use size_t for indexing (!46)
To: <dimpase...@gmail.com>


Ivan Komarov created a merge request:

Project:Branches: IvaKom/sage:size_t to sagemath/sage:develop
Author: Ivan Komarov
Assignees:

Reduce the amount of warnings, mostly with changing for (int i=0;
i<vector.size(); ++i) to for (size_t i=0; i<vector.size(); ++i)

Dima Pasechnik

unread,
Jul 29, 2020, 5:30:38 AM7/29/20
to sage-devel

please  have a look

---------- Forwarded message ---------
From: Ivan Komarov <git...@mg.gitlab.com>
Date: Sun, Jul 26, 2020 at 4:42 AM
Subject: Re: sage | Fix some compiler warnings, mostly use size_t for indexing (!46)
To: <dimpase...@gmail.com>


Ivan Komarov started a new discussion on src/sage/geometry/triangulation/triangulations.cc:

18 18
 void triangulations::find_hash_position(const compact_simplices& t,
19 19
                                         hash_value& pos, bool& is_new) const
20 20
 {
21
-  hash_value freespace;

unused

Reply all
Reply to author
Forward
0 new messages