summaryrefslogtreecommitdiffhomepage
AgeCommit message (Collapse)Author
2019-06-25Merge pull request #4517 from ↵Yukihiro "Matz" Matsumoto
shuujii/remove-unneeded-statement-in-SET_OBJ_VALUE-with-boxing_word.h Remove unneeded statement in `SET_OBJ_VALUE` with `boxing_word.h`
2019-06-25Merge pull request #4528 from shuujii/fix-argument-specs-to-ArrayYukihiro "Matz" Matsumoto
Fix argument specs to `Array`
2019-06-25Merge pull request #4530 from ↵Yukihiro "Matz" Matsumoto
shuujii/compare-obj-pointer-directly-instead-of-using-mrb_obj_eq-in-mrb_gc_unregister Compare obj pointer directly instead of using mrb_obj_eq in mrb_gc_unregister
2019-06-25Fixed `mrb_iv_remove` with immediate objects; fix #4519Yukihiro "Matz" Matsumoto
The #4520 tried to address the issue, but it changes the type of `mrb_check_frozen` argument; close #4520
2019-06-25Fixed a bug caused by `to_s` that returns `nil`; fix 4504Yukihiro "Matz" Matsumoto
2019-06-25Fix `mrb_str_to_str()` to handle symbols.Yukihiro "Matz" Matsumoto
2019-06-25Renamed `stacked` to `onstack`; ref #4523Yukihiro "Matz" Matsumoto
2019-06-25Unify loops to minimize bytecode sizeRyan Lopopolo
2019-06-24Compare obj pointer directly instead of using mrb_obj_eq in mrb_gc_unregisterKOBAYASHI Shuji
Because immediate values are not registered.
2019-06-23Fix argument specs to `Array`KOBAYASHI Shuji
2019-06-23Use explicit block parameterRyan Lopopolo
2019-06-23Optimize String#each_lineRyan Lopopolo
2019-06-23Merge pull request #4526 from shuujii/refine-Hash-rehash-exampleYukihiro "Matz" Matsumoto
Refine `Hash#rehash` example [ci skip]
2019-06-23Merge pull request #4523 from dearblue/use-stackYukihiro "Matz" Matsumoto
Use stack memory for small name of Struct members
2019-06-23Merge pull request #4525 from dearblue/utf8len-overflowYukihiro "Matz" Matsumoto
Fix potential overflow in `utf8len()`
2019-06-22Fix the unnecessary `mrb_str_modify()` calldearblue
Now to be calls `mrb_str_modify()` only once when 2 or more characters.
2019-06-22Delete the unnecessary block brace in `mrb_str_reverse_bang`dearblue
2019-06-22Fix string brakes for one UTF-8 charactordearblue
2019-06-22Add test for one UTF-8 charactordearblue
2019-06-22Change to UTF-8 string reversing with in placedearblue
Reverses UTF-8 strings without allocated heap for working memory. 1. String before reversing: ``` "!yburmの界世" # byte unit [33, 121, 98, 117, 114, 109, 227, 129, 174, 231, 149, 140, 228, 184, 150] ``` 2. Reverse the byte order of each character: ``` [33, 121, 98, 117, 114, 109, 174, 129, 227, 140, 149, 231, 150, 184, 228] ``` 3. Reverse the whole byte order and complete: ``` [228, 184, 150, 231, 149, 140, 227, 129, 174, 109, 114, 117, 98, 121, 33] # string "世界のmruby!" ```
2019-06-22Replacement to function for string reversingdearblue
2019-06-22Speed up base case by 2xRyan Lopopolo
Make non-paragraph mode twice as fast. Performance is within a factor of 2 of the original implementation.
2019-06-22Add paragraph mode to String#each_line in mrblibRyan Lopopolo
mruby/mruby#4511 demonstrated an infinite loop in `String#each_line` when given an empty string separator. In MRI, an empty separator places String#each_line in paragraph mode, where the String is separated on successive runs of newlines. In paragraph mode, the String `"abc\n\n\ndef\nxyz"` is split into `["abc\n\n\n", "def\nxyz"]`. This commit makes the String#each_line implementation as close to ruby/spec compliant as possible given the limitations of mruby core. With this patch, the following specs fail for `String#each_line`: - uses `$/` as the separator when none is given (can be fixed by aliasing and redefining the method to use $/ as the default value of separator in mruby-io) - when no block is given returned Enumerator size should return nil (`Enumerator#size` is not supported on mruby) - tries to convert the separator to a string using to_str (`String#to_str` is not implemented on mruby) This patch has similar memory consumption compared to the prior implementation and is takes 4x the time the prior implementation takes to execute: ```console /usr/bin/time -l ./bin/mruby -e '("aaa\n\nbbbbb\n\n\n\n\ncccc" * 100000).each_line("\n") { }'; ```
2019-06-22Refine `Hash#rehash` example [ci skip]KOBAYASHI Shuji
Previous example doesn't work because string key (frozen) can't be modified.
2019-06-22Fix potential overflow in `utf8len()`dearblue
For example on 32 bit mode, when `p = 0xfffffffd`, `e = 0xfffffffe` and `len = 4`, the sum of `p` and `len` can be to `1`, and comparison with `e` will to be false. As a result, a segmentation fault occurs by referring to address 0.
2019-06-22Merge pull request #4518 from ↵Yukihiro "Matz" Matsumoto
shuujii/add-ISO-section-number-to-Kernel-local_variables Add ISO section number to `Kernel.#local_variables` [ci skip]
2019-06-22Use stack memory for small name of Struct membersdearblue
2019-06-22Merge pull request #4521 from ↵Yukihiro "Matz" Matsumoto
shuujii/move-Kernel-__send__-test-to-core-from-mruby-metaprog Move `Kernel#__send__` test to core from `mruby-metaprog`
2019-06-21Move `Kernel#__send__` test to core from `mruby-metaprog`KOBAYASHI Shuji
2019-06-20Add ISO section number to `Kernel.#local_variables` [ci skip]KOBAYASHI Shuji
2019-06-19Remove unneeded statement in `SET_OBJ_VALUE` with `boxing_word.h`KOBAYASHI Shuji
`(r).value.bp` and `v` have the same value due to assignment of the line preceding the removed line.
2019-06-19Merge pull request #4510 from shuujii/remove-unneeded-mrb_str_dup-in-Module-nameYukihiro "Matz" Matsumoto
Remove unneeded `mrb_str_dup()` in `Module#name`
2019-06-19Merge pull request #4514 from mimaki/fix-error.hYukihiro "Matz" Matsumoto
Fix path of `error.h`.
2019-06-18Fix `struct RRange` overflow on 32-bit CPU with `MRB_NAN_BOXING`KOBAYASHI Shuji
2019-06-18Fix path of `error.h`.Hiroshi Mimaki
2019-06-17Support parsing a Regexp literal with 'o' optionRyan Lopopolo
CRuby supports a 'o' `Regexp` option for `Regexp` literals, e.g. `/foo/o` that disables multiple interpolation passes. This commit adds support for parsing such literals. Nothing is done with the option, it is consumed and ignored by the parser.
2019-06-17Remove unneeded `mrb_str_dup()` in `Module#name`KOBAYASHI Shuji
`mrb_class_path()` always returns a new string or `nil`.
2019-06-17Merge pull request #4508 from ↵Yukihiro "Matz" Matsumoto
shuujii/fix-cvar-ivar-const-and-method-can-be-removed-to-frozen-object Fix cvar, ivar, const and method can be removed to frozen object
2019-06-17Merge pull request #4507 from shuujii/fix-index-in-error-message-of-Struct-arefYukihiro "Matz" Matsumoto
Fix index in error message of `Struct#[]`
2019-06-17Fixed indentation in parse.yYukihiro "Matz" Matsumoto
2019-06-17Allow newlines and comments between method calls.Yukihiro "Matz" Matsumoto
2019-06-17Support `&.` at the beginning of the line.Yukihiro "Matz" Matsumoto
2019-06-17Fixed wrong behavior on `..` at the beginning of the line.Yukihiro "Matz" Matsumoto
2019-06-17Merge pull request #4502 from ↵Yukihiro "Matz" Matsumoto
shuujii/adjust-allocation-size-in-mrb_id_attrset-mruby-struct Adjust allocation size in `mrb_id_attrset()` (`mruby-struct`)
2019-06-16Fix cvar, ivar, const and method can be removed to frozen objectKOBAYASHI Shuji
2019-06-15Fix index in error message of `Struct#[]`KOBAYASHI Shuji
Before this patch: $ bin/mruby -e 'Struct.new(:a,:b).new[-3]' #=> offset -1 too small for struct(size:2) (IndexError) After this patch (same as Ruby): $ bin/mruby -e 'Struct.new(:a,:b).new[-3]' #=> offset -3 too small for struct(size:2) (IndexError)
2019-06-14Merge pull request #4506 from ↵Yukihiro "Matz" Matsumoto
shuujii/remove-a-meaningless-branch-condition-in-mruby-struct Remove a meaningless branch condition in `mruby-struct`
2019-06-14Remove a meaningless branch condition in `mruby-struct`KOBAYASHI Shuji
The following branch condition is always true: // mrbgems/mruby-struct/src/struct.c:187 in make_struct_define_accessors() if (is_local_id(mrb, name) || is_const_id(mrb, name)) {
2019-06-14Merge pull request #4505 from shuujii/fix-class-name-validation-in-Struct.newYukihiro "Matz" Matsumoto
Fix class name validation in `Struct.new`
2019-06-13Fix class name validation in `Struct.new`KOBAYASHI Shuji
Before this patch: $ bin/mruby -e 'p Struct.new("A-")' #=> Struct::"A-" After this patch: $ bin/mruby -e 'p Struct.new("A-")' #=> NameError: identifier A- needs to be constant