Re: new compiler warnings in master

2015-03-23 Thread Jürgen Spitzmüller
2015-03-22 18:27 GMT+01:00 Georg Baum:

 This would look like the attached. Before I commit: Any wishes for other
 names?


Perhaps:
hyphen - softhyphen
slash - breakableslash

to make their semantics more explicit.

Jürgen




 Georg


Re: new compiler warnings in master

2015-03-23 Thread Georg Baum
Jürgen Spitzmüller wrote:

 Perhaps:
 hyphen - softhyphen
 slash - breakableslash
 
 to make their semantics more explicit.

Good idea, I did it like that.


Georg



Re: new compiler warnings in master

2015-03-23 Thread Jürgen Spitzmüller
2015-03-22 18:27 GMT+01:00 Georg Baum:

> This would look like the attached. Before I commit: Any wishes for other
> names?
>

Perhaps:
hyphen -> softhyphen
slash -> breakableslash

to make their semantics more explicit.

Jürgen


>
>
> Georg


Re: new compiler warnings in master

2015-03-23 Thread Georg Baum
Jürgen Spitzmüller wrote:

> Perhaps:
> hyphen -> softhyphen
> slash -> breakableslash
> 
> to make their semantics more explicit.

Good idea, I did it like that.


Georg



Re: new compiler warnings in master

2015-03-22 Thread Georg Baum
Georg Baum wrote:

 OK, I will change it. But since the LyX file format really has nothing to
 do with LaTeX (except for math), I will remove all backslashes and {}
 pairs from InsetSpecialChar. BTW, LaTeX2E is not the only one which is
 different.

This would look like the attached. Before I commit: Any wishes for other 
names?


Georgdiff --git a/lib/lyx2lyx/LyX.py b/lib/lyx2lyx/LyX.py
index 89b08c8..d96b7b4 100644
--- a/lib/lyx2lyx/LyX.py
+++ b/lib/lyx2lyx/LyX.py
@@ -85,7 +85,7 @@ format_relation = [(0_06,[200], minor_versions(0.6 , 4)),
(1_6, list(range(277,346)), minor_versions(1.6 , 10)),
(2_0, list(range(346,414)), minor_versions(2.0, 8)),
(2_1, list(range(414,475)), minor_versions(2.1, 0)),
-   (2_2, list(range(475,483)), minor_versions(2.2, 0))
+   (2_2, list(range(475,484)), minor_versions(2.2, 0))
   ]
 
 
diff --git a/lib/lyx2lyx/lyx_2_2.py b/lib/lyx2lyx/lyx_2_2.py
index b4ce71e..5d91085 100644
--- a/lib/lyx2lyx/lyx_2_2.py
+++ b/lib/lyx2lyx/lyx_2_2.py
@@ -645,6 +645,49 @@ def revert_phrases(document):
 i += 1
 
 
