Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • Sep 30 2021 15:28
    elliotchance commented #888
  • Sep 30 2021 15:02
    tijeco commented #888
  • Sep 30 2021 13:27
    elliotchance commented #888
  • Sep 30 2021 02:50
    tijeco opened #888
  • Aug 14 2021 19:54

    elliotchance on v0.26.10

    (compare)

  • Aug 14 2021 19:54

    elliotchance on master

    Ignore keywords when part of a … (compare)

  • Aug 14 2021 19:54
    elliotchance closed #887
  • Aug 14 2021 19:54
    elliotchance closed #886
  • Aug 14 2021 04:56
    jcburley commented #887
  • Aug 14 2021 04:22
    mpictor synchronize #887
  • Aug 14 2021 04:19
    elliotchance edited #887
  • Aug 14 2021 04:19
    mpictor opened #887
  • Aug 14 2021 04:07
    mpictor commented #886
  • Aug 14 2021 00:47
    mpictor commented #886
  • Aug 14 2021 00:46
    mpictor commented #886
  • Aug 13 2021 04:50
    mpictor opened #886
  • Jul 15 2021 17:24
    elliotchance commented #885
  • Jul 15 2021 12:29
    zazola commented #885
  • Jul 15 2021 12:29
    zazola edited #885
  • Jul 15 2021 12:26
    zazola opened #885
Elliot Chance
@elliotchance
That was almost exclusively the fantastic work of @Konstantin8105 !
Konstantin
@Konstantin8105
Thank you @elliotchance for your reviews, notes! And one more point - Thanks for clear design of project.
Konstantin
@Konstantin8105
Hello @elliotchance , could you check in priority #558 and if it acceptable for you , then merge that PR, because that PR is main for others.
Carsten Hagemann
@carstenhag
@Konstantin8105 could you answer on elliotchance/c2go#570 please? :)
Konstantin
@Konstantin8105
Done
Elliot Chance
@elliotchance
@Konstantin8105 #558 still has merge conflicts
@Konstantin8105 sorry it does not, GitHub thought I wanted to rebase it
Konstantin
@Konstantin8105
Dear @elliotchance , could you approve to changing approach for my work from "many small PR" to "big patch", because like I understood you haven't enought free time for approve each small PRs, also that approach don't stop myself.
Elliot Chance
@elliotchance
Small PRs that fix a specific issue is still the best option. I can review and merge small PRs faster than trying to comb through really large PRs that change many things - I am also more hesitant to merge in big PRs
the branch will be squashed when its merged which is also better for version control and being able to find when a bug was introduced and what caused it
I'm sorry that I haven't been reviewing the last few days I was away for the weekend for my bucks party, but I'll get right on it now :)
@Konstantin8105 ^
Konstantin
@Konstantin8105
Dear @elliotchance , I also like small PR.
Elliot Chance
@elliotchance
@Konstantin8105 I know you merged several PRs into one big on, but they were already reviewed. Do you mind reopening them?
Konstantin
@Konstantin8105
@elliotchance , now, the best way is merge the large PR.
Konstantin
@Konstantin8105
Dear @elliotchance , could you merge #558
Konstantin
@Konstantin8105
Few month ago, you say somethink like - squashing of git is not easy. Like I found https://github.com/blog/2141-squash-your-commits - you can choose merge+squash in github options.
Elliot Chance
@elliotchance
I have merged #558 in but please don't combine multiple PRs, small PRs that represent each issue fixed is much better
also, I do squash the changes, so it only creates one commit per PR
Konstantin
@Konstantin8105
Konstantin
@Konstantin8105
Dear @elliotchance , Is possible merge my PR. Or project c2go is closed?
Elliot Chance
@elliotchance
@Konstantin8105 - im sorry for the slow reply on your PRs this week, I have been very busy, c2go is definitely still going - I will get onto those PRs now :)
Konstantin
@Konstantin8105
@elliotchance , It is a good news.
Konstantin
@Konstantin8105
Dear @elliotchance could you merge next PR
Konstantin
@Konstantin8105
Dear @elliotchance, prepare PR
Elliot Chance
@elliotchance
I want to extend a big thank you to @kamphaus for all your recent work and to bring us to v0.23.0 Berkelium 2018-04-27!
Dong-hee Na
@corona10
Hi I sent my first PR(elliotchance/c2go#714) for this project. PTAL
Ben Lubar
@BenLubar

https://gist.github.com/9f9d1fc6bbfe9297b0446a264678b140.git

Looks like two regexes don't work for espeak:

panic: could not match regexp with string
^(?P<address>[0-a-fx]+) <(?P<position>.*)>(?P<inherited> Inherited)?(?P<default> Default)?[\s]*$
x55668c652fa8 <line:600:13> Implicit Default
panic: could not match regexp with string
^(?P<address>[0-9a-fx]+) (?:prev (?P<prev>0x[0-9a-f]+) )?(?:parent (?P<parent>0x[0-9a-f]+) )?<(?P<position>.*)>(?P<position2> .+:\d+)?(?P<used> used)?(?P<referenced> referenced)?(?P<name> \w+)? '(?P<type>.+?)'(?P<type2>:'.*?')?(?P<extern> extern)?(?P<static> static)?(?P<cinit> cinit)?(?P<register> register)?[\s]*$
0x55c911da71e0 <col:2, col:110> col:18 used x 'double' register cinit

There's also an implementation of basename that I can just delete.

Elliot Chance
@elliotchance
@BenLubar you can fix panics issues by adding new test cases to the ast package
Here is an example: elliotchance/c2go#712
Or if you do not have the desire to fix it yourself, please open an issue so somebody else can pick it up
Hajime Hoshi
@hajimehoshi
Hi all,
I was wondering if c2go is now actively developed.
Elliot Chance
@elliotchance
Hi @hajimehoshi, I am no longer actively working on it but pull requests are still welcomed
Hajime Hoshi
@hajimehoshi
I thought if we could apply c2go to tcc (Tiny C Compiler), we could get a C compiler in Go
I confirmed this require a lot of configuration, and I'm not sure this is possible
Elliot Chance
@elliotchance
Yes, that's probably true
Crystal-Lilith
@Crystal-Lilith
Hello
(last message was in 2018)
I was wondering if there is a way to include all of a folder using c2go
Denys Smirnov
@dennwc
Hey guys! I though you might be interested in another C-to-Go transpiler :)
https://github.com/gotranspile/cxgo
It's based on an (awesome) cznic's CC, so no C dependencies in the transpiler itself.
Let me know what you think, or if you find a good use case for it.
Andrew Bastien
@volundmush
I'm trying to get c2go to compile and install so I can actually try it to begin with
ah there, got the binary in the path properly so...
dennwc
@dennwc:nwca.xyz
[m]
Hi @volundmush ! Did you got it working in the end?
Tinker42
@Tinker42
@elliotchance Hello, I downloaded your c2go source code but I don't see comprehensive instructions on how to compile or install it. The README has instructions to download and use but there seems to be a step missing.
Elliot Chance
@elliotchance
@Tinker42 sorry for the slow response, this is a Go project so to build the binary you just need to run go build in the repository, this will produce the binary in the same directory as the project
Andrew Bastien
@volundmush
oh so I did muck with this thing before. lessee....
Andrew Bastien
@volundmush
got the command responding so.... now I need to figure out how to best try to transpile uh... well, this
problem is that it includes sqlite3 so I doubt that it's going to work.... I might get some useful code out of it though. But, I'm fairly sure that I'm going to need to 'transpile/port' the project the hard way