![]() |
#136 |
Ex-Helpdesk Junkie
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 19,421
Karma: 85400180
Join Date: Nov 2012
Location: The Beaten Path, USA, Roundworld, This Side of Infinity
Device: Kindle Touch fw5.3.7 (Wifi only)
|
Wanted: Volunteers to Update the Sigil User's Guide
Kevin Hendricks (@KevinH) is the current maintainer of Sigil, he's the one who started this thread. ![]() |
![]() |
![]() |
#137 |
Sigil Developer
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 8,487
Karma: 5703586
Join Date: Nov 2009
Device: many
|
Hi,
Most of the warnings I see not in the 3rdparty section of the source, are related to: Code:
sizet n = length(of something); for (int i=0; i < n; i++) { blah; } or int n = length(of something); for (uint i = 0; i <n; i++) { blah; } No n will be big enough for this to be an issue of overflow of the non-negative range of values of an int i and adding casts just make the code messy. None of these are in any way real issues. The only other warning type in your file is reorder warning as it does not like constructor member initialization lists to be in a different order than the order they are declared in the header for that class. AFAIK, none of these are bugs either. In the C++ spec, the order of initialization in the constructor has nothing to do with the order that members of a class are constructed, destructed, or initialized as that is solely determined by the order of their declaration in the header. So this is a warning about nothing, other than understand the C++ spec on the order of member initialization. The only other warning type in your list is a unused variable warning that again is there becase of other code that is ifdef'd out at the moment but kept around. So nothing to see here either. Nothing to worry about here. Thanks, KevinH ps. I have just pushed changes to Sigil master to hopefully prevent these warnings. If you do another clean build any time soon please let me know if you see anything warningwize in Sigil. Thanks! Kevin Last edited by KevinH; 11-04-2015 at 09:02 AM. Reason: added ps to say pushed warning removal changes |
![]() |
![]() |
#138 |
A curiosus lector!
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() Posts: 463
Karma: 2015140
Join Date: Jun 2012
Device: Sony PRS-T1, Kobo Touch
|
@crutledge
There are no problems with your epub, I was just "playing with words", see my PM to you! |
![]() |
![]() |
|
![]() |
||||
Thread | Thread Starter | Forum | Replies | Last Post |
Sigil-0.8.900 released for testing - Wait for Sigil-0.8.901 | KevinH | Sigil | 106 | 10-04-2015 10:41 AM |
Sigil on Nook vs Sigil on Kobo vs Sigil on iBook | rosshalde | Sigil | 12 | 11-13-2014 09:34 AM |
Sigil 0.7 Beta (0.6.901) Available | user_none | Sigil | 51 | 02-10-2013 07:50 PM |
Sigil 0.5.901 for Mac | ajay42 | Sigil | 14 | 08-30-2012 09:22 AM |
Sigil 0.2.0 Release Candidates | Valloric | Sigil | 46 | 05-10-2010 09:55 PM |