+def convert_specialchar_internal(document, forward):
+specialchars = {\\-:hyphen, \\textcompwordmark{}:ligaturebreak, \
+\\@.:endofsentence, \\ldots{}:ldots, \
+\\menuseparator:menuseparator, \\slash{}:slash, \
+\\nobreakdash-:nobreakdash-, \\LyX:LyX, \
+\\TeX:TeX, \\LaTeX2e:LaTeX2e, \
+\\LaTeX:LaTeX # must be after LaTeX2e
+}
+
+i = 0
+while i  len(document.body):
+words = document.body[i].split()
+if len(words)  1 and words[0] == \\begin_inset and \
+   words[1] in [CommandInset, External, Formula, Graphics, listings]:
+# see convert_phrases
+j = find_end_of_inset(document.body, i)
+if j == -1:
+document.warning(Malformed LyX document: Can't find end of Formula inset at line  + str(i))
+i += 1
+else:
+i = j
+continue
+for key, value in specialchars.iteritems():
+if forward:
+document.body[i] = document.body[i].replace(\\SpecialChar  + key, \\SpecialChar  + value)
+document.body[i] = document.body[i].replace(\\SpecialCharNoPassThru  + key, \\SpecialCharNoPassThru  + value)
+else:
+document.body[i] = document.body[i].replace(\\SpecialChar  + value, \\SpecialChar  + key)
+document.body[i] = document.body[i].replace(\\SpecialCharNoPassThru  + value, \\SpecialCharNoPassThru  + key)
+i += 1
+
+
+def convert_specialchar(document):
+convert special characters to new syntax
+convert_specialchar_internal(document, True)
+
+
+def revert_specialchar(document):
+convert special characters to old syntax
+convert_specialchar_internal(document, False)
+
+
+
 ##
 # Conversion hub
 #
@@ -661,10 +704,12 @@ convert = [
[479, []],
[480, []],
[481, [convert_dashes]],
-   [482, [convert_phrases]]
+   [482, [convert_phrases]],
+   [483, [convert_specialchar]]
   ]
 
 revert =  [
+   [482, [revert_specialchar]],
[481, [revert_phrases]],
[480, [revert_dashes]],
[479, [revert_question_env]],
diff --git a/src/insets/InsetSpecialChar.cpp b/src/insets/InsetSpecialChar.cpp
index 6d967bf..c3ef2ec 100644
--- a/src/insets/InsetSpecialChar.cpp
+++ b/src/insets/InsetSpecialChar.cpp
@@ -283,37 +283,37 @@ void InsetSpecialChar::write(ostream  os) const
 	string command;
 	switch (kind_) {
 	case HYPHENATION:
-		command = \\-;
+		command = hyphen;
 		break;
 	case LIGATURE_BREAK:
-		command = \\textcompwordmark{};
+		command = ligaturebreak;
 		break;
 	case END_OF_SENTENCE:
-		command = \\@.;
+		command = endofsentence;
 		break;
 	case LDOTS:
-		command = \\ldots{};
+		command = ldots;
 		break;
 	case MENU_SEPARATOR:
-		command = \\menuseparator;
+		command = menuseparator;
 		break;
 	case SLASH:
-		command = \\slash{};
+		command = slash;
 		break;
 	case NOBREAKDASH:
-		command = \\nobreakdash-;
+		command = nobreakdash;
 		break;
 	case PHRASE_LYX:
-		command = \\LyX;
+		command = LyX;
 		break;
 	case PHRASE_TEX:
-		command = \\TeX;
+		command = TeX;
 		break;
 	case PHRASE_LATEX2E:
-		command = \\LaTeX2e;
+		command = LaTeX2e;
 		break;
 	case PHRASE_LATEX:
-		command = \\LaTeX;
+		command = LaTeX;
 		break;
 	}
 	os  \\SpecialChar   command  \n;
@@ -326,27 +326,27 @@ void InsetSpecialChar::read(Lexer  lex)
 	lex.next();
 	string const command = lex.getString();
 
-	if (command == \\-)
+	if (command == hyphen)
 		kind_ = HYPHENATION;
-	else if (command == \\textcompwordmark{})
+	else if (command == ligaturebreak)
 		kind_ = LIGATURE_BREAK;
-	else if (command == \\@.)
+	else if (command == endofsentence)
 		kind_ = 

Re: new compiler warnings in master

2015-03-22 Thread Jean-Marc Lasgouttes

Le 22/03/15 18:27, Georg Baum a écrit :

Georg Baum wrote:


OK, I will change it. But since the LyX file format really has nothing to
do with LaTeX (except for math), I will remove all backslashes and {}
pairs from InsetSpecialChar. BTW, LaTeX2E is not the only one which is
different.


This would look like the attached. Before I commit: Any wishes for other
names?


Fine with me.

JMarc



Re: new compiler warnings in master

2015-03-22 Thread Georg Baum
Georg Baum wrote:

> OK, I will change it. But since the LyX file format really has nothing to
> do with LaTeX (except for math), I will remove all backslashes and {}
> pairs from InsetSpecialChar. BTW, LaTeX2E is not the only one which is
> different.

This would look like the attached. Before I commit: Any wishes for other 
names?


Georgdiff --git a/lib/lyx2lyx/LyX.py b/lib/lyx2lyx/LyX.py
index 89b08c8..d96b7b4 100644
--- a/lib/lyx2lyx/LyX.py
+++ b/lib/lyx2lyx/LyX.py
@@ -85,7 +85,7 @@ format_relation = [("0_06",[200], minor_versions("0.6" , 4)),
("1_6", list(range(277,346)), minor_versions("1.6" , 10)),
("2_0", list(range(346,414)), minor_versions("2.0", 8)),
("2_1", list(range(414,475)), minor_versions("2.1", 0)),
-   ("2_2", list(range(475,483)), minor_versions("2.2", 0))
+   ("2_2", list(range(475,484)), minor_versions("2.2", 0))
   ]
 
 
diff --git a/lib/lyx2lyx/lyx_2_2.py b/lib/lyx2lyx/lyx_2_2.py
index b4ce71e..5d91085 100644
--- a/lib/lyx2lyx/lyx_2_2.py
+++ b/lib/lyx2lyx/lyx_2_2.py
@@ -645,6 +645,49 @@ def revert_phrases(document):
 i += 1
 
 
+def convert_specialchar_internal(document, forward):
+specialchars = {"\\-":"hyphen", "\\textcompwordmark{}":"ligaturebreak", \
+"\\@.":"endofsentence", "\\ldots{}":"ldots", \
+"\\menuseparator":"menuseparator", "\\slash{}":"slash", \
+"\\nobreakdash-":"nobreakdash-", "\\LyX":"LyX", \
+"\\TeX":"TeX", "\\LaTeX2e":"LaTeX2e", \
+"\\LaTeX":"LaTeX" # must be after LaTeX2e
+}
+
+i = 0
+while i < len(document.body):
+words = document.body[i].split()
+if len(words) > 1 and words[0] == "\\begin_inset" and \
+   words[1] in ["CommandInset", "External", "Formula", "Graphics", "listings"]:
+# see convert_phrases
+j = find_end_of_inset(document.body, i)
+if j == -1:
+document.warning("Malformed LyX document: Can't find end of Formula inset at line " + str(i))
+i += 1
+else:
+i = j
+continue
+for key, value in specialchars.iteritems():
+if forward:
+document.body[i] = document.body[i].replace("\\SpecialChar " + key, "\\SpecialChar " + value)
+document.body[i] = document.body[i].replace("\\SpecialCharNoPassThru " + key, "\\SpecialCharNoPassThru " + value)
+else:
+document.body[i] = document.body[i].replace("\\SpecialChar " + value, "\\SpecialChar " + key)
+document.body[i] = document.body[i].replace("\\SpecialCharNoPassThru " + value, "\\SpecialCharNoPassThru " + key)
+i += 1
+
+
+def convert_specialchar(document):
+"convert special characters to new syntax"
+convert_specialchar_internal(document, True)
+
+
+def revert_specialchar(document):
+"convert special characters to old syntax"
+convert_specialchar_internal(document, False)
+
+
+
 ##
 # Conversion hub
 #
@@ -661,10 +704,12 @@ convert = [
[479, []],
[480, []],
[481, [convert_dashes]],
-   [482, [convert_phrases]]
+   [482, [convert_phrases]],
+   [483, [convert_specialchar]]
   ]
 
 revert =  [
+   [482, [revert_specialchar]],
[481, [revert_phrases]],
[480, [revert_dashes]],
[479, [revert_question_env]],
diff --git a/src/insets/InsetSpecialChar.cpp b/src/insets/InsetSpecialChar.cpp
index 6d967bf..c3ef2ec 100644
--- a/src/insets/InsetSpecialChar.cpp
+++ b/src/insets/InsetSpecialChar.cpp
@@ -283,37 +283,37 @@ void InsetSpecialChar::write(ostream & os) const
 	string command;
 	switch (kind_) {
 	case HYPHENATION:
-		command = "\\-";
+		command = "hyphen";
 		break;
 	case LIGATURE_BREAK:
-		command = "\\textcompwordmark{}";
+		command = "ligaturebreak";
 		break;
 	case END_OF_SENTENCE:
-		command = "\\@.";
+		command = "endofsentence";
 		break;
 	case LDOTS:
-		command = "\\ldots{}";
+		command = "ldots";
 		break;
 	case MENU_SEPARATOR:
-		command = "\\menuseparator";
+		command = "menuseparator";
 		break;
 	case SLASH:
-		command = "\\slash{}";
+		command = "slash";
 		break;
 	case NOBREAKDASH:
-		command = "\\nobreakdash-";
+		command = "nobreakdash";
 		break;
 	case PHRASE_LYX:
-		command = "\\LyX";
+		command = "LyX";
 		break;
 	case PHRASE_TEX:
-		command = "\\TeX";
+		command = "TeX";
 		break;
 	case PHRASE_LATEX2E:
-		command = "\\LaTeX2e";
+		command = "LaTeX2e";
 		break;
 	case PHRASE_LATEX:
-		command = "\\LaTeX";
+		command = "LaTeX";
 		break;
 	}
 	os << "\\SpecialChar " << command << "\n";
@@ -326,27 +326,27 @@ void InsetSpecialChar::read(Lexer & lex)
 	lex.next();
 	string const command = lex.getString();
 
-	if (command == "\\-")
+	if (command == "hyphen")
 		kind_ = HYPHENATION;
-	else if 

Re: new compiler warnings in master

2015-03-22 Thread Jean-Marc Lasgouttes

Le 22/03/15 18:27, Georg Baum a écrit :

Georg Baum wrote:


OK, I will change it. But since the LyX file format really has nothing to
do with LaTeX (except for math), I will remove all backslashes and {}
pairs from InsetSpecialChar. BTW, LaTeX2E is not the only one which is
different.


This would look like the attached. Before I commit: Any wishes for other
names?


Fine with me.

JMarc



Re: Re: new compiler warnings in master

2015-03-21 Thread José Matos
On Friday 20 March 2015 21:57:43 Georg Baum wrote:
 OK, I will change it. But since the LyX file format really has nothing to do 
 with LaTeX (except for math), I will remove all backslashes and {} pairs 
 from InsetSpecialChar. BTW, LaTeX2E is not the only one which is different.
 
 
 Georg

+1

:-)
-- 
José Abílio


Re: Re: new compiler warnings in master

2015-03-21 Thread José Matos
On Friday 20 March 2015 21:57:43 Georg Baum wrote:
> OK, I will change it. But since the LyX file format really has nothing to do 
> with LaTeX (except for math), I will remove all backslashes and {} pairs 
> from InsetSpecialChar. BTW, LaTeX2E is not the only one which is different.
> 
> 
> Georg

+1

:-)
-- 
José Abílio


Re: new compiler warnings in master

2015-03-20 Thread Georg Baum
Jean-Marc Lasgouttes wrote:

 Le 18/03/15 20:27, Georg Baum a écrit :
 Oops! Thanks for the fix.

 Why do you think that the LyX file format needs to follow the LaTeX
 restrictions? IMHO it does not need to match (therefore I did not include
 the {} in the LyX file format, which is done for other special
 characters).
 
 Well, it really looks like it (why the backslashes otherwise?). I'd say
 that we shall either be completely different, or either identical.

OK, I will change it. But since the LyX file format really has nothing to do 
with LaTeX (except for math), I will remove all backslashes and {} pairs 
from InsetSpecialChar. BTW, LaTeX2E is not the only one which is different.


Georg



Re: new compiler warnings in master

2015-03-20 Thread Georg Baum
Jean-Marc Lasgouttes wrote:

> Le 18/03/15 20:27, Georg Baum a écrit :
>> Oops! Thanks for the fix.
>>
>> Why do you think that the LyX file format needs to follow the LaTeX
>> restrictions? IMHO it does not need to match (therefore I did not include
>> the {} in the LyX file format, which is done for other special
>> characters).
> 
> Well, it really looks like it (why the backslashes otherwise?). I'd say
> that we shall either be completely different, or either identical.

OK, I will change it. But since the LyX file format really has nothing to do 
with LaTeX (except for math), I will remove all backslashes and {} pairs 
from InsetSpecialChar. BTW, LaTeX2E is not the only one which is different.


Georg



Re: new compiler warnings in master

2015-03-18 Thread Jean-Marc Lasgouttes

Le 18/03/2015 08:30, Uwe Stöhr a écrit :

Within the last 2 weeks a commit lead to these new compiler warnings:
(The same that I constantly report here. I am wondering why the appear
so often with recent commits.)

It's my fault. I will fix this.

JMarc


   ..\..\..\src\insets\InsetSpecialChar.cpp(57): warning C4244: '=':
convesion from 'double' to 'int', possible dataloss
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]

   ..\..\..\src\insets\InsetSpecialChar.cpp(63): warning C4244: '=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(69): warning C4244: '=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(76): warning C4244: '=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(149): warning C4244: '-=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(150): warning C4244:
'Argument': Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(151): warning C4244: '-=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(161): warning C4244: '-=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(162): warning C4244:
'Argument': Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(163): warning C4244: '-=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(174): warning C4244: '+=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(176): warning C4244:
'Argument': Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(193): warning C4244: '-=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(196): warning C4244:
'Argument': Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(197): warning C4244: '-=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]

regards Uwe




Re: new compiler warnings in master

2015-03-18 Thread Jean-Marc Lasgouttes

Le 18/03/2015 08:30, Uwe Stöhr a écrit :

Within the last 2 weeks a commit lead to these new compiler warnings:
(The same that I constantly report here. I am wondering why the appear
so often with recent commits.)


The warnings should be gone now. Please confirm.

THis uses integer arithmetic now and therefore Abdel's port to Raspberry 
should become snappier :)


Georg, note that there was an error for LaTeX 2e (\LaTeX2e instead of 
\LaTeXe). I fixed it in LaTeX output, but I suspect that we should 
change the file format too for consistency sake.


JMarc



Re: new compiler warnings in master

2015-03-18 Thread Georg Baum
Jean-Marc Lasgouttes wrote:

 Georg, note that there was an error for LaTeX 2e (\LaTeX2e instead of
 \LaTeXe). I fixed it in LaTeX output, but I suspect that we should
 change the file format too for consistency sake.

