Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(76)

Side by Side Diff: third_party/sfntly/src/subsetter/subsetter_impl.cc

Issue 7572039: Fix compile warning. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Created 9 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2011 Google Inc. All Rights Reserved. 2 * Copyright 2011 Google Inc. All Rights Reserved.
3 * 3 *
4 * Licensed under the Apache License, Version 2.0 (the "License"); 4 * Licensed under the Apache License, Version 2.0 (the "License");
5 * you may not use this file except in compliance with the License. 5 * you may not use this file except in compliance with the License.
6 * You may obtain a copy of the License at 6 * You may obtain a copy of the License at
7 * 7 *
8 * http://www.apache.org/licenses/LICENSE-2.0 8 * http://www.apache.org/licenses/LICENSE-2.0
9 * 9 *
10 * Unless required by applicable law or agreed to in writing, software 10 * Unless required by applicable law or agreed to in writing, software
(...skipping 236 matching lines...) Expand 10 before | Expand all | Expand 10 after
247 } 247 }
248 } 248 }
249 } 249 }
250 250
251 glyph_id_processed->insert(*i); 251 glyph_id_processed->insert(*i);
252 } 252 }
253 253
254 glyph_id_remaining.clear(); 254 glyph_id_remaining.clear();
255 glyph_id_remaining = comp_glyph_id; 255 glyph_id_remaining = comp_glyph_id;
256 } 256 }
257
258 return true;
257 } 259 }
258 260
259 CALLER_ATTACH Font* SubsetterImpl::Subset(const IntegerSet& glyph_ids) { 261 CALLER_ATTACH Font* SubsetterImpl::Subset(const IntegerSet& glyph_ids) {
260 // The tables are already checked in ResolveCompositeGlyphs(). 262 // The tables are already checked in ResolveCompositeGlyphs().
261 GlyphTablePtr glyph_table = 263 GlyphTablePtr glyph_table =
262 down_cast<GlyphTable*>(font_->GetTable(Tag::glyf)); 264 down_cast<GlyphTable*>(font_->GetTable(Tag::glyf));
263 LocaTablePtr loca_table = down_cast<LocaTable*>(font_->GetTable(Tag::loca)); 265 LocaTablePtr loca_table = down_cast<LocaTable*>(font_->GetTable(Tag::loca));
264 266
265 // Setup font builders we need. 267 // Setup font builders we need.
266 FontBuilderPtr font_builder; 268 FontBuilderPtr font_builder;
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
326 TableBuilderPtr dereference; 328 TableBuilderPtr dereference;
327 dereference.Attach( 329 dereference.Attach(
328 font_builder->NewTableBuilder(i->first, i->second->ReadFontData())); 330 font_builder->NewTableBuilder(i->first, i->second->ReadFontData()));
329 } 331 }
330 } 332 }
331 333
332 return font_builder->Build(); 334 return font_builder->Build();
333 } 335 }
334 336
335 } // namespace sfntly 337 } // namespace sfntly
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698