Register Guidelines E-Books Today's Posts Search

Go Back   MobileRead Forums > E-Book Software > Sigil

Notices

Closed Thread
 
Thread Tools Search this Thread
Old 11-03-2015, 09:08 AM   #136
eschwartz
Ex-Helpdesk Junkie
eschwartz ought to be getting tired of karma fortunes by now.eschwartz ought to be getting tired of karma fortunes by now.eschwartz ought to be getting tired of karma fortunes by now.eschwartz ought to be getting tired of karma fortunes by now.eschwartz ought to be getting tired of karma fortunes by now.eschwartz ought to be getting tired of karma fortunes by now.eschwartz ought to be getting tired of karma fortunes by now.eschwartz ought to be getting tired of karma fortunes by now.eschwartz ought to be getting tired of karma fortunes by now.eschwartz ought to be getting tired of karma fortunes by now.eschwartz ought to be getting tired of karma fortunes by now.
 
eschwartz's Avatar
 
Posts: 19,422
Karma: 85397180
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.
eschwartz is offline  
Old 11-03-2015, 09:23 AM   #137
KevinH
Sigil Developer
KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.KevinH ought to be getting tired of karma fortunes by now.
 
Posts: 7,644
Karma: 5433388
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;
}
where the only reason the uint is used to is try to get around warnigns from other compilers that used to complain in the reverse direction because the length of some objects may be size_t, int, unsigned int depending on platform and compiler used.

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

Quote:
Originally Posted by gbm View Post
I just did a new build of Sigil--Linix Mint 17.2--and had a lot of:
Code:
warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
Are they expected?
The build seems to work ok.

bernie

Last edited by KevinH; 11-04-2015 at 09:02 AM. Reason: added ps to say pushed warning removal changes
KevinH is offline  
Old 11-03-2015, 09:42 AM   #138
Arios
A curiosus lector!
Arios ought to be getting tired of karma fortunes by now.Arios ought to be getting tired of karma fortunes by now.Arios ought to be getting tired of karma fortunes by now.Arios ought to be getting tired of karma fortunes by now.Arios ought to be getting tired of karma fortunes by now.Arios ought to be getting tired of karma fortunes by now.Arios ought to be getting tired of karma fortunes by now.Arios ought to be getting tired of karma fortunes by now.Arios ought to be getting tired of karma fortunes by now.Arios ought to be getting tired of karma fortunes by now.Arios ought to be getting tired of karma fortunes by now.
 
Arios's Avatar
 
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!
Arios is offline  
Closed Thread


Forum Jump

Similar Threads
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


All times are GMT -4. The time now is 04:45 PM.


MobileRead.com is a privately owned, operated and funded community.