Oops! Thanks for the fix.

Why do you think that the LyX file format needs to follow the LaTeX 
restrictions? IMHO it does not need to match (therefore I did not include 
the {} in the LyX file format, which is done for other special characters).


Georg



new compiler warnings in master

2015-03-18 Thread Uwe Stöhr

Within the last 2 weeks a commit lead to these new compiler warnings:
(The same that I constantly report here. I am wondering why the appear 
so often with recent commits.)


  ..\..\..\src\insets\InsetSpecialChar.cpp(57): warning C4244: '=': 
convesion from 'double' to 'int', possible dataloss 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]


  ..\..\..\src\insets\InsetSpecialChar.cpp(63): warning C4244: '=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(69): warning C4244: '=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(76): warning C4244: '=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(149): warning C4244: '-=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(150): warning C4244: 
'Argument': Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(151): warning C4244: '-=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(161): warning C4244: '-=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(162): warning C4244: 
'Argument': Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(163): warning C4244: '-=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(174): warning C4244: '+=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(176): warning C4244: 
'Argument': Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(193): warning C4244: '-=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(196): warning C4244: 
'Argument': Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(197): warning C4244: '-=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]


regards Uwe


Re: new compiler warnings in master

2015-03-18 Thread Jean-Marc Lasgouttes

Le 18/03/15 20:27, Georg Baum a écrit :

