Parrot 6.1.0 "Black-collared Lovebird" | parrot.org/ | Log: irclog.perlgeek.de/parrot | #parrotsketch meeting Tuesday 19:30 UTC
Set by moderator on 18 February 2014.
00:12 kid51 joined 02:14 FROGGS__ joined 02:31 Chirag joined 02:32 Chirag_ joined 03:49 cooper joined 05:46 Chirag joined, Chirag_ joined 06:50 Hunger joined 07:51 FROGGS joined 14:08 User joined
dalek rrot: b47b476 | rurban++ | ChangeLog:
ChangeLog: add GH #1061 warning (-O1 fails in nqp)
14:40
rrot/rurban/morph-gh372: 2b41d35 | rurban++ | src/pmc/fixedpmcarray.pmc:
morph: add morph methods to undef and fixedpmcarray

fixes the test to morph fixedpmcarray to integer t/src/extend_vtable.t 9
t/pmc/undef.t: 10 still broken: String morph to Undef, get_class Undef is a PMCProxy
14:52
14:58 stryx` joined 15:03 travis-ci joined
travis-ci [travis-ci] parrot/parrot#992 (master - b47b476 : Reini Urban): The build passed. 15:03
[travis-ci] Change view : github.com/parrot/parrot/compare/8...7b47689f0c
[travis-ci] Build details : travis-ci.org/parrot/parrot/builds/20458640
15:03 travis-ci left
dalek rrot: dfe541d | rurban++ | / (2 files):
[imcc] fix GH #1061, nqp -O2: set_addr label marks a basic_block

set_addr label does mark a basic_block, dead_code_remove() needs the label. disable old premature optimization
Fixes nqp/#160 All nqp tests pass now with parrot -O2.
15:33
15:43 denis_boyun joined 16:03 travis-ci joined
travis-ci [travis-ci] parrot/parrot#993 (master - dfe541d : Reini Urban): The build passed. 16:03
[travis-ci] Change view : github.com/parrot/parrot/compare/b...e541df0f1b
[travis-ci] Build details : travis-ci.org/parrot/parrot/builds/20462590
16:03 travis-ci left 16:13 benabik joined
dalek rrot/rurban/morph-gh372: 0e72e0f | rurban++ | / (6 files):
[core] WIP fix morph [GH #372]

pass the correct type info to pmc_reuse allow the 2nd argument to be a Object PMC, so use Parrot_pmc_reuse_init instead.
Define some FixedPMCArray morphs.
TODO:
  - get_class Undef returns a PMCProxy object
  - More basic arrays, Integer, String and Hash morphes
  - non-builtin types via PARROT_CLASS(type)->id
   and Parrot_pmc_reuse_by_class
  - morph old values into new type (pmc specific).
   e.g. *Array => Integer needs to assign the elements to the get_integer iv
16:37
rrot/rurban/morph-gh372: 70f6245 | rurban++ | src/pmc/fixedpmcarray.pmc:
morph: add morph methods to undef and fixedpmcarray

fixes the test to morph fixedpmcarray to integer t/src/extend_vtable.t 9
t/pmc/undef.t: 10 still broken: String morph to Undef, get_class Undef is a PMCProxy
rrot/rurban/morph-gh372: 64b47ff | rurban++ | t/op/morph.t:
[test] start t/op/morph.t
rrot/ptc/c_header_line_length-gh1056: 7965a8f | paultcochrane++ | t/codingstd/linelength.t:
[codingstd] checking C headers for line length violations
16:49
rrot/ptc/c_header_line_length-gh1056: a1203d1 | paultcochrane++ | include/parrot/ (19 files):
[include] wrapping lines to within 100 characters

This matches the coding standards for other C-language files. The tests also pass after these changes.
rrot/ptc/c_header_line_length-gh1056: f8a9805 | paultcochrane++ | src/io/io_private.h:
[io] wrapping lines to within 100 characters
rrot/ptc/c_header_line_length-gh1056: ddc6b9a | paultcochrane++ | include/parrot/atomic/gcc_ppc.h:
[include] wrapping lines to 100 characters

This change is specific for PPC and thus I could not test it (Linux x86_64). The change is trivial and *should* work, however it would be preferable for someone with a PPC architecture to check that everything still works. This is the reason that this commit is separate to the others.
rrot/ptc/c_header_line_length-gh1056: ac25bf2 | rurban++ | include/parrot/ (2 files):
[codingstd] fix c_parens.t 1 for thr_pthread.h and gc_api.h

while(0) => while (0)
rrot: 7965a8f | paultcochrane++ | t/codingstd/linelength.t:
[codingstd] checking C headers for line length violations
17:24
rrot: a1203d1 | paultcochrane++ | include/parrot/ (19 files):
[include] wrapping lines to within 100 characters

This matches the coding standards for other C-language files. The tests also pass after these changes.
rrot: f8a9805 | paultcochrane++ | src/io/io_private.h:
[io] wrapping lines to within 100 characters
rrot: 7b59cd5 | paultcochrane++ | include/parrot/atomic/gcc_ppc.h:
[include] wrapping lines to 100 characters

This change is specific for PPC and thus I could not test it (Linux x86_64). The change is trivial and *should* work, however it would be preferable for someone with a PPC architecture to check that everything still works. This is the reason that this commit is separate to the others.
  rurban: tested ok on PPC
rrot: 5d6852b | rurban++ | include/parrot/ (2 files):
[codingstd] fix c_parens.t 1 for thr_pthread.h and gc_api.h

while(0) => while (0)
rrot: ac97bfe | paultcochrane++ | config/auto/warnings.pm:
[config] warnings for missing declarations or prototypes now errors

Parrot builds successfully with these warnings elevated to errors.
17:29
rrot: 21e6c25 | paultcochrane++ | config/auto/warnings.pm:
[config] Correcting debug message text in warnings.pm

What looks like a copy-paste error meant that a debug message from
  `config/auto/warnings.pm` was mentioning attributes instead of warnings.
This change corrects this issue.
17:31
rrot: 9a504bf | paultcochrane++ | docs/project/cage_cleaners_guide.pod:
[docs] mentioning the `--cage` configure option for more warnings
17:42
rrot: 1bb39fa | paultcochrane++ | docs/project/cage_cleaners_guide.pod:
[docs] adding notes to cage cleaners guide about FORTIFY_SOURCE

This should address the issue GH #711.
17:45 Chirag joined 17:59 travis-ci joined
travis-ci [travis-ci] parrot/parrot#994 (master - 5d6852b : Reini Urban): The build passed. 17:59
[travis-ci] Change view : github.com/parrot/parrot/compare/d...6852b18385
[travis-ci] Build details : travis-ci.org/parrot/parrot/builds/20470669
17:59 travis-ci left 18:06 vincent22 joined
dalek Heuristic branch merge: pushed 21 commits to parrot/ptc/fix_pod_errors-gh1055 by rurban 18:09
Heuristic branch merge: pushed 21 commits to parrot by rurban 18:11
rrot/rurban/SHIM-gh1052: cd26cab | rurban++ | / (43 files):
[cage] change UNUSED(arg) to SHIM in the decl #1052

UNUSED just adds code to please the compiler unused warnings, but we should really declare the args as such with SHIM() We need UNUSED only in .pmc declaration because we have no control over SHIM, the pmc2c compiler does it for us there.
Also do not return unused values, rather cast to (void) and strip PARROT_WARN_UNUSED_RESULT attributes on them.
UNUSED Exceptions: io vtable methods keep the 2nd handle arg. The C abi defines caller cleanup, so we could declare some of them with empty args, but for the sake of API sanity we accept at least 2 args.
Headerizer does not grok #ifdef (yet), so we need to UNUSED some args.
18:20
18:20 TonyC_ joined 18:28 stryx` joined 18:30 travis-ci joined
travis-ci [travis-ci] parrot/parrot#995 (master - ac97bfe : Paul Cochrane): The build passed. 18:30
[travis-ci] Change view : github.com/parrot/parrot/compare/5...97bfe7a8ba
[travis-ci] Build details : travis-ci.org/parrot/parrot/builds/20471025
18:30 travis-ci left 18:42 denis_boyun__ joined, FROGGS joined
dalek rrot/rurban/SHIM-gh1052: 750f536 | rurban++ | / (43 files):
[cage] change UNUSED(arg) to SHIM in the decl #1052

