Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Assign into the argument binding directly in ↵ | Ms2ger | 2014-05-05 | 1 | -9/+11 |
| | | | | instantiateJSToNativeConversionTemplate. | ||||
* | Move the assignment outside the if when dealing with optional arguments. | Ms2ger | 2014-05-05 | 1 | -6/+6 |
| | |||||
* | Move the assignment of 'None' in the no-argument-passed case into an else ↵ | Ms2ger | 2014-05-05 | 1 | -3/+3 |
| | | | | | | | branch. This is the only case where we assign into an argument local twice, so removing it will allow us to make that binding immutable. | ||||
* | Remove getJSToNativeConversionTemplate's initialValue return value. | Ms2ger | 2014-05-05 | 1 | -13/+8 |
| | | | | This commit does not change the generated code. | ||||
* | Move the handling of default values in getJSToNativeConversionTemplate into ↵ | Ms2ger | 2014-05-05 | 1 | -75/+51 |
| | | | | | | | | | | | | | | handleOptional. This is a first step towards moving responsibility for default values into the caller, which should improve their code (in particular for the dictionary codegen. This also introduces a Dictionary::empty function, whose implementation is a hack. The implementation will be improved once its codegen has access to the raw default values. Besides dictionaries, this commit does not change the generated code. | ||||
* | Use CGIfWrapper in instantiateJSToNativeConversionTemplate. | Ms2ger | 2014-05-05 | 1 | -13/+2 |
| | |||||
* | Remove references to unused substitution variables in ↵ | Ms2ger | 2014-05-05 | 1 | -26/+1 |
| | | | | getJSToNativeConversionTemplate. | ||||
* | auto merge of #2320 : glennw/servo/codegen-warnings, r=jdm | bors-servo | 2014-05-05 | 1 | -1/+1 |
|\ | | | | | | | | | | | Is it reasonable to always expect success here for now? Fixes #2246. | ||||
| * | Prevent unused result warning in codegen. Always expect success for now. | Glenn Watson | 2014-05-05 | 1 | -1/+1 |
| | | |||||
* | | auto merge of #2316 : Ms2ger/servo/owned-str-tests, r=jdm | bors-servo | 2014-05-05 | 2 | -3/+3 |
|\ \ | |||||
| * | | Use str.to_owned() in tests. | Ms2ger | 2014-05-04 | 2 | -3/+3 |
| |/ | |||||
* | | Changes based on review | Matt Murphy | 2014-05-04 | 9 | -17/+17 |
| | | |||||
* | | Convert Vec::new() to vec!() | Matt Murphy | 2014-05-04 | 39 | -134/+133 |
| | | |||||
* | | ~[] to Vec in harness/contenttest/contenttest.rs | Matt Murphy | 2014-05-04 | 1 | -2/+2 |
| | | |||||
* | | ~[] to Vec in net/resource_task.rs | Matt Murphy | 2014-05-04 | 2 | -3/+3 |
| | | |||||
* | | ~[] to Vec in util/cache.rs | Matt Murphy | 2014-05-04 | 1 | -3/+3 |
| | | |||||
* | | ~[] to Vec in style/selectors.rs | Matt Murphy | 2014-05-04 | 1 | -30/+30 |
| | | |||||
* | | ~[] to Vec in util/time.rs | Matt Murphy | 2014-05-04 | 1 | -17/+17 |
| | | |||||
* | | ~[] to Vec in script/script_task.rs | Matt Murphy | 2014-05-04 | 1 | -1/+1 |
| | | |||||
* | | ~[] to Vec in script/script_task.rs | Matt Murphy | 2014-05-04 | 1 | -2/+2 |
| | | |||||
* | | ~[] to Vec in script/html/hubbub_html_parser.rs | Matt Murphy | 2014-05-04 | 1 | -2/+2 |
| | | |||||
* | | ~[] to Vec in style/properties.rs.mako and selector_matching.rs | Matt Murphy | 2014-05-04 | 2 | -10/+10 |
| | | |||||
* | | ~[] to Vec in script/dom/node.rs | Matt Murphy | 2014-05-04 | 1 | -1/+1 |
| | | |||||
* | | ~[] to Vec in gfx/text/text_run.rs | Matt Murphy | 2014-05-04 | 1 | -4/+4 |
| | | |||||
* | | ~[] to Vec in gfx/text/shaping/harfbuzz.rs | Matt Murphy | 2014-05-04 | 2 | -7/+7 |
| | | |||||
* | | ~[] to std::vec::Vec in gfx/text/glyph.rs | Matt Murphy | 2014-05-04 | 1 | -9/+9 |
| | | |||||
* | | ~[] to Vec in gfx/font_list.rs | Matt Murphy | 2014-05-04 | 1 | -2/+2 |
| | | |||||
* | | ~[] to Vec in glut_windowing | Matt Murphy | 2014-05-04 | 1 | -2/+2 |
| | | |||||
* | | ~[] to Vec in glfw_windowing and servo | Matt Murphy | 2014-05-04 | 2 | -3/+3 |
| | | |||||
* | | ~[] to Vec in table, table_colgroup, and text | Matt Murphy | 2014-05-04 | 3 | -14/+13 |
| | | |||||
* | | ~[] to Vec in layout flow, block, table, row, rowgroup, wrapper | Matt Murphy | 2014-05-04 | 6 | -70/+70 |
| | | |||||
* | | ~[] to Vec in main | Matt Murphy | 2014-05-04 | 2 | -3/+2 |
| | | |||||
* | | ~[] to Vec in main/constellation.rs | Matt Murphy | 2014-05-04 | 1 | -16/+16 |
| | | |||||
* | | ~[] to Vec in main/compositing/compositor_layer.rs | Matt Murphy | 2014-05-04 | 1 | -5/+5 |
| | | |||||
* | | ~[] to Vec in main/layout/wrapper.rs | Matt Murphy | 2014-05-04 | 1 | -5/+5 |
| | | |||||
* | | ~[] to Vec in main/layout/layout_task.rs, script/dom/clientrectlist.rs, | Matt Murphy | 2014-05-04 | 1 | -1/+1 |
| | | | | | | | | script/layout_interface.rs | ||||
* | | ~[] to Vec in main/layout/box_.rs, gfx/text/util.rs, main/layout/text.rs | Matt Murphy | 2014-05-04 | 3 | -13/+14 |
| | | |||||
* | | ~[] to Vec in main/layout/construct.rs and associated files | Matt Murphy | 2014-05-04 | 2 | -3/+54 |
| | | |||||
* | | ~[] to std::vec::Vec in layout | Matt Murphy | 2014-05-04 | 2 | -11/+11 |
| | | |||||
* | | ~[] to Vec in main/compositing and associated files | Matt Murphy | 2014-05-04 | 6 | -57/+57 |
| | | |||||
* | | ~[] to Vec in style/media_quieres.rs, selectors.rs, stylesheets.rs | Matt Murphy | 2014-05-04 | 3 | -16/+17 |
| | | |||||
* | | ~[] to Vec in FontStyle and FontGroup | Matt Murphy | 2014-05-04 | 5 | -16/+16 |
|/ | |||||
* | Call find_equiv to avoid allocation strings. | Ms2ger | 2014-05-04 | 1 | -3/+3 |
| | |||||
* | Reorder some comparisons to avoid allocating strings for them. | Ms2ger | 2014-05-04 | 4 | -39/+39 |
| | |||||
* | Replace most ~"string"s with "string".to_owned(). | Ms2ger | 2014-05-04 | 35 | -210/+210 |
| | |||||
* | Replace all ~"" with "".to_owned(). | Ms2ger | 2014-05-03 | 78 | -290/+290 |
| | |||||
* | Address review comments. | Josh Matthews | 2014-05-03 | 83 | -374/+316 |
| | |||||
* | Fix up the virtual method unsoundness. | Josh Matthews | 2014-05-03 | 4 | -24/+26 |
| | |||||
* | Make dictionaries contain Root<T> values instead of JS<T>, ensuring that ↵ | Josh Matthews | 2014-05-03 | 5 | -11/+20 |
| | | | | they will not be collected while the dictionary is alive. | ||||
* | Address review comments. | Josh Matthews | 2014-05-03 | 99 | -380/+388 |
| |