Oops! Thanks for the fix.

Why do you think that the LyX file format needs to follow the LaTeX
restrictions? IMHO it does not need to match (therefore I did not include
the {} in the LyX file format, which is done for other special characters).


Well, it really looks like it (why the backslashes otherwise?). I'd say 
that we shall either be completely different, or either identical.


JMarc



Re: new compiler warnings in master

2015-03-18 Thread Abdelrazak Younes

On 18/03/2015 11:39, Jean-Marc Lasgouttes wrote:

Le 18/03/2015 08:30, Uwe Stöhr a écrit :

Within the last 2 weeks a commit lead to these new compiler warnings:
(The same that I constantly report here. I am wondering why the appear
so often with recent commits.)


The warnings should be gone now. Please confirm.

THis uses integer arithmetic now and therefore Abdel's port to 
Raspberry should become snappier :)


Well, I hope that, by the time I come back to LyX development, Raspberry 
will not support already 128 floating point arithmetic :-)


Cheers,
Abdel.



new compiler warnings in master

2015-03-18 Thread Uwe Stöhr

Within the last 2 weeks a commit lead to these new compiler warnings:
(The same that I constantly report here. I am wondering why the appear 
so often with recent commits.)


  ..\..\..\src\insets\InsetSpecialChar.cpp(57): warning C4244: '=': 
convesion from 'double' to 'int', possible dataloss 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]


  ..\..\..\src\insets\InsetSpecialChar.cpp(63): warning C4244: '=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(69): warning C4244: '=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(76): warning C4244: '=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(149): warning C4244: '-=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(150): warning C4244: 
