Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 8537f1e

Browse files
dmaclachthomasvl
authored andcommittedNov 16, 2017
Fix up warnings from Xcode 9.1 (protocolbuffers#3887)
Fix up warnings from Xcode 9.1
1 parent 0d46688 commit 8537f1e

25 files changed

+78
-52
lines changed
 

‎Makefile.am

+14-9
Original file line numberDiff line numberDiff line change
@@ -558,15 +558,20 @@ objectivec_EXTRA_DIST= \
558558
objectivec/Tests/GPBWireFormatTests.m \
559559
objectivec/Tests/iOSTestHarness/AppDelegate.m \
560560
objectivec/Tests/iOSTestHarness/en.lproj/InfoPlist.strings \
561-
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/Contents.json \
562-
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/iPad6.png \
563-
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/iPad6_2x.png \
564-
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/iPad7.png \
565-
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/iPad7_2x.png \
566-
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/iPhone6.png \
567-
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/iPhone6_2x.png \
568-
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/iPhone7_2x.png \
569-
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/iPhone7_3x.png \
561+
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/appicon_20.png \
562+
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/appicon_29.png \
563+
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/appicon_40.png \
564+
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/appicon_58.png \
565+
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/appicon_60.png \
566+
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/appicon_76.png \
567+
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/appicon_80.png \
568+
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/appicon_87.png \
569+
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/appicon_120.png \
570+
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/appicon_152.png \
571+
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/appicon_167.png \
572+
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/appicon_180.png \
573+
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/appicon_1024.png \
574+
objectivec/Tests/iOSTestHarness/Images.xcassets/AppIcon.appiconset/Contents.json \
570575
objectivec/Tests/iOSTestHarness/Images.xcassets/LaunchImage.launchimage/Contents.json \
571576
objectivec/Tests/iOSTestHarness/Info.plist \
572577
objectivec/Tests/iOSTestHarness/LaunchScreen.xib \

‎objectivec/ProtocolBuffers_iOS.xcodeproj/project.pbxproj

-2
Original file line numberDiff line numberDiff line change
@@ -880,7 +880,6 @@
880880
buildSettings = {
881881
CLANG_ENABLE_MODULES = YES;
882882
FRAMEWORK_SEARCH_PATHS = (
883-
"\"$(SDKROOT)/Developer/Library/Frameworks\"",
884883
"\"$(DEVELOPER_LIBRARY_DIR)/Frameworks\"",
885884
"$(inherited)",
886885
);
@@ -916,7 +915,6 @@
916915
buildSettings = {
917916
CLANG_ENABLE_MODULES = YES;
918917
FRAMEWORK_SEARCH_PATHS = (
919-
"\"$(SDKROOT)/Developer/Library/Frameworks\"",
920918
"\"$(DEVELOPER_LIBRARY_DIR)/Frameworks\"",
921919
"$(inherited)",
922920
);
Original file line numberDiff line numberDiff line change
@@ -1,72 +1,103 @@
11
{
22
"images" : [
33
{
4+
"size" : "20x20",
45
"idiom" : "iphone",
6+
"filename" : "appicon_40.png",
7+
"scale" : "2x"
8+
},
9+
{
10+
"size" : "20x20",
11+
"idiom" : "iphone",
12+
"filename" : "appicon_60.png",
13+
"scale" : "3x"
14+
},
15+
{
516
"size" : "29x29",
17+
"idiom" : "iphone",
18+
"filename" : "appicon_29.png",
619
"scale" : "1x"
720
},
821
{
9-
"idiom" : "iphone",
1022
"size" : "29x29",
23+
"idiom" : "iphone",
24+
"filename" : "appicon_58.png",
1125
"scale" : "2x"
1226
},
1327
{
14-
"idiom" : "iphone",
1528
"size" : "29x29",
29+
"idiom" : "iphone",
30+
"filename" : "appicon_87.png",
1631
"scale" : "3x"
1732
},
1833
{
19-
"idiom" : "iphone",
2034
"size" : "40x40",
35+
"idiom" : "iphone",
36+
"filename" : "appicon_80.png",
2137
"scale" : "2x"
2238
},
2339
{
24-
"idiom" : "iphone",
2540
"size" : "40x40",
41+
"idiom" : "iphone",
42+
"filename" : "appicon_120.png",
2643
"scale" : "3x"
2744
},
2845
{
29-
"size" : "57x57",
3046
"idiom" : "iphone",
31-
"filename" : "iPhone6.png",
47+
"size" : "57x57",
3248
"scale" : "1x"
3349
},
3450
{
35-
"size" : "57x57",
3651
"idiom" : "iphone",
37-
"filename" : "iPhone6_2x.png",
52+
"size" : "57x57",
3853
"scale" : "2x"
3954
},
4055
{
4156
"size" : "60x60",
4257
"idiom" : "iphone",
43-
"filename" : "iPhone7_2x.png",
58+
"filename" : "appicon_120.png",
4459
"scale" : "2x"
4560
},
4661
{
4762
"size" : "60x60",
4863
"idiom" : "iphone",
49-
"filename" : "iPhone7_3x.png",
64+
"filename" : "appicon_180.png",
5065
"scale" : "3x"
5166
},
5267
{
68+
"size" : "20x20",
5369
"idiom" : "ipad",
54-
"size" : "29x29",
70+
"filename" : "appicon_20.png",
5571
"scale" : "1x"
5672
},
5773
{
74+
"size" : "20x20",
5875
"idiom" : "ipad",
59-
"size" : "29x29",
76+
"filename" : "appicon_40.png",
6077
"scale" : "2x"
6178
},
6279
{
80+
"size" : "29x29",
6381
"idiom" : "ipad",
64-
"size" : "40x40",
82+
"filename" : "appicon_29.png",
6583
"scale" : "1x"
6684
},
6785
{
86+
"size" : "29x29",
6887
"idiom" : "ipad",
88+
"filename" : "appicon_58.png",
89+
"scale" : "2x"
90+
},
91+
{
6992
"size" : "40x40",
93+
"idiom" : "ipad",
94+
"filename" : "appicon_40.png",
95+
"scale" : "1x"
96+
},
97+
{
98+
"size" : "40x40",
99+
"idiom" : "ipad",
100+
"filename" : "appicon_80.png",
70101
"scale" : "2x"
71102
},
72103
{
@@ -80,37 +111,42 @@
80111
"scale" : "2x"
81112
},
82113
{
83-
"size" : "72x72",
84114
"idiom" : "ipad",
85-
"filename" : "iPad6.png",
115+
"size" : "72x72",
86116
"scale" : "1x"
87117
},
88118
{
89-
"size" : "72x72",
90119
"idiom" : "ipad",
91-
"filename" : "iPad6_2x.png",
120+
"size" : "72x72",
92121
"scale" : "2x"
93122
},
94123
{
95124
"size" : "76x76",
96125
"idiom" : "ipad",
97-
"filename" : "iPad7.png",
126+
"filename" : "appicon_76.png",
98127
"scale" : "1x"
99128
},
100129
{
101130
"size" : "76x76",
102131
"idiom" : "ipad",
103-
"filename" : "iPad7_2x.png",
132+
"filename" : "appicon_152.png",
133+
"scale" : "2x"
134+
},
135+
{
136+
"size" : "83.5x83.5",
137+
"idiom" : "ipad",
138+
"filename" : "appicon_167.png",
104139
"scale" : "2x"
105140
},
106141
{
107-
"idiom" : "car",
108-
"size" : "120x120",
142+
"size" : "1024x1024",
143+
"idiom" : "ios-marketing",
144+
"filename" : "appicon_1024.png",
109145
"scale" : "1x"
110146
}
111147
],
112148
"info" : {
113149
"version" : 1,
114150
"author" : "xcode"
115151
}
116-
}
152+
}
Loading
Loading
Loading
Loading
Loading
Loading
Loading
Loading
Loading
Loading
Loading
Loading
Loading
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.

‎objectivec/Tests/iOSTestHarness/Images.xcassets/LaunchImage.launchimage/Contents.json

+6-19
Original file line numberDiff line numberDiff line change
@@ -3,43 +3,30 @@
33
{
44
"orientation" : "portrait",
55
"idiom" : "ipad",
6-
"minimum-system-version" : "7.0",
76
"extent" : "full-screen",
8-
"scale" : "2x"
9-
},
10-
{
11-
"orientation" : "landscape",
12-
"idiom" : "ipad",
137
"minimum-system-version" : "7.0",
14-
"extent" : "full-screen",
158
"scale" : "1x"
169
},
1710
{
1811
"orientation" : "landscape",
1912
"idiom" : "ipad",
20-
"minimum-system-version" : "7.0",
2113
"extent" : "full-screen",
22-
"scale" : "2x"
23-
},
24-
{
25-
"orientation" : "portrait",
26-
"idiom" : "iphone",
2714
"minimum-system-version" : "7.0",
28-
"scale" : "2x"
15+
"scale" : "1x"
2916
},
3017
{
3118
"orientation" : "portrait",
32-
"idiom" : "iphone",
19+
"idiom" : "ipad",
20+
"extent" : "full-screen",
3321
"minimum-system-version" : "7.0",
34-
"subtype" : "retina4",
3522
"scale" : "2x"
3623
},
3724
{
38-
"orientation" : "portrait",
25+
"orientation" : "landscape",
3926
"idiom" : "ipad",
40-
"minimum-system-version" : "7.0",
4127
"extent" : "full-screen",
42-
"scale" : "1x"
28+
"minimum-system-version" : "7.0",
29+
"scale" : "2x"
4330
}
4431
],
4532
"info" : {

0 commit comments

Comments
 (0)
Please sign in to comment.