UNUSED just adds code to please the compiler unused warnings, but we should really declare the args as such with SHIM() We need UNUSED only in .pmc declaration because we have no control over SHIM, the pmc2c compiler does it for us there.
Also do not return unused values, rather cast to (void) and strip PARROT_WARN_UNUSED_RESULT attributes on them.
UNUSED Exceptions: io vtable methods keep the 2nd handle arg. The C abi defines caller cleanup, so we could declare some of them with empty args, but for the sake of API sanity we accept at least 2 args.
Headerizer does not grok #ifdef (yet), so we need to UNUSED some args.
18:51
rrot/rurban/SHIM-gh1052: 1f0d1e8 | rurban++ | / (4 files):
[cage] remove redundant duplicate declarations

seen with --cage
rurban gone for a few hours 18:56
18:57 travis-ci joined
travis-ci [travis-ci] parrot/parrot#996 (master - 21e6c25 : Paul Cochrane): The build passed. 18:57
[travis-ci] Change view : github.com/parrot/parrot/compare/a...e6c25e5420
[travis-ci] Build details : travis-ci.org/parrot/parrot/builds/20471222
18:57 travis-ci left 19:25 travis-ci joined
travis-ci [travis-ci] parrot/parrot#997 (master - 1bb39fa : Paul Cochrane): The build passed. 19:25
[travis-ci] Change view : github.com/parrot/parrot/compare/2...b39fab93e9
[travis-ci] Build details : travis-ci.org/parrot/parrot/builds/20472089
19:25 travis-ci left 19:45 travis-ci joined
travis-ci [travis-ci] parrot/parrot#998 (master - 9a67ff6 : Reini Urban): The build passed. 19:45
[travis-ci] Change view : github.com/parrot/parrot/compare/1...67ff6a680e
[travis-ci] Build details : travis-ci.org/parrot/parrot/builds/20474286
19:45 travis-ci left 21:51 rurban1 joined 23:00 rurban1 joined 23:04 kid51 joined 23:05 rurban2 joined 23:06 Psyche^ joined
dalek rrot: 86e504c | rurban++ | / (43 files):
[cage] change UNUSED(arg) to SHIM in the decl [GH #1052]

UNUSED just adds code to please the compiler unused warnings, but we should really declare the args as such with SHIM() We need UNUSED only in .pmc declaration because we have no control over SHIM, the pmc2c compiler does it for us there.
Also do not return unused values, rather cast to (void) and strip PARROT_WARN_UNUSED_RESULT attributes on them.
UNUSED Exceptions: io vtable methods keep the 2nd handle arg. The C abi defines caller cleanup, so we could declare some of them with empty args, but for the sake of API sanity we accept at least 2 args.
Headerizer does not grok #ifdef (yet), so we need to UNUSED some args.
23:10
rrot: 9bf1f1f | rurban++ | / (4 files):
[cage] remove redundant duplicate declarations

seen with --cage
23:28 travis-ci joined
travis-ci [travis-ci] parrot/parrot#999 (master - 9bf1f1f : Reini Urban): The build passed. 23:28
[travis-ci] Change view : github.com/parrot/parrot/compare/9...f1f1fc8778
[travis-ci] Build details : travis-ci.org/parrot/parrot/builds/20493659
23:28 travis-ci left