'Argument': Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(151): warning C4244: '-=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(161): warning C4244: '-=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(162): warning C4244: 
'Argument': Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(163): warning C4244: '-=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(174): warning C4244: '+=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(176): warning C4244: 
'Argument': Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(193): warning C4244: '-=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(196): warning C4244: 
'Argument': Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
  ..\..\..\src\insets\InsetSpecialChar.cpp(197): warning C4244: '-=': 
Konvertierung von 'double' in 'int', möglicher Datenverlust 
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]


regards Uwe


Re: new compiler warnings in master

2015-03-18 Thread Jean-Marc Lasgouttes

Le 18/03/2015 08:30, Uwe Stöhr a écrit :

Within the last 2 weeks a commit lead to these new compiler warnings:
(The same that I constantly report here. I am wondering why the appear
so often with recent commits.)

It's my fault. I will fix this.

JMarc


   ..\..\..\src\insets\InsetSpecialChar.cpp(57): warning C4244: '=':
convesion from 'double' to 'int', possible dataloss
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]

   ..\..\..\src\insets\InsetSpecialChar.cpp(63): warning C4244: '=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(69): warning C4244: '=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(76): warning C4244: '=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(149): warning C4244: '-=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(150): warning C4244:
'Argument': Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(151): warning C4244: '-=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(161): warning C4244: '-=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(162): warning C4244:
'Argument': Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(163): warning C4244: '-=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(174): warning C4244: '+=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(176): warning C4244:
'Argument': Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(193): warning C4244: '-=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(196): warning C4244:
'Argument': Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]
   ..\..\..\src\insets\InsetSpecialChar.cpp(197): warning C4244: '-=':
Konvertierung von 'double' in 'int', möglicher Datenverlust
[C:\LyXGit\Master\compile-result\src\insets\insets.vcxproj]

regards Uwe




Re: new compiler warnings in master

2015-03-18 Thread Jean-Marc Lasgouttes

Le 18/03/2015 08:30, Uwe Stöhr a écrit :

Within the last 2 weeks a commit lead to these new compiler warnings:
(The same that I constantly report here. I am wondering why the appear
so often with recent commits.)


The warnings should be gone now. Please confirm.

THis uses integer arithmetic now and therefore Abdel's port to Raspberry 
should become snappier :)


Georg, note that there was an error for LaTeX 2e (\LaTeX2e instead of 
\LaTeXe). I fixed it in LaTeX output, but I suspect that we should 
change the file format too for consistency sake.


JMarc



Re: new compiler warnings in master

2015-03-18 Thread Georg Baum
Jean-Marc Lasgouttes wrote:

> Georg, note that there was an error for LaTeX 2e (\LaTeX2e instead of
> \LaTeXe). I fixed it in LaTeX output, but I suspect that we should
> change the file format too for consistency sake.

Oops! Thanks for the fix.

Why do you think that the LyX file format needs to follow the LaTeX 
restrictions? IMHO it does not need to match (therefore I did not include 
the {} in the LyX file format, which is done for other special characters).


Georg



Re: new compiler warnings in master

2015-03-18 Thread Jean-Marc Lasgouttes

Le 18/03/15 20:27, Georg Baum a écrit :

Oops! Thanks for the fix.

Why do you think that the LyX file format needs to follow the LaTeX
restrictions? IMHO it does not need to match (therefore I did not include
the {} in the LyX file format, which is done for other special characters).


Well, it really looks like it (why the backslashes otherwise?). I'd say 
that we shall either be completely different, or either identical.


JMarc



Re: new compiler warnings in master

2015-03-18 Thread Abdelrazak Younes

On 18/03/2015 11:39, Jean-Marc Lasgouttes wrote:

Le 18/03/2015 08:30, Uwe Stöhr a écrit :

Within the last 2 weeks a commit lead to these new compiler warnings:
(The same that I constantly report here. I am wondering why the appear
so often with recent commits.)


The warnings should be gone now. Please confirm.

THis uses integer arithmetic now and therefore Abdel's port to 
Raspberry should become snappier :)


Well, I hope that, by the time I come back to LyX development, Raspberry 
will not support already 128 floating point arithmetic :-)


Cheers,
Abdel.