Merge commit 'b9b97ba6526aebbe920e0bd29b8dc8dc35267cfc' into harfbuzz
Conflicts: src/load_map.cpp
This commit is contained in:
commit
c4cdd89076
8 changed files with 37 additions and 0 deletions
|
@ -153,6 +153,14 @@ If you see bits of code around that do not follow these please don't hesitate to
|
|||
// more...
|
||||
}
|
||||
|
||||
#### Prefer `empty()` over `size() == 0` if container supports it
|
||||
|
||||
This avoids implicit conversions to bool and reduces compiler warnings.
|
||||
|
||||
if (container.empty()) // please
|
||||
|
||||
if (container.size() == 0) // no
|
||||
|
||||
|
||||
### Other C++ style resources
|
||||
|
||||
|
|
|
@ -0,0 +1,7 @@
|
|||
x,y,line
|
||||
0,0,"
|
||||
many
|
||||
lines
|
||||
of text
|
||||
with unix newlines
|
||||
"
|
|
|
@ -0,0 +1,2 @@
|
|||
x,y,line
|
||||
0,0,"\r\nmany\r\nlines\r\nof text\r\nwith unix newlines\r\n"
|
|
7
tests/data/csv/mac_newlines_with_unix_inline.csv
Normal file
7
tests/data/csv/mac_newlines_with_unix_inline.csv
Normal file
|
@ -0,0 +1,7 @@
|
|||
x,y,line
|
||||
0,0,"
|
||||
many
|
||||
lines
|
||||
of text
|
||||
with unix newlines
|
||||
"
|
|
2
tests/data/csv/mac_newlines_with_unix_inline_escaped.csv
Normal file
2
tests/data/csv/mac_newlines_with_unix_inline_escaped.csv
Normal file
|
@ -0,0 +1,2 @@
|
|||
x,y,line
|
||||
0,0,"many\n lines\n of text\n with unix newlines"
|
|
7
tests/data/csv/windows_newlines_with_unix_inline.csv
Normal file
7
tests/data/csv/windows_newlines_with_unix_inline.csv
Normal file
|
@ -0,0 +1,7 @@
|
|||
x,y,line
|
||||
0,0,"
|
||||
many
|
||||
lines
|
||||
of text
|
||||
with unix newlines
|
||||
"
|
|
|
@ -0,0 +1,2 @@
|
|||
x,y,line
|
||||
0,0,"many\n lines\n of text\n with unix newlines"
|
|
|
@ -6,6 +6,8 @@ from nose.tools import *
|
|||
from utilities import execution_path
|
||||
|
||||
import os, mapnik
|
||||
# make the tests silent since we intentially test error conditions that are noisy
|
||||
mapnik.logger.set_severity(mapnik.severity_type.None)
|
||||
|
||||
def setup():
|
||||
# All of the paths used are relative, if we run the tests
|
||||
|
|
Loading…
Add table
Reference in a new issue