Nicely done
On Jun 13, 2016 1:00 PM, "lvlagic" <notifications@github.com> wrote:

> OK dude, here is my full report.
>
> Just tested 10 consecutive headshots. Each headshot killed the enemy
> player, but only 3 registered on the scoreboard. I logged the time of each
> shot (within a few seconds) by typing in chat just before i took the shot.
> I have added next to each time entry whether the shot registered or not,
> (Yes) or (No). The shots were timed at around:
>
> *(No)* 20:24:59 | (Global) Magic: 1
> *(Yes)* 20:26:11 | (Global) Magic: 2
> *(No)* 20:27:21 | (Global) Magic: 3
> *(Yes)* 20:28:28 | (Global) Magic: 4
> *(Yes)* 20:29:36 | (Global) Magic: 5
> *(No)* 20:31:08 | (Global) Magic: 6
> *(No)* 20:32:29 | (Global) Magic: 7
> *(No)* 20:34:44 | (Global) Magic: 8
> *(No)* 20:35:52 | (Global) Magic: 9
> *(No)* 20:36:59 | (Global) Magic: 10
>
> Below is the server RPT covering the whole period from 20:24:59 -
> 20:36:59. There does not appear to be any entry at all for the failed
> shots, so it is like it never happened. I have some videos i can put up if
> you want to see the number typed in chat and then the shot but shadow play
> is on the blink and the screen keeps flashing its a pain to watch, but i
> can upload them if you wish. At 20:33:36 we both suicided.
>
> 20:24:52 "TERRITORY SYSTEM: _realLoopTime was 10.0129"
> 20:25:02 "TERRITORY SYSTEM: _realLoopTime was 10.0369"
> 20:25:10 Server: Object 6:141 not found (message Type_427)
> 20:25:10 Error: Object(6 : 141) not found
> 20:25:12 "TERRITORY SYSTEM: _realLoopTime was 10.0271"
> 20:25:22 "TERRITORY SYSTEM: _realLoopTime was 10.01"
> 20:25:25 Server: Object 6:145 not found (message Type_120)
> 20:25:25 Server: Object 6:147 not found (message Type_120)
> 20:25:25 Server: Object 6:148 not found (message Type_120)
> 20:25:25 Server: Object 6:149 not found (message Type_427)
> 20:25:32 "TERRITORY SYSTEM: _realLoopTime was 10.0029"
> 20:25:42 "TERRITORY SYSTEM: _realLoopTime was 10.0181"
> 20:25:52 "TERRITORY SYSTEM: _realLoopTime was 10.0029"
> 20:26:02 "TERRITORY SYSTEM: _realLoopTime was 10.0171"
> 20:26:12 "TERRITORY SYSTEM: _realLoopTime was 10.0271"
> 20:26:21 "registerKillScore: [R Alpha 2-1:1 ([nWo] Frank) REMOTE,R Echo 1-1:1 (Magic) REMOTE,R Alpha 2-1:1 ([nWo] Frank) REMOTE], registered: false, isPlayer: true, killerDead: false, killerGroup: R Echo 1-1"
> 20:26:22 "TERRITORY SYSTEM: _realLoopTime was 10.0349"
> 20:26:32 "TERRITORY SYSTEM: _realLoopTime was 10.032"
> 20:26:36 Server: Object 6:155 not found (message Type_120)
> 20:26:36 Server: Object 6:157 not found (message Type_120)
> 20:26:36 Server: Object 6:158 not found (message Type_120)
> 20:26:36 Server: Object 6:154 not found (message Type_127)
> 20:26:36 Server: Object 6:159 not found (message Type_427)
> 20:26:42 "TERRITORY SYSTEM: _realLoopTime was 10.0181"
> 20:26:52 "TERRITORY SYSTEM: _realLoopTime was 10.0168"
> 20:27:02 "TERRITORY SYSTEM: _realLoopTime was 10.022"
> 20:27:05 "SERVER CLEANUP: Deleted 1 expired objects"
> 20:27:05 "SERVER CLEANUP: Deleted 0 empty groups"
> 20:27:09 "A3W - 0 vehicles have been saved with extDB"
> 20:27:12 "TERRITORY SYSTEM: _realLoopTime was 10.002"
> 20:27:22 "TERRITORY SYSTEM: _realLoopTime was 10.0381"
> 20:27:28 Server: Object 6:161 not found (message Type_427)
> 20:27:29 Error: Object(6 : 161) not found
> 20:27:32 "TERRITORY SYSTEM: _realLoopTime was 10.0378"
> 20:27:42 "TERRITORY SYSTEM: _realLoopTime was 10.033"
> 20:27:43 Server: Object 6:165 not found (message Type_120)
> 20:27:43 Server: Object 6:167 not found (message Type_120)
> 20:27:43 Server: Object 6:168 not found (message Type_120)
> 20:27:44 Server: Object 6:169 not found (message Type_427)
> 20:27:52 "TERRITORY SYSTEM: _realLoopTime was 10.0122"
> 20:28:02 "TERRITORY SYSTEM: _realLoopTime was 10.0149"
> 20:28:12 "TERRITORY SYSTEM: _realLoopTime was 10.011"
> 20:28:22 "TERRITORY SYSTEM: _realLoopTime was 10.0249"
> 20:28:32 "TERRITORY SYSTEM: _realLoopTime was 10.0242"
> 20:28:35 Server: Object 6:171 not found (message Type_92)
> 20:28:35 "registerKillScore: [R Alpha 2-1:1 ([nWo] Frank) REMOTE,R Echo 1-1:1 (Magic) REMOTE,R Alpha 2-1:1 ([nWo] Frank) REMOTE], registered: false, isPlayer: true, killerDead: false, killerGroup: R Echo 1-1"
> 20:28:35 Error: Object(6 : 171) not found
> 20:28:42 "TERRITORY SYSTEM: _realLoopTime was 10.0349"
> 20:28:49 Server: Object 6:175 not found (message Type_120)
> 20:28:50 Server: Object 6:178 not found (message Type_427)
> 20:28:50 Server: Object 6:177 not found (message Type_120)
> 20:28:50 Server: Object 6:179 not found (message Type_427)
> 20:28:50 Error: Object(6 : 178) not found
> 20:28:52 "TERRITORY SYSTEM: _realLoopTime was 10.0129"
> 20:29:02 "TERRITORY SYSTEM: _realLoopTime was 10.009"
> 20:29:12 "TERRITORY SYSTEM: _realLoopTime was 10.0132"
> 20:29:22 "TERRITORY SYSTEM: _realLoopTime was 10.0078"
> 20:29:32 "TERRITORY SYSTEM: _realLoopTime was 10.021"
> 20:29:42 "TERRITORY SYSTEM: _realLoopTime was 10.0281"
> 20:29:44 "A3W - 80 baseparts and objects have been saved with extDB"
> 20:29:45 Server: Object 6:181 not found (message Type_427)
> 20:29:45 "registerKillScore: [R Alpha 2-1:1 ([nWo] Frank) REMOTE,R Echo 1-1:1 (Magic) REMOTE,R Alpha 2-1:1 ([nWo] Frank) REMOTE], registered: false, isPlayer: true, killerDead: false, killerGroup: R Echo 1-1"
> 20:29:45 Error: Object(6 : 181) not found
> 20:29:53 "TERRITORY SYSTEM: _realLoopTime was 10.0391"
> 20:30:00 Server: Object 6:185 not found (message Type_120)
> 20:30:00 Server: Object 6:187 not found (message Type_120)
> 20:30:00 Server: Object 6:188 not found (message Type_120)
> 20:30:00 Server: Object 6:189 not found (message Type_427)
> 20:30:03 "TERRITORY SYSTEM: _realLoopTime was 10.0139"
> 20:30:13 "TERRITORY SYSTEM: _realLoopTime was 10.0051"
> 20:30:23 "TERRITORY SYSTEM: _realLoopTime was 10.0088"
> 20:30:33 "TERRITORY SYSTEM: _realLoopTime was 10.0232"
> 20:30:43 "TERRITORY SYSTEM: _realLoopTime was 10.0239"
> 20:30:53 "TERRITORY SYSTEM: _realLoopTime was 10.0381"
> 20:30:57 Error: Object(6 : 192) not found
> 20:31:03 "TERRITORY SYSTEM: _realLoopTime was 10.0239"
> 20:31:13 "TERRITORY SYSTEM: _realLoopTime was 10.0281"
> 20:31:14 Server: Object 6:193 not found (message Type_92)
> 20:31:14 Error: Object(6 : 193) not found
> 20:31:23 "TERRITORY SYSTEM: _realLoopTime was 10.031"
> 20:31:29 Server: Object 6:197 not found (message Type_120)
> 20:31:29 Server: Object 6:199 not found (message Type_120)
> 20:31:29 Server: Object 6:200 not found (message Type_120)
> 20:31:29 Server: Object 6:201 not found (message Type_427)
> 20:31:33 "TERRITORY SYSTEM: _realLoopTime was 10.0029"
> 20:31:43 "TERRITORY SYSTEM: _realLoopTime was 10.0039"
> 20:31:53 "TERRITORY SYSTEM: _realLoopTime was 10.0032"
> 20:32:03 "TERRITORY SYSTEM: _realLoopTime was 9.99683"
> 20:32:09 "SERVER CLEANUP: Deleted 4 expired objects"
> 20:32:10 "SERVER CLEANUP: Deleted 0 empty groups"
> 20:32:13 "TERRITORY SYSTEM: _realLoopTime was 10.001"
> 20:32:13 Error: Object(6 : 203) not found
> 20:32:14 "A3W - 0 vehicles have been saved with extDB"
> 20:32:23 "TERRITORY SYSTEM: _realLoopTime was 10.0391"
> 20:32:33 "TERRITORY SYSTEM: _realLoopTime was 10.0271"
> 20:32:34 Server: Object 6:204 not found (message Type_427)
> 20:32:34 Error: Object(6 : 204) not found
> 20:32:43 "TERRITORY SYSTEM: _realLoopTime was 10.01"
> 20:32:49 Server: Object 6:208 not found (message Type_120)
> 20:32:49 Server: Object 6:210 not found (message Type_120)
> 20:32:49 Server: Object 6:211 not found (message Type_120)
> 20:32:49 Server: Object 6:212 not found (message Type_427)
> 20:32:53 "TERRITORY SYSTEM: _realLoopTime was 9.99902"
> 20:33:03 "TERRITORY SYSTEM: _realLoopTime was 10.03"
> 20:33:13 "TERRITORY SYSTEM: _realLoopTime was 10.012"
> 20:33:23 "TERRITORY SYSTEM: _realLoopTime was 10.0339"
> 20:33:33 "TERRITORY SYSTEM: _realLoopTime was 10.0061"
> 20:33:36 "registerKillScore: [R Echo 1-1:1 (Magic) REMOTE,<NULL-object>,R Echo 1-1:1 (Magic) REMOTE], registered: false, isPlayer: true, killerDead: true, killerGroup: <NULL-group>"
> 20:33:36 "registerKillScore: [R Alpha 2-1:1 ([nWo] Frank) REMOTE,<NULL-object>,R Alpha 2-1:1 ([nWo] Frank) REMOTE], registered: false, isPlayer: true, killerDead: true, killerGroup: <NULL-group>"
> 20:33:43 "TERRITORY SYSTEM: _realLoopTime was 10.0078"
> 20:33:50 Server: Object 6:217 not found (message Type_120)
> 20:33:51 Server: Object 6:219 not found (message Type_120)
> 20:33:51 Server: Object 6:220 not found (message Type_120)
> 20:33:51 Server: Object 6:221 not found (message Type_427)
> 20:33:51 Server: Object 3:21 not found (message Type_120)
> 20:33:51 Server: Object 3:23 not found (message Type_92)
> 20:33:51 Server: Object 3:24 not found (message Type_120)
> 20:33:51 Server: Object 3:25 not found (message Type_92)
> 20:33:52 Error: Object(3 : 28) not found
> 20:33:53 "TERRITORY SYSTEM: _realLoopTime was 10.012"
> 20:34:03 "TERRITORY SYSTEM: _realLoopTime was 10.0061"
> 20:34:13 "TERRITORY SYSTEM: _realLoopTime was 10.01"
> 20:34:23 "TERRITORY SYSTEM: _realLoopTime was 10.033"
> 20:34:33 "TERRITORY SYSTEM: _realLoopTime was 10.033"
> 20:34:33 Error: Object(6 : 223) not found
> 20:34:43 "TERRITORY SYSTEM: _realLoopTime was 10.0242"
> 20:34:49 "A3W - 80 baseparts and objects have been saved with extDB"
> 20:34:49 Server: Object 6:224 not found (message Type_92)
> 20:34:49 Error: Object(6 : 224) not found
> 20:34:53 "TERRITORY SYSTEM: _realLoopTime was 10.0288"
> 20:35:03 "TERRITORY SYSTEM: _realLoopTime was 10.0132"
> 20:35:04 Server: Object 6:228 not found (message Type_120)
> 20:35:04 Server: Object 6:230 not found (message Type_120)
> 20:35:04 Server: Object 6:231 not found (message Type_120)
> 20:35:04 Server: Object 6:232 not found (message Type_427)
> 20:35:13 "TERRITORY SYSTEM: _realLoopTime was 10.009"
> 20:35:23 "TERRITORY SYSTEM: _realLoopTime was 10.0339"
> 20:35:33 "TERRITORY SYSTEM: _realLoopTime was 10.0168"
> 20:35:43 "TERRITORY SYSTEM: _realLoopTime was 10.041"
> 20:35:53 "TERRITORY SYSTEM: _realLoopTime was 10.0342"
> 20:35:58 Server: Object 6:235 not found (message Type_427)
> 20:35:58 Error: Object(6 : 235) not found
> 20:36:03 "TERRITORY SYSTEM: _realLoopTime was 10.0391"
> 20:36:13 Server: Object 6:239 not found (message Type_120)
> 20:36:13 Server: Object 6:241 not found (message Type_120)
> 20:36:13 Server: Object 6:242 not found (message Type_120)
> 20:36:13 "TERRITORY SYSTEM: _realLoopTime was 10.022"
> 20:36:13 Server: Object 6:243 not found (message Type_427)
> 20:36:46 Performance warning: SimpleSerialization::Read 'messageSystem' is using type of ,'TEXT' which is not optimized by simple serialization, falling back to generic serialization, use generic type or ask for optimizations for these types
> 20:36:51 "WASTELAND SERVER - Assault Mission waiting to run: mission_Smugglers"
> 20:36:51 Performance warning: SimpleSerialization::Write 'messageSystem' is using type of ',TEXT' which is not optimized by simple serialization, falling back to generic serialization, use generic type or ask for optimizations for these types
> 20:36:51 Performance warning: SimpleSerialization::Read 'messageSystem' is using type of ,'TEXT' which is not optimized by simple serialization, falling back to generic serialization, use generic type or ask for optimizations for these types
> 20:36:53 "TERRITORY SYSTEM: _realLoopTime was 10.0269"
> 20:37:03 "TERRITORY SYSTEM: _realLoopTime was 10.0261"
> 20:37:06 Server: Object 6:245 not found (message Type_120)
> 20:37:10 Error: Object(7 : 16) not found
> 20:37:13 "TERRITORY SYSTEM: _realLoopTime was 10.0308"
> 20:37:15 "SERVER CLEANUP: Deleted 4 expired objects"
> 20:37:15 "SERVER CLEANUP: Deleted 0 empty groups"
> 20:37:19 "A3W - 0 vehicles have been saved with extDB"
> 20:37:21 Server: Object 6:249 not found (message Type_120)
> 20:37:21 Server: Object 6:251 not found (message Type_92)
> 20:37:21 Server: Object 6:252 not found (message Type_120)
> 20:37:21 Server: Object 6:253 not found (message Type_92)
> 20:37:21 Error: Object(6 : 251) not found
> 20:37:23 "TERRITORY SYSTEM: _realLoopTime was 10.0381"
> 20:37:34 "TERRITORY SYSTEM: _realLoopTime was 10.0249"
> 20:37:44 "TERRITORY SYSTEM: _realLoopTime was 10.0244"
> 20:37:54 "TERRITORY SYSTEM: _realLoopTime was 10"
> 20:38:04 "TERRITORY SYSTEM: _realLoopTime was 10.0386"
> 20:38:14 "TERRITORY SYSTEM: _realLoopTime was 10.0132"
> 20:38:24 "TERRITORY SYSTEM: _realLoopTime was 10.0142"
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/A3Wasteland/ArmA3_Wasteland.Altis/commit/04fcc689ac24a9ca07ddde9d0d10e34e2073e650#commitcomment-17849906>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AI6sJmYsNINCL1VMukCFBOZ-Edz2i_6kks5qLbbGgaJpZM4IvWj8>
> .
>


120694307_Fault_ChillerSWTSensorOffset (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1653 of 1654 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.94%25-yellow.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.93%25-green.svg)


Good idea. I will look into that.


Лучше создать свой тип исключений и бросать в этом месте.


fixes #534


This change really makes no sense, the optional parameter is a much cleaner design since all the no-param overload is doing is calling the other method. It's not doing any different behavior that would warrant the separate overload.


yes


Is this change intended? If so, other pages will be affected as well. Maybe we should see tomorrow if we would want that to change globally indeed.


Why are these options are being added?


Зачем `var` убрал?


no I haven't built with it. I'm not interested in that

we "should" have ALL patches same as cm13 in build repo

On 06/13/2016 05:25 PM, FaArIsH wrote:
>
> (ya this commit doesn't have anything to do with this problem. )
>
> yes its working perfectly in cm13. but in aicp11 its not working tried
> many times with different ways.
>
> did u tried building with WITH_DEXPREOPT := true ?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/AICP/build/commit/349d762dc688b0726eb58aa7ade41b0c16d88e0a#commitcomment-17844534>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AAqWXl6oT3fLchw7jYovzVYtpUmC5d8Iks5qLXZ1gaJpZM4Iz1n0>.
>


@metal3d default value to datetime causes panics for some users (MariaDB) so needs to be done in code.


-The size of the test, 30 x 30, seems a little bit large. Is there another test problem of a similar size you are basing this on? I think we usually try to limit test problems to 20 x 20 or less.


А этот аргумент точно нужен? Для чего он используется? Почему просто нельзя проскроллить к посту?

И да, метод лучше переименовать в `selectAndScrollTo`.


This changes the behavior of lovd_isAuthorized() for when passing multiple entries and checking ownership. Can we expect issues?
(first, we'd set `$bOwner` true if we have *a* match, now we set to true if *all* matches)


Нужно вынести в файл .properties


No.  Not all plugins are setup this way.  That is just how most of our plugins are setup.  Normally we would catch this during a PR review.

Were you attempting to trigger a release?


`title=">>${ num }"`


The second sentence should go in a note that’s a separate paragraph.  Telling people what’s out-of-scope here is not the same as telling them what this proposal is about.  Furthermore, there’s no particular reason to think that an integer API cleanup and generic-programming-usefulness upgrade would have anything at all to do with mixed-type arithmetic.


but also the se.cfg should be changed, right? why not use liberty.se also on windows?


15 -> 16 :)


Not necessarily, there is always a default. When you select your combination you store it in the db and so the entry is created.


якщо немає jar ліб - видаляй рядок


Опечатка.


Confusion has nothing to do with what you go on to describe below.  I’d say "Swift's integer protocols don't currently provide a suitable basis for generic programming."


воно і так null, не треба явно ставити


ahahaha


For this test file I'm following phase_field/tests/phase_field_kernels/SplitCahnHilliard.i. Sure, I can make it smaller for a fast test.


implements java.io.Serializable
JavaBean conventions:
https://en.wikipedia.org/wiki/JavaBeans


через ButterKnife @OnClick


Тому що при навигации по картинкам очевиднее, егда идет выравнивание по картинке. а не по посту. ОСОБЕННО это заметно, если картинка находится в оп-посте.


якщо буде ButterKnife @OnClick - цього взагалі не треба буде


Adicionado dfs recursivo


120796739_ObjListAutoPopulate (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2244 of 2252 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.64%25-yellow.svg)


@anthony-moendir This commit broke the build.


Do you see a quick way how we would be able to prevent this function (and the queries) to run twice? If not, we can leave it like this, and maybe put a FIXME there or so, so that we might look at it later.


I also followed this from the SplitCahnHilliard.i test block. I'm not sure about its use, so I just keep it here. Should I just delete this?


Add_Generator_MicroTurbine (jmarrec) - x86_64-MacOS-10.10-clang: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-79.16%25-red.svg)


використовуй приставки, щоб видно було звідки ноги ростуть, toolbar -> main_toolbar (ще є підхід префікс типу ставити типу венгерської нотації - але як на мене це шлак)


Hi again,
can i contact you for support?
my email jaimerodg@gmail.com
regards


1. There is no `Arithmetic` proposal
2. The way the binary operations are defined does not in itself make generic programming impossible.  This works just fine.

    ```swift
        func hypSquared<T: IntegerArithmetic>(x: T, y: T) -> T {
          return x * x + y * y
        }
    ```

However, it doesn’t generalize to floating point, as it should, because the protocol that has arithmetic is not designed to cover FP.


Лучше использовать java.util.Date


переходь на світлу сторону, на Retrofit :)


Cheers, I'd like to get this sorted out and try to do a thorough job if i can. I am so pleased with the new death message system, i had been hoping for it to happen for ages so its a shame that a little bug somewhere is currently letting the whole thing down!


android:theme="@style/AppTheme.NoActionBar"></activity>
->
android:theme="@style/AppTheme.NoActionBar"/>


"main_act"; -> MainActivity.class.getSimpleName();


do not just rename a file like that please!


відмовляйся від AsyncTask на користь ExecutorService/IntentService


Либо пост имеет длинный текст, да. Тогда лучше `node` переименовать в `scrollNode` и использовать сам элемент, а не его ООП представление, т.е. написать `scrollNode = this.el` выше, а вызывать так: `data.post.selectCurrent(data.post.images.first.el);`.


Просто оченьл раздражает, что везде все скроллилось в топ, а клавиатурная навигация шла по центру. В итоге кликаешь по >>ссылке, нажимаешь `J` и скачешь хуй знает куда.


Is not it surprising, that no VAT No field is there in the info-config? Though it is  mandatory in almost every country to be printed on the invoice / receipt.


перед комітом і після роботи над класом роби автоформат Ctrl+Alt+L


для логів використовуй Timber (https://github.com/JakeWharton/timber)


ikr, i just cant be bothered to fix them


через ButterKnife


This looks much better. ;)


This proposal doesn’t leave us in a different place w.r.t. (with respect to) the existence of safe conversions AFAICT.
What it does do:

1. Makes it possible to do these conversions *generically* (IIRC. Please check).
2. Removes the artificial limitation that there is a maximum width integer that’s needed in order to do these conversions, and set by the implementation.  If someone introduced `Int256` in today’s system, it wouldn’t interoperate properly with `numericCast`.


Nice job!


вкладені класи мають бути в самому кінці


відмовляйся від m суфікса для глобальних змінних (mDataset -> dataset)


треба бути завжди якомога ближче до інтерфейсу, тому ArrayList -> List


для ViewHolder теж використовуй ButterKnife


ArrayList -> List


You also have to do this for `FW: Pug 4B`, when you have to jump back to Syndicate space from the Pug worlds to use the graviton... thing.


такі переноси не мають сенсу, використовуй переноси якщо реально довгі рядки


Lol why even...
`Duration speed = scriptEntry.getdObject("speed");`
Makes life easier


I could do that, yes, but I'd rather mark the places where version selectors are used so in the future I can remove the comments entirely.


View v, ViewHolder vh - назви терба давати більш повні і зрозумілі, а то в коді чередуються v/vh - важко зрозуміти


mDataset.size() -> mDataset != null ? mDataset.size() : 0;


ah - i see. i just scripted a nade explosion when ai get run over. so the wheels of vehicles get damaged. i think i had the missions running all the time.


Although first this needs to be arg.asType(Duration.class)


Why even is all this like this


@danielfernandez You are right. Fixed this. I got one question though about inline comments. I use this:

`/*[-` */ This code will be removed at thymeleaf parsing time! /* -]*/`

And this works fine. Although not sure if this is the best way to use comments in `TEXT `mode.


The above would be more realistic like this:

```Swift
var x: Int8 = 42
let y = 1
let z = 0

x <<= (1 as Int16) // error: binary operator '<<' cannot be applied to operands of type 'Int8' and 'Int16’
if (x > z) { … }    // error: binary operator '>' cannot be applied to operands of type 'Int8' and 'Int'
```

Also, currently, bit-shifting a negative number of (or too many) bits causes a trap on some platforms.  That makes low-level bit manipulation needlessly dangerous and unpredictable.


в сінглтоні краще використовувати synchronized ЛИШЕ при першій ініціалізації instance, далі ж лише доступ по volatile:
https://github.com/bethrobson/Head-First-Design-Patterns/blob/master/src/headfirst/designpatterns/singleton/dcl/Singleton.java


I am confused -
https://github.com/avoidwork/tiny-worker/issues/1
the tiny-worker library uses child_process, not webworker threads!


Missing “The” at the beginning of the sentence.


працюй на рівні інтерфейсу


It does work it already supports 0.15 this commit was just to update the motd


Use name(), not toString() here.. Just in case


Lol no problem I messed the whole branch up with my latest update. Haven't had the time to research how to roll it back on Github.


Remove this line ASAP


Looks nice. Inactive people section in not changed?


Nope, just trying to save additional manual work having to bump version when releasing.


remove this line ASAP


Nice! 400th commit!


hey mate can u fix the wood error pls.

Notice: Undefined offset: 3 in phar:///home/tolo/mc2/PocketMine-iTX.phar/src/pocketmine/block/Wood2.php on line 38
[22:23:58] [Server thread/CRITICAL]: TypeError: "Return value of pocketmine\block\Wood2::getName() must be of the type string, null returned" (EXCEPTION) in "/src/pocketmine/block/Wood2" at line 38
[22:23:58] [Server thread/CRITICAL]: [Network] Stopped interface pocketmine\network\RakLibInterface due to Return value of pocketmine\block\Wood2::getName() must be of the type string, null returned


not sure I'm getting the building of rows and elements.  I entered a couple as example. For instance my rows have values for 11 columns: Agency name, city, state....etc.   Could you let me know if I am on the right track for the couple I started before I fill in more?


Nice :-)


well the -pc_type lu is commented out anyway (I believe the # makes it a comment just like in an input file). So that should be removed if it's just a comment because it's not doing anything (you have specified pc_type lu in the input file anyways). max_parallel is probably there to make sure it doesn't run as a parallel job because the input file is not set up to run as a parallel job, you would need to change the petsc options to allow that. So, I think you should leave max_parallel = 1 in, although I'm not completely sure it's necessary.


This is identical to xen/include/asm-x86/p2m.h so no need to redeclare it. The function should just be moved from asm-x86/p2m.h to xen/include/xen/p2m-common.h


No problem, I actually messed my whole branch up with my latest update. Haven't had the time to research how to un-do a bad commit.

Thanks for putting this together I've been having way to much fun trolling on Twitter with this

On Mon, Jun 13, 2016 at 2:37 PM -0400, "Jake" <notifications@github.com> wrote:

lol sorry about that

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Test Comment


Hi,

I tried to compile this driver in Red Hat Linux 7.2 and it was failed. It seems the error is from cfg80211.c. Can you please make it work for Red Hat Linux 7.2. My gcc version is:
 gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-4)

Thanks,
Guangtu


This is Sara


Thanks for this! Make sure to reject my PRs in the future if they lack CHANGELOG notes. ;)


Clear, I will figure it out. Thanks.


3rd edit


Also included Baine rename fix
Baine: Fix for Renaming Script using ' instead of ` (as supplied by anidb when queried for series name)
quick and dirty, maybe adding a replace filter or option would be ideal, up to discussion


merci :-)


run ci


Create a new issue, it's off topic here


Hey thanks for the script, in my case its not working tho. I follow the instruction correctly, i even double check everything to make sure that i was not missing anything but its giving me an error. This is what i am getting i hit the download button. Now i am a noob on this and i dont know what it means but i figure that i might as well report it. Hopefully it helps!

```
['siriuspr', 'WOW1988@@a', 'Gintama', 'https://kissanime.to/Anime/Gintama', '1', '1', '500', 'C:\\Downloads\\Gintama\\', '1920x1080.mp4', 'kissanime.to']
siriuspr
WOW1988@@a
Gintama
https://kissanime.to/Anime/Gintama
1
1
500
C:\Downloads\Gintama\
1920x1080.mp4
kissanime.to
Exception in Tkinter callback
Traceback (most recent call last):
  File "C:\Users\adria\AppData\Local\Programs\Python\Python35-32\lib\tkinter\__init__.py", line 1549, in __call__
    return self.func(*args)
  File "C:\Users\adria\Desktop\KissDownloader-master\KissDownloaderGUI.py", line 125, in run_download
    KissDownloader(params)
  File "C:\Users\adria\Desktop\KissDownloader-master\KissDownloader.py", line 48, in __init__
    self.driver = webdriver.Firefox()
  File "C:\Users\adria\AppData\Local\Programs\Python\Python35-32\lib\site-packages\selenium\webdriver\firefox\webdriver.py", line 81, in __init__
    self.binary, timeout)
  File "C:\Users\adria\AppData\Local\Programs\Python\Python35-32\lib\site-packages\selenium\webdriver\firefox\extension_connection.py", line 51, in __init__
    self.binary.launch_browser(self.profile, timeout=timeout)
  File "C:\Users\adria\AppData\Local\Programs\Python\Python35-32\lib\site-packages\selenium\webdriver\firefox\firefox_binary.py", line 68, in launch_browser
    self._wait_until_connectable(timeout=timeout)
  File "C:\Users\adria\AppData\Local\Programs\Python\Python35-32\lib\site-packages\selenium\webdriver\firefox\firefox_binary.py", line 98, in _wait_until_connectable
    raise WebDriverException("The browser appears to have exited "
selenium.common.exceptions.WebDriverException: Message: The browser appears to have exited before we could connect. If you specified a log_file in the FirefoxBinary constructor, check it for details.
```


I don't think we ever split like this; we never had problems with exploding on `\r\n`, but if we expect problems we should change it everywhere I think...?


Would it not be more useful to contact the person who did this action? `$_AUTH['name']` and `$_AUTH['email']`? If they'd email me, I wouldn't know what to do about it...


@zephiK Hello, sorry for my message
I just want to ask..
How to build Chroma for mako? I translated some strings, made a little edits in code and want to build
Previously I'm worked on translation to russian simpleaosp and succesfully built it
Commands in manifest didn't help me


@arco  finally i got calls from phone.,..

I will now test tests...one think i feel is an overall reduced volume


Thanks for the notice, I talked to Pascal about it and have submitted a [CQ request to IPZilla](https://github.com/eclipse/whiskers/commit/22ae50d6fe13fdbff2c0100911f8736f5771ad51).


Did you have formatting issues on Windows computers? This can't be necessary as far as I know (and we never do it)...


120796739_ObjListAutoPopulate (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1102 of 1102 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-34.0%25-green.svg)


It's just collapsed in the screenshot. The people who had a GitHub account
will have an avatar. I will upload a new screenshot later or tomorrow.
On Jun 13, 2016 23:25, "Kacper Michajłow" <notifications@github.com> wrote:

> Looks nice. Inactive people section in not changed?
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/mpc-hc/mpc-hc.org/commit/21f4384352a762eb7fdd66e5f7f4246de32ed0ba#commitcomment-17850281>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AAVVtZUn3k54NM0ttp2KRkoNDnjTuG5zks5qLby4gaJpZM4I0mrH>
> .
>


120796739_ObjListAutoPopulate (Nigusse) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


quando elevamos a quantidade de elementos na tela a a aplicação quebra!!!


There's a process that proceeds putting the logo up where we verify that we do in fact have the right to put the logo up. The result of that process is them sending us the file, which we usually then compress.

It's a silly hurdle to ask people who just gave us money to them host this file.

I agree that we should remove unused images across the website.


Ok, I wonder how it looks for people without GH account. I believe there is an email in place of "follow" button?


Don't drink and code ;)


![screenshot 73](https://cloud.githubusercontent.com/assets/19911622/16022376/2b2282e6-31d5-11e6-9555-96ebd79a1049.png)


Personally, I prefer not to use variables in situations like these, where they're just used once, on the line directly below. It doesn't add anything to the readability in my opinion. If you don't care about either way, shall we just put it in one command?
(doesn't need to be one line per se)


оказывается, есть такой Issue #804


Top :)


hmmmmm. this works pending some sort of error checking... but I wonder if we should think about background workers to do this rather than in the web request :/


Nice Job me!!!


Wtf?


wow lol


AddOpeningAreaToIVRS (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (2214 of 2214 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.namedfile/19/violations
```
plone/namedfile/copy.py:0:1: I001 isort found changes, run it on the file
plone/namedfile/file.py:291:1: C901 'getImageInfo' is too complex (15)
plone/namedfile/handler.py:0:1: I001 isort found changes, run it on the file
plone/namedfile/scaling.py:173:1: C901 'DefaultImageScalingFactory.__call__' is too complex (13)
plone/namedfile/scaling.py:226:35: S001 found %r formatter
plone/namedfile/testing.py:37:10: Q000 Remove bad quotes.
plone/namedfile/testing.py:42:10: Q000 Remove bad quotes.
plone/namedfile/tests/test_blobfile.py:0:1: I001 isort found changes, run it on the file
plone/namedfile/tests/test_blobfile.py:40:14: Q000 Remove bad quotes.
plone/namedfile/tests/test_blobfile.py:45:14: Q000 Remove bad quotes.
plone/namedfile/tests/test_blobfile.py:50:14: Q000 Remove bad quotes.
plone/namedfile/tests/test_blobfile.py:55:14: Q000 Remove bad quotes.
plone/namedfile/tests/test_doctests.py:0:1: I001 isort found changes, run it on the file
plone/namedfile/tests/test_image.py:0:1: I001 isort found changes, run it on the file
plone/namedfile/tests/test_scaling.py:108:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:109:70: S001 found % formatter
plone/namedfile/tests/test_scaling.py:126:14: D001 found failIf( replace it with assertFalse
plone/namedfile/tests/test_scaling.py:175:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:176:70: S001 found % formatter
plone/namedfile/tests/test_scaling.py:183:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:184:72: S001 found % formatter
plone/namedfile/tests/test_scaling.py:191:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:192:72: S001 found % formatter
plone/namedfile/tests/test_scaling.py:243:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:244:70: S001 found % formatter
plone/namedfile/tests/test_scaling_functional.py:116:24: Q000 Remove bad quotes.
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Woops :sweat_smile:
I fixed it in my commit :wink:
I saw tests had been written as well, shouldn't this have been picked up?


Copied and pasted the setup.py from pintail and forgot to change the version. No point downgrading it now.


@junwan6 Looks simple. Please submit as a pull request to the plugin owner, but without the core edit surrounding comment blocks.


Also this example I'd really like to merge into one line; it also saves you from having to name a variable, which then sticks around in memory (and isn't according to the coding standard here).
(I've fixed this one in this case)


Dude! :disappointed:  It's [Snoop D.O. _**Double**_ G](http://genius.com/Snoop-dogg-snoop-do-double-g-lyrics), not :dog:.


This doesn't help me at all :8ball:


Thanks for catching this. Made the same mistake in pintail. Fixed now.


@ForbesLindesay Is there any reason beyond making this place to be similar to CSS specs for valid classnames for this test? If there are no issues beyond the CSS spec, it would be really nice to remove this test, as classnames like this still can exist, are can sometimes be used. Example: https://github.com/kizu/bemto/issues/81#issuecomment-219309916 — in this case it is used as a shorthand that is later expanded to a proper classname (however, strictly speaking, in _HTML_ the classname could be just anything).

As Pug is a template engine for HTML and not CSS, only having some of the CSS-selector-like syntax sugar, we're not obliged to follow the CSS specs so fanatically as long as it won't break anything. And I don't think that this exact case would break things as it worked just fine in Jade.


I will add advanced option to use legacy toolbar. I know people got sentimental about things. :)


I am sorry for spelling error.
I temporarily use whole study case size as each readers read case size. Because we don't save these information in summary file.


Run command


Transfer command


We don't throw exceptions... to be honest, I think they're user unfriendly. But, on a side note, I think (didn't test it), that a query error will not even trigger this exception... or?


Плохо сегшдня соображаю, однако.


What is the story for how users configure this? I see no error handling for if the configuration is missing, and no defaults. Is the intention for each mycroft device to create their own app credentials, or for us to proxy tweets on behalf of users via cerberus and they add mycroft as a verified twitter app?

Please answer these questions before moving forward with a merge.


Build command


Example .fiber.yml


Sure, I guess :D


I'm in Boston, so time and date is flexible, etc. -- if we want to turn this into a whole weekend of music notation hackathon or something, that's also something I'd be glad to help with doing at MIT as well (I should've already organized one recently...).  I could probably make it to Silicon Valley or London w/ some notice so long as it didn't interfere too much w/ teaching.


TeamCity Glyssen :: Glyssen pull requests [Build *.*.97.2f30fce2c1c88f023c85bebab2518aa8b9391068](http://build.palaso.org/viewLog.html?buildId=73611&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 2 (2 new), passed: 1046, ignored: 16) Build time: 00:07:50
### Failed tests
```
GlyssenTests.dll: GlyssenTests.BookScriptTests.SplitBlock_AttemptToSplitAtBeginningOfBlockThatIsNotPartOfMultiBlockQuote_ThrowsInvalidOperationException: <no details avaliable>

GlyssenTests.dll: GlyssenTests.BookScriptTests.SplitBlock_AttemptToSplitAtEndOfBlockThatIsNotPartOfMultiBlockQuote_ThrowsInvalidOperationException: <no details avaliable>

```


121132573-AirflowNetworkDistribution-outputs-for-node-pressure-and-flow-report-values-from-the-previous-timestep-when-the-distribution-system-is-off-#5687 (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2244 of 2247 tests passed, 0 test warnings)

 * 3 tests had: ESO big diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.87%25-yellow.svg)


@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.namedfile/20/violations
```
plone/namedfile/copy.py:0:1: I001 isort found changes, run it on the file
plone/namedfile/file.py:291:1: C901 'getImageInfo' is too complex (15)
plone/namedfile/scaling.py:173:1: C901 'DefaultImageScalingFactory.__call__' is too complex (13)
plone/namedfile/tests/test_scaling.py:108:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:109:70: S001 found % formatter
plone/namedfile/tests/test_scaling.py:175:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:176:70: S001 found % formatter
plone/namedfile/tests/test_scaling.py:183:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:184:72: S001 found % formatter
plone/namedfile/tests/test_scaling.py:191:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:192:72: S001 found % formatter
plone/namedfile/tests/test_scaling.py:243:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:244:70: S001 found % formatter
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Nit: It's `version.txt`, and you may want to specify the subdir, i.e. `xlutils/version.txt`.


For what it's worth, I liked the use of dates. It made it easy to tell if I was on a very old version of the library :)

What about writing something like "1.1.1 (released 2016-05-20)" - ie. both the minor version and the date.


The `not` is there since the beam should already be read in [before](https://github.com/keflavich/spectral-cube/blob/7b92facdb45d3e950d3be540f22f995886bdf063/spectral_cube/spectral_cube.py#L147) the units are checked. So this was to force trying to load the beam info if the units are "Jy/bm" and `read_beam=False`.

I guess it forces `read_beam=True` whenever the units are 'Jy/bm'. Is there a case when this isn't desirable?


plus device parser, will add myself.


nooooo, this breaks worldgen.


121132573-AirflowNetworkDistribution-outputs-for-node-pressure-and-flow-report-values-from-the-previous-timestep-when-the-distribution-system-is-off-#5687 (lgu1234) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


@gep13 the code provided by DarwinJS is an example of how he tackles the issue. Most of the code appears to be outside the scope for this function since we are only interested in retrieving the key. I think the only pertinent portion is we would want to add something like:

```powershell
if (([IntPtr]::size -eq 4) -and (Test-Path env:\PROCESSOR_ARCHITEW6432)) {
  Write-Debug "Detected 32-bit process running on WoW"
  # launch instance of 64-bit powershell.exe to get correct keys
  $startPosh64 = "$env:windir\sysnative\WindowsPowerShell\v1.0\powershell.exe"
  & $startPosh64 -Command {#existing code to enumerate WoW6432Node keys}
}
```

A possibly better, but more difficult solution, would require rewriting the cmdlets we are using in this function (Get-Item/Property) that are handling the Registry Objects so we can directly specify redirection in a powershell 32-bit process running in a WoW environment so the keys are read properly.


it doesnt do anything yet xD


That will fail if table name contains underscores.


checked by @Giorgiofox , tnx


@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.namedfile/21/violations
```
plone/namedfile/file.py:291:1: C901 'getImageInfo' is too complex (15)
plone/namedfile/scaling.py:173:1: C901 'DefaultImageScalingFactory.__call__' is too complex (13)
plone/namedfile/tests/test_scaling.py:108:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:109:70: S001 found % formatter
plone/namedfile/tests/test_scaling.py:175:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:176:70: S001 found % formatter
plone/namedfile/tests/test_scaling.py:183:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:184:72: S001 found % formatter
plone/namedfile/tests/test_scaling.py:191:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:192:72: S001 found % formatter
plone/namedfile/tests/test_scaling.py:243:32: S001 found %s formatter
plone/namedfile/tests/test_scaling.py:244:70: S001 found % formatter
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Do not use builders (BarBuilder, HistogramBuilder), because we do not use Pandas DataFrames therefore there is no advantage. Make APIs similar in Python and Scala again


yes: I want to instantiate a new `SpectralCube` object and assign its `beam` attribute after the fact.  But I'll revert the change and find another way to achieve this.  Might be helpful to have some comments about `read_beam` though.


conservation-tests (rraustad) - i386-Windows-7-VisualStudio-12: OK (2210 of 2213 tests passed, 8 test warnings)

 * 9 tests had: ERR diffs.
 * 3 tests had: EIO diffs.
 * 3 tests had: ESO big diffs.
 * 3 tests had: MTR big diffs.
 * 3 tests had: Table big diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.86%25-yellow.svg)


It adds worlgen stuff.


automatically?


Thanks, I haven't been using the bot anymore, but nice to see it being maintained.


Nice!


Jeah jeah jeah jeah jeah jeah


last time we had to use biome tweaker


Is it not the right path to commit these in 0.15 branch and leave master at 0.14.x for a while until all bugs/fixes are done for 0.15 ?


The following doc(s) have been deployed to the [1663tls](https://1663tls-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/ssl-tls](https://1663tls-static-docs.pantheonsite.io/docs/ssl-tls)


this function has also been migrated to an internal helper, so any fixes here should also be applied over there https://github.com/chocolatey/choco/commits/master/src/chocolatey.resources/helpers/functions/Get-UninstallRegistryKey.ps1


Nice. Reminds me of something I meant to ask the other day...

Isn't it a little weird (from the data's perspective) that we write timestamps to the kwik file the same way we would to the open ephys format (viz. in 1024 bit increments)? In some ways, it seems like it might make more sense to write one timestamp per sample... if they really were timestamps, that's what you'd want. Then again, since these are just sample numbers, there's at least as good an argument for omitting the timestamps array altogether. But I don't really see a good reason to store an array of consecutive multiples of 1024... is there one?


MoreDaylightingSensors (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (2213 of 2213 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


@manisandro: seems to be save :)
All builds are green: https://buildd.debian.org/status/package.php?p=gimagereader&suite=unstable


![image](https://cloud.githubusercontent.com/assets/5445550/16023068/e60e100c-31b2-11e6-9531-db54d1068ce4.png)
a world i made like 5 seconds ago


wtf? good catch! patch upstream?


This is nice, thanks


    worldgen {
        B:"Enables the world generation of the Bones! (Default = true)"=false


Good work!


TeamCity Glyssen :: Glyssen pull requests [Build *.*.98.749b02c8d0c10bab2de403cc07fda27c2f805d66](http://build.palaso.org/viewLog.html?buildId=73614&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 1255, ignored: 17 Build time: 00:07:34


@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.namedfile/22/violations
```
plone/namedfile/file.py:291:1: C901 'getImageInfo' is too complex (15)
plone/namedfile/scaling.py:173:1: C901 'DefaultImageScalingFactory.__call__' is too complex (13)
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


This won't work if your image file has non-equal voxel sizes. That is possible in MINC (via non-regular slice sampling), not sure NIFTI supports it.


No, since 0.15.0 is already out, it is tradition to do all in master branch


TeamCity Glyssen :: Glyssen pull requests [Build *.*.???.{build.vcs.number.1}](http://build.palaso.org/viewLog.html?buildId=73619&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **UNKNOWN**
Summary: Canceled Build time: 00:00:30


               "anchorCache": {
                             "cats": [
                                23
                        ],
                        "headers": {
				"X-Powered-By": "anchorCache(?:/([\\d.]+))?\\;version:\\1"
			},
                        "meta": {
	                        "generator": "anchorCache",
	                        "keywords": "anchorCache"
	                },
	                "html": "<!--[^>]+anchorCache®",
                        "icon": "anchorCache.png",
                        "implies": "PHP",
                        "website": "www.anchorcache.eu"
                },


always use strict equal `===`


`const` instead of `var`


please no space between key and `:`.

eg.

```
{id: 'foo'}
```

instead of

```
{id : 'foo'}
```


Yeah xD


A check could be added to look for a beam in `self.meta` first, then avoid calls to `self._try_load_beam` if it finds one.

Setting `self.pixels_per_beam` would have to be moved from [here](https://github.com/radio-astro-tools/spectral-cube/blob/master/spectral_cube/spectral_cube.py#L253), but maybe it's more appropriate to as a property.


IMO something that represents the information below could be very helpful.  For example,

```
                +--------------+  +-------------+
        +------>+  Arithmetic  |  | Comparable  |
        |       +   (+,-,*,/)  |  | (==,<,>,...)|
        |       +-------------++  +---+---------+
        |                     ^       ^
+-------+------------+        |       |
|  SignedArithmetic  |      +-+-------+----+
|     (unary -)      |      |  Integer     |
+------+-------------+      | (words,%,...)|
       ^                    +----+---------+
       |         +-----------^   ^     ^-----------+
       |         |               |                 |
+------+---------++    +---------+-------------+    ++------------------+
|  SignedInteger  |    |  FixedWidthInteger    |    |  UnsignedInteger  |
|                 |    | (bitwise,overflow,...)|    |                   |
+---------------+-+    +-+-----------------+---+    ++------------------+
                ^        ^                 ^       ^
                |        |                 |       |
                |        |                 |       |
               ++--------+-+             +-+-------+-+
               |Int family |-+            |UInt family|-+
               +-----------+ |            +-----------+ |
                 +-----------+              +-----------+
```


“allows using *mixed* integer types in generic functions?"


would be nice to also have the `!` version, so that the original hash is modified instead of only returning a new one.


s/like, for example/such as/


don't access the store from render methods directly, use `this.state.activePage`, example how to populate it:

https://github.com/mfischbo/couchdb-fauxton/blob/master/app/addons/cluster/cluster.react.jsx#L23-L31

later, usage in render method:

https://github.com/mfischbo/couchdb-fauxton/blob/master/app/addons/cluster/cluster.react.jsx#L53


one “the” too many: s/the concrete/concrete/


2 spaces


“The proposed model for interoperation between integer types eliminates…"


@innir Great, thanks!


What's this? :) Feel free to do a pull request if you want to improve upon light!

I recommend you first create an issue, then do a fork with a commit that closes said issue, and then a pull request from that point in time.


kill the comma


I'm not sure if the additional permutations in the "normal" tests are overkill. If you want, I can just make a PR for the precision tests.


With semver, would it be more appropriate to label it 3.0.0-alpha, or something of the sort?


Nice work there


@remotevase


develop (Unknown) - x86_64-MacOS-10.10-clang: OK (2381 of 2434 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-97.82%25-yellow.svg)


Good, but edit the footer.
1. website not by you
2. It still has the loren ipsum dolorum


“concept previously used to interoperate between integer types (see `toIntMax` in the current model) and instead provides access to machine words."


Lines 773 to 778 seem to be an unintended duplicate of lines 766 to to 771.


Thanks for cleaning that up. I think we could go a bit further here and handle this directly in the templates. Please check out experimental branch 554b1f2f216f874011792a402090ab33eb6cd396


you don't need to map/loop here if you name the labels like the radiobuttons, right?


The `SignedArithmetic` protocol is for numbers that can be negated.


its a template


Cool :)


(created by me, u know what i mean)


It provides a suitable basis for arithmetic on scalars such as integers and floating point numbers.


Ahh this is actually fixed in https://github.com/discourse/discourse/commit/1fe499e893eceb07294b0adf6edd97c8c41798a6.

The only case where `posts.id` is `nil` is when it is a reflection so we don't need to check for the ids here.


finsih foot later


ok


Entering an underscore or asterisk in the middle of a word (i.e. preceded by non-white space) still results in double characters - should this perhaps only happen at a word boundary?  Use case - adding a todo item for a function name, I type "fix function some_function_here" and each underscore is doubled when I type it. Is there a way to disable this?


consider renaming this variable to something more meaningful.


drop the final “the"


If you want this to be updatable, via `glbl.dark_window`, then remove this argument entirely and evaluate `period = glbl.dk_window` in the body of the function. As written, this will grab the current value of `glbl.dk_window` at import time but never update that value if `glbl.dk_window` is ever changed.


this.sate.activePage is per default undefined, no need to explicitly set it undefined


add “the” at the end of the line.


hmm, might be, but it doesn't do anything else, so it isn't needed.


Каковы наши успехи на данный момент?


drop the comma.


These initializers allow us to construct values of one type from


drop the first comma


You might as well remove this arg. Since this function is to be called by the RunEngine, not by the user or any xpdAcq function, it's reasonable to just assume `dark_dict_list` is always `glbl.dark_dict_list`.


You've lost the aspect ratio on the image.


What happens when the checks fail?


This function is good but can be improved. Consider the order in which the statements in this function would run and see if you can re-order it to save on computations and bail out earlier.


+1 for making this an argument -- that will make testing easier


Having docs or just having travis build your docs?


“respectively” is wrong here; you’d need a pair to match up with maximum/minimum.

Why not just say “clamp the result to the representable range of `Self`?"


Issue #25


In case you are interested:
new RTMPdump version taken from http://git.ffmpeg.org/rtmpdump (where previous one was also taken from)
http://mirrors.kodi.tv/build-deps/sources/rtmpdump-fa8646daeb19dfd12c181f7d19de708d623704c0.tar.gz

new zlib:
http://mirrors.kodi.tv/build-deps/sources/zlib-1.2.8.tar.gz


121132573-AirflowNetworkDistribution-outputs-for-node-pressure-and-flow-report-values-from-the-previous-timestep-when-the-distribution-system-is-off-#5687 (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1097 of 1097 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-34.0%25-green.svg)


s/refers to the type that// s/an/the/


No paragraph break here.


s/who’s/whose/


s/adds/protocol adds/


@Daniel15 then just look at the [release tags](https://github.com/eligrey/FileSaver.js/releases)

A semantic version number is better for npm and bower, and also for keeping it up to date with what can be/is a breaking change patch or a feature


Your solution looks solid - But remember to commit after each and every test you pass. It's good to get in the habit of making good commits often.


You need a section somewhere describing masking bitshifts and their motivation.


Indent code


I believe I fixed a different error ;)

I was seeing errors in `tl.post.user.username_lower` where `user` was nil ;)


you could pass down props from the parent component instead. right now the component renders 2 times. i'll explain when we walk through :)


Is this worth it?


`If elses` should be formatted like this:

```
if {
     ...
} else {
     ...
}
```


Hi @fsfrazao , does the new urlcode.py works? I run it on my laptop and comes error, is there something I should notice about the new input in urlcode.py?


Remove console.logs from commits


All escapes start with a `["\\"]`, so it may be more readable if used as a common factor.


I don’t know what the above is supposed to contribute to the proposal.  Why wouldn’t you just write out the code?


Looks nice indeed.
Should we link all names to their GH accounts maybe?


what does this line do?


Minor syntax issues - but overall - your solution is awesome. I have been seeing huge improvements in your problem-solving ability and attitude. I'm seriously impressed with how you have been progressing in the course. KEEP IT UP DOOD!


consider new names for these variables.


compress_embedded_files (kbenne) - x86_64-MacOS-10.10-clang: Tests Failed (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


You noted on GChat that you have to call `_validate_dark` both here and in `periodic_dark` above. You are right to think that this is a little awkward. We should probably should have include this code as part of `periodic_dark` itself. So, this function could be delete (along with the line in `CustomizedRunEngine` where it is applied. And the logic should be incorporated in `periodic_dark`.


Love this comment - v clean. Your message could be a little more descriptive - Tell me what you're cleaning up.


/cc @jcredding


I would recommend matching `op_add` and `op_sub` inside of the block since right now, `left: _op_expr()` is not optimized and will be called twice in the case of `op_sub`.


Looks like I lost the original numbers, but it's one alloc per row, which certainly shows up on both the alloc and time parts of the Insert and the Update benchmarks


Perhaps you could consider writing a private function for creating a new node since you use it in a couple of different places in your module.


Same here.


Great work 🎆 ! Very clean and clear

Blanket critique: Good syntax and formatting but there are a few places where the indentation can be improved, please update. Otherwise, good work 👍


The proposal should include a reference to your implementation of `DoubleWidth` that demonstrates that effective generic programming is now possible, showing how it can be used to implement `Int128` and `Int256`.


Remember to use `use strict` mode


And here with `unary_dec` and `unary_inc`.


On Sun, Jun 12, 2016 at 11:40:03AM -0700, LA wrote:
> libavcodec/videotoolbox.c still has references to h->pps:
>
> > libavcodec/videotoolbox.c:107:35: error: no member named 'pps' in 'struct H264Context'
>     memcpy(p + 4, h->pps.data, h->pps.data_size);
> > libavcodec/videotoolbox.c:109:17: error: no member named 'pps' in 'struct H264Context'
>     p += 4 + h->pps.data_size;

Should have been fixed by ubitux in aafdad1b454c1656e6ba08822de49fb7caf924f3
and 4df944920c2b92686751fa44e4ff7429eea1ff28

--
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Frequently ignored answer#1 FFmpeg bugs should be sent to our bugtracker. User
questions about the command line tools should be sent to the ffmpeg-user ML.
And questions about how to use libav* should be sent to the libav-user ML.


Have this a documentation?


this comment and the one below are slightly inaccurate because nothing is being copied here


Stretch Goals: Make a new branch: Doubly Linked List and modify your solution.


The third annotation (critical remark, attached to `accidental` in voice one) does not make it to the console log or export docs for me.


Stretch Goals: Make a new branch: Circular Linked List and implement it.


Yes tradition for pocketmine coders :) But the real coding standard says otherwise  :) (Never break master code (stables/beta's), development branches are there to break things.
So people can decide what branch they want to use.
Like me, I am still at 0.14.2, i a not upgrading any device/win10/server version.
I whised that you guys where working on the crafting problem intrduced in 0.14.3 so i can go to 0.14.3.

What is a 0.15.0 server without the features of 0.15.0? I bet we have to wait very long before any 0.15.0 MCPE feature is implemented in any opensourced pocketmine version. If you guys work very hard on that and be the first on creating all the features then youre the best of all opensourced clones.

Why go to 0.15.0 if many 0.14.x features are still not implemented and a lot bugs are still existing to be solved. Work on that instead changing protocol numbers and useless readme changes :)
When all 0.14.x features are working, then work on 0.15 features and merge 0.15 to master branch when there is actualy a feature implemented :P


I remember that a post will not have a user only when the user has been deleted as a spammer.


Exactly ;)


This looks like a copy-paste artifact.


ip-table-conversions (nealkruis) - Win64-Windows-7-VisualStudio-12: OK (2207 of 2213 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.73%25-yellow.svg)


Given that you deal with scope at the end of the document, I don’t think this part belongs here at all.


You can just send the string if `record->event.pressed` is true and break out of the switch statement unconditionally after that.


Did you check this even compiled when merging?


This line is unnecesary since you included it in the devDependencies section of package.json
:+1: :camel: == Nice camello
PD: just testing github comments


Is that depending on this commit?


Hab eben rumprobiert aber es noch nicht zum laufen bekommen, morgen mehr dazu.


True story. Done and done.


@clarkepaul Can you please fix the `discribes` and `massage` spelling throughout the commits?


This is breaking change, can we introduce private_settings for backward compatibility for short period of time (weeks with warning to update the code base) ?


this is just a difference in range value/indentation.


Nice use of template literals.
:+1: :camel: == Nice camello
PD: just testing github comments


I am ashame


Great to have some input on this. Note that in classes with the attribute `byte_data = True`, the data comes in as single bytes with the real part packed into the first four bits and the imaginary part packed into the second. Maybe we can move this code into the reference implementation (`ReferenceSqAccumMixin`) where data comes in as complex floating point numbers.


I think it might be the `if (alive _unit) exitWith {};` I added in serverPlayerDied.sqf along with custom messages. I'm gonna remove it to see how it goes.


GroundDomainUpdate (mitchute) - x86_64-MacOS-10.9-clang: OK (2200 of 2207 tests passed, 547 test warnings)

 * 552 tests had: AUD diffs.
 * 4 tests had: EIO diffs.
 * 5 tests had: ESO big diffs.
 * 4 tests had: Table big diffs.
 * 3 tests had: ESO small diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.68%25-yellow.svg)


Well, my `void-0.7.1` is installed with `semigroups-0.17` and lots of things depend on `void`, so if I reinstall it, I break everything.  Is the 0.18 lower bound necessary?


Few comments just from looking at the code and not actually trying it out. should all of the com sections be 'ge' and all the typ sections 'e'? Also can you etther add in comments line on what "thing" is or use something more descriptive? PO and I have had some troubles at figuring our what variables actually mean.


120796739_ObjListAutoPopulate (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1655 of 1661 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.64%25-yellow.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.78%25-green.svg)


This makes the output of "cc -h" ugly


121132573-AirflowNetworkDistribution-outputs-for-node-pressure-and-flow-report-values-from-the-previous-timestep-when-the-distribution-system-is-off-#5687 (lgu1234) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


@nortxort self.do_set_braodcast_pass should be self.do_set_broadcast_pass.


Why not simply `isSpace c`?  Newlines have been handled in the case before already.


TeamCity NEventStore :: MongoDB :: Build [Build 5.3.5-PullRequest.36+3](http://build.prxm.it:8813/viewLog.html?buildId=3530&buildTypeId=nes_mongo_build&guest=1) outcome was **FAILURE**
Summary: Error running GitVersion Build time: 00:02:00


You can cause the bot to flood and either 1) get it autokicked/banned, or 2) force it to quit due to excess flood.
It's basically a denial of service vulnerability, and there really is no good reason to leave it in.


Also `elem c [a]` can be written simply `c == a`.


sos grande sabelo +10 y a favoritos


τον controller του component των βγάζεις εκτός function ... ακόμα και σε διαφορετικό αρχείο αν χρειάζεται μόνο αν γίνει τεράστιος δεν κερδίζεις τίποτα υπερκατακερματίζοντας τον κώδικα


we might need an equivalent `isJava` / `isScala` on `Path`. These incoming queries could be entries in a jar file (imagine the user opens up sources of a dependency and asks for type at point in that file)


Sad.  Were these really needed?  We tried to avoid this, and use `j-expr(j-paren(...))` if we really wanted parens


The build has passed now.


παράδειγμα πως γίνονται bind οι τιμές


there are no guarantees `toFile` will succeed, we'll need to be careful everywhere we call it.


Well, there may be abother fix. But this is a fix, at least.


στο $onInit παίρνουμε ότι δεδομένα θέλει το component εκτός αν έρχονται από το parent component που τότε τα μεταφέρουμε από το ένα component στο άλλο ως binding variables


This must be `add_definitions("-Wno-tautological-compare")` for clang to work.


`toAbsolutePath` is also suspect. I don't know how this API would handle a jar reference to be honest.


All typ sections should be "e", but all the com sections should not be the same. Good catch. I copied too much.

So, I struggled with coming up with name for thing because thing can really be anything. I feel like commit message for e890af851e02ae1 makes it clear what is going on. If you have any better variable name ideas, let me know.


What are these `var`s for, here and below?


what about make release ?


That's a cute busy-loop ;-)


`AbstractFile` or one of its siblings can definitely handle entries inside jars.


@Blaisorblade followed the recommendation in https://ghc.haskell.org/trac/ghc/wiki/Migration/8.0#Librarychanges. I don't know if the 0.18 lower bound is necessary.


Just noticed that these tests should avoid depending on order.


bien vu, merci !


Is this just a test of tuple-bindings and tuple-constructions?  Seems odd not to just reuse `tup` in the next line...


Do as you see fit, I guess. Just thought I'd bring it up.


With above change the code now compiles without trouble.

Yet, when it comes to linking I do see this:
```
[ 81%] Linking CXX static library libjsond.a
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: libjsond.a(JSONAllocator.cpp.o) has no symbols
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: libjsond.a(JSONMemory.cpp.o) has no symbols
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: libjsond.a(JSONNode_Mutex.cpp.o) has no symbols
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: libjsond.a(JSONPreparse.cpp.o) has no symbols
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: libjsond.a(libjson.cpp.o) has no symbols
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: libjsond.a(JSONAllocator.cpp.o) has no symbols
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: libjsond.a(JSONMemory.cpp.o) has no symbols
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: libjsond.a(JSONNode_Mutex.cpp.o) has no symbols
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: libjsond.a(JSONPreparse.cpp.o) has no symbols
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ranlib: file: libjsond.a(libjson.cpp.o) has no symbols
[ 81%] Built target libjson
```


LGTM


Also, there are more comments that help explain everything under the section for ge than for gea, geo and geao. I didn't want to copy and paste the comments 4 times though.

Really, the similar code for these 4 commands should all be built into a function somewhere to avoid repetition. Again, more work to make it look pretty.


Good work!


Yeah I was a bit confused about which sections have the 4 bit packed data
vs which sections have the complex64 data.
This is only for the reference mixing part. This code is slow. I'll move it
there.
Thanks!

-- Shriharsh

McGill Space Institute

On Mon, Jun 13, 2016 at 5:55 PM, Kiyoshi Masui <notifications@github.com>
wrote:

> Great to have some input on this. Note that in classes with the attribute byte_data
> = True, the data comes in as single bytes with the real part packed into
> the first four bits and the imaginary part packed into the second. Maybe we
> can move this code into the reference implementation (
> ReferenceSqAccumMixin) where data comes in as complex floating point
> numbers.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/CHIMEFRB/ch_L1mock/commit/351c77ece1d12d1a4c7e95442790fd02c9698228#commitcomment-17851622>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AMzBAT_vshqxHprL49qI_rujDxZ3Eji4ks5qLdHbgaJpZM4I0xlZ>
> .
>


same unnecessary 'left'


scholarly.annotations.inp is obsolete. It was a fake file *not* exported from lilypond.

The current annotate.tex is *correct* in its use of annotate.annotations.inp, which is the file in scholarly/usage-examples which results from annotate.ly. The most recent commit to annotate.tex doesn't use scholarly.annotations.inp anywhere (as far as I can tell).


Accidentally pushed changes meant for another repo. This commit reverts those changes.


Looks good


ip-table-conversions (nealkruis) - i386-Windows-7-VisualStudio-12: OK (2207 of 2213 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.73%25-yellow.svg)


Commit


TeamCity NEventStore :: MongoDB :: Build [Build 5.3.5-PullRequest.36+4](http://build.prxm.it:8813/viewLog.html?buildId=3531&buildTypeId=nes_mongo_build&guest=1) outcome was **FAILURE**
Summary: Error running GitVersion Build time: 00:02:02


Thank you Shawn

We're also looking at conversejs.org   You're probably aware of it.  is
there a reason you'd recommend LetsChat?
Will anyone be maintaining the master going forward, or would that fall
on us?

On 6/12/2016 7:58 PM, Shawn Gustaw wrote:
>
> @shoqvalue <https://github.com/shoqvalue> master should be working
> fine. We still use it at work on a daily basis.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/sdelements/lets-chat/commit/2e5c8578cc4bd3785dfacbeef1ce8ec6c7cbf7a3#commitcomment-17837079>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AJy_7s4URawMYB5LJ2o2VxBio-xm0RZnks5qLJ0NgaJpZM4Iz5t1>.
>


Yes. This condition in quotes "if (skinnedMesh != nullptr" resolves to true before skinnedMesh is loaded because it has 0xcdcdcdcdcdcdcdcd as it's address not 0x0000000000000000 which makes it not equal to nullptr.0xcdcdcdcdcdcdcdcd is the address assigned to skinnedMesh on my machine when it's declared.After it is initialized ( I guess when it is loaded)it has 0x00000192a0617170 as it's address  which is fine.The function: "virtual void getOverlayText(VulkanTextOverlay *textOverlay)"    is called once before it is loaded apparently.So setting skinnedMesh to nullptr at declaration is preventing your condition statement from being fooled.
I'm just sharing how I understand it in case I need corrected, I'm sure you have it figured out.
On my machine a variable or pointer variable seems to be whatever is already there in memory initially, tillsomething is assigned to it. My AMD my be different from someone else's Intel; this may not happen on every system. Not sure.
Anyway thank you for responding. Your samples are the best I've seen and I'm learning a lot from them. I wanted to at least share this problem and fix with you hoping that it was of help and a time saver. I know your busy.There is a time lag because I live in the U.S.A.
Thank YouNelson

    On Monday, June 13, 2016 7:22 AM, Sascha Willems <notifications@github.com> wrote:

 That's odd. This should work fine. Does it crash because the skinned mesh is not loaded?—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Assuming you mean "have an eachLoop that works on MutableStringDicts" -- you need to be careful about concurrent modifications.  Typical solution for that is to equip the MSD with a "version counter" that increments on every mutation, and then in `each` cache the current version number, and check on each iteration whether it no longer matches.


@ccx-testerbot runtests master


Thank you for the request. It has been added to the queue (id: 238). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/238) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


I have to add the the links from the home page, i need help adding a blog, any one have suggestions/ code to share? Thanks!


Note, that `Store` [exposes observable as a symbol](https://github.com/reactjs/redux/blob/master/src/createStore.js#L251).


PS: Looks like one of these duplicate lines is already visible at zoom=3, so probably an issue in the map creation?


OCD triggered


#5516-Heat-recovery-from-multi-speed-dx-coil-does-not-appear-to-balance-with-water-heater-source-energy (rraustad) - i386-Windows-7-VisualStudio-12: OK (2213 of 2213 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Hello,

How do you install this plugin (SMS/MMS Display Attached Media) into OpenVBX?

Thank you in advance,
Austin


Which `.dat` file? They already exist in the [`/dat`](https://github.com/libretro/libretro-database/tree/master/dat) folder. This just updates the .rdb files to the latest, except for ScummVM, which is fixed by https://github.com/libretro/libretro-database/pull/167 .


This, assuming it works correctly, only partially solves the issue you note and wish to resolve. Chars such as \t, \n, ... will remain in the middle of the string. We need a better way to resolve  this issue. Regardless we will need the _strip_whitespace_ function and I added it to its own separate file for modularity sake and the fact I wish to add more utility functions latter.


/cc @jcredding


Um...write in comments when you're done map work, please.
Then I will readd my changes... :construction_worker:


Yeah, having a reference implementation is *super* important IMO. I encourage you to write a unit test for it (in ch_L1mock/tests/test_L0.py). That will facilitate debugging. If you aren't super familiar with the python unit testing framework, let me know and I can get one started.


121319197-Replace-a-field-name-of-Controlled-Zone-Name-in-two-objects-#5664 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2247 of 2247 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


this should probably use `goog.object/get`?


customizable favorites LIST ^^


Our build is broken now because of change in NewAuth API signature.


The following doc(s) have been deployed to the [1632newre](https://1632newre-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/debug-mysql-new-relic](https://1632newre-static-docs.pantheonsite.io/docs/debug-mysql-new-relic)
- [docs/alternative-php-cache](https://1632newre-static-docs.pantheonsite.io/docs/alternative-php-cache)
- [docs/application-containers](https://1632newre-static-docs.pantheonsite.io/docs/application-containers)
- [docs/new-relic](https://1632newre-static-docs.pantheonsite.io/docs/new-relic)


You can use tags - https://github.com/sendgridlabs/go-kinesis/tree/1.0.1


@Nedelosk this code crashes, even after I tried fixing some things, so I reverted it for now. After you fix it, please make a PR so I can review. Do not merge a big feature without review, it goes straight to CurseForge right now.


Wow, that is fast :open_mouth:


Hmmmmm, I like the idea as a sort of interactive protection, but I feel like the number of problematic strings created by explicitly changing the encoding is probably effectively 0 :/


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | 13/13 |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | 27/27 |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | 21/21 |
| _WTV | 13/13 |
| _XDS | 34/34 |
| _Options | **86/87** |
| _DTVCC | 14/14 |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](https://ccextractor.canihavesome.coffee/reports/238) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


121132573-AirflowNetworkDistribution-outputs-for-node-pressure-and-flow-report-values-from-the-previous-timestep-when-the-distribution-system-is-off-#5687 (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1656 of 1656 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.96%25-green.svg)


We definitely need "name". I'd rather see us get rid of "title" but unfortunately I don't think wagtail has an easy way to handle that.


121319197-Replace-a-field-name-of-Controlled-Zone-Name-in-two-objects-#5664 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


It is a bug.
I was certain that it is working as you say it should, strange.

> Is there a way to disable this?

You can set `"auto_match_enabled": false` in user PT settings file.

I will try to fix it.


uh


Cool! :)

I'll take a look at the speaker mixers tomorrow.

Can you also test without this patch: http://review.cyanogenmod.org/#/c/149129/
It's probably not needed with the new HAL, but we need to be sure.


@arco
Of course i can test!

Searching for speaker i found that Speaker DMIC Endfire is missing from our mixer paths.
Could that affect ?


You fixed wrong method.


`handle => CryptoNative_CmsGetEmbeddedContent(handle)`.  You bound it to the local variable `cms`, not the delegate input `handle`.


I'm seeing output like

```
make test
protoc -I /usr/local/bin//../include -I. -Ithird_party/googleapis --plugin=bin/protoc-gen-swagger --swagger_out=logtostderr=true,Mgoogle/protobuf/descriptor.proto=github.com/golang/protobuf/protoc-gen-go/descriptor,Mgoogle/api/annotations.proto=github.com/gengo/grpc-gateway/third_party/googleapis/google/api,Mexamples/sub/message.proto=github.com/gengo/grpc-gateway/examples/sub:. examples/examplepb/echo_service.proto examples/examplepb/a_bit_of_everything.proto
protoc -I /usr/local/bin//../include -I. -Ithird_party/googleapis --plugin=bin/protoc-gen-swagger --swagger_out=logtostderr=true,Mgoogle/protobuf/descriptor.proto=github.com/golang/protobuf/protoc-gen-go/descriptor,Mgoogle/api/annotations.proto=github.com/gengo/grpc-gateway/third_party/googleapis/google/api,Mexamples/sub/message.proto=github.com/gengo/grpc-gateway/examples/sub:. examples/examplepb/echo_service.proto examples/examplepb/a_bit_of_everything.proto
go test -race github.com/gengo/grpc-gateway/...
examples/examplepb/a_bit_of_everything.pb.go:11:8: cannot find package "google/protobuf" in any of:
	/usr/local/Cellar/go/1.6.2/libexec/src/google/protobuf (from $GOROOT)
	/Users/tmc/go/src/google/protobuf (from $GOPATH)
```
since this commit


fix


Add_Generator_MicroTurbine (jmarrec) - x86_64-MacOS-10.10-clang: OK (2381 of 2423 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-98.27%25-yellow.svg)


Adding 2 new files :

- script.cpp
-     script.h

Added in the following commit


I see the issue under speaker-dmic-endfire

<ctl name="SLIM TX8 MUX" value="1" />

should be

<ctl name="SLIM TX8 MUX" value="DEC3" />


ok i can edit (i think) the mixer path and add this
(I check the platform.c file to check the names and edit them like others.shouldnt be hard)


Access modifiers are required.  Presumably you want `internal`.


Yea you just need to change the value. I don't think anything else needs to be changed.


@dookiedude  just edit ur commit...codes are not visible in github


Haha yea I noticed lol. edited out the < so it would show the lines


the original value is ZERO not '1'


Actually, the Create Table shouldn't have LatestEpisodeAirDate since its added by the updated schema.


"" => string.Empty (when possible)


This method of dispatch_operation is raising the error: org.freedesktop.Telepathy.Error.ed NotImplemented: The requested Handler does not exist

If I set bundle to `None` instead of `bundle = registry.get_bundle(bundle_id)` in ActivityInvite (for collabing an activity) the same error is raised!


Add more game explanation.


not the original value the one under path name="speaker-dmic-endfire"


Congratulations @diml !
There is no hurry for the expect tests: we can do the move gradually.
The change was essential for typing-gadts (the combined output with -principal is great), less so fo others.


MoreDaylightingSensors (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (2213 of 2213 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


oh yea...sorry.trully worked :D


Delete this comment, it's covered by the summary.


Since you're not masking anymore, the name `relevant` no longer seems... relevant.  How about `tag`?


@arco should I replaced the proposed change on gerrit with your mixer_paths? Since its working yours is much cleaner anyhow.


@andreasabel semigroups-0.17 + GHC 7.8.4 seems fine here, even after cf87310695046d90536a3092baa9af146854cfca which merged the *clients* of the new package.

But I very much don't guarantee that lowering the bound will actually test OK on all platforms, I just think it should.


style.css
1) text decoration of .trackListLabel class


Explain your commit please!


@RonDrabkin you can marshal json in this case by using golang's built-in `json` encoding. An excellent example is available [here](https://gobyexample.com/json). Your code might look something like:

```go
params := map[string]string{
  "nces_id":"BB981479",
  "name":"Woodlands Christian Academy",
  "address":"5900 Academy Way",
  "city": "The Woodlands",
  "state": "TX",
  "zip": "77384",
  "low_grade": "K",
  "high_grade": "12"
}
jsonParams, _ := json.Marshal(params)
client := &http.Client{}
req, _ := http.NewRequest("POST", "https://partner.opened.com/1/schools", &jsonParams)
req.Header.Add("Content-Type", `W/"application/json"`)
token,err := opened.GetToken ("","","","") //needed
resp, err := client.Do(req)
if err != nil {
	fmt.Println("Errored when sending request to the server")
	return
}
defer resp.Body.Close()
resp_body, _ := ioutil.ReadAll(resp.Body)
fmt.Println(resp.Status)
fmt.Println(string(resp_body))
```


Speaker started working? @dkati


Why?


@dookiedude  yes it did..now i got to control some volumes since are too low....

Thanks everyone...REALLY thanks


Great addition!


We need both HOST and PORT for openshift's start script (check it out)


See OpenSslX509CertificateReader::ExtractValidityDateTime.  Copy the date parsing code from the "if length 13" block.


I think having a required "title" field is a side effect of Wagtail wanted everything to be a Page instead of some free-floating Author Entity. But I do think this gives us the flexibility to have pages titled something other than the person's name.

If it's less confusing, though, I'd support re-designating this field to something like "author_name" or giving it some help text.


Can't use this solution outside OPENSHIFT (I think we better keep all openshift's ENV vars inside OPENSHIFT's action_hooks)


The blank line is good.  But the spaces on it are bad. Please delete all the spaces on this line.


@dookiedude Yeah that's probably better as a start. The other one has a lot of stuff that's not relevant for msm8960 devices.


I always keep empty line between:
PART1: Standard library imports

PART2: Third party modules

PART3: User made modules


If the formatter isn't fixing this, you should add the space here by hand.


是的，已知bug，我还没有搞清楚原因。


[![Study for the Assumption of the Virgin](http://api.artsmia.org/images/1399/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1399)


[![The Glorification of Andrea Doria](http://api.artsmia.org/images/1489/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1489)


[![Duchesse d'enfant brisÃ©e (childâs day bed)](http://api.artsmia.org/images/9433/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=9433)


[![Massacre of the Innocents](http://api.artsmia.org/images/10235/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=10235)


[![Bib-fronted Band](http://api.artsmia.org/images/123546/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123546)


[![Two Jabot Ends](http://api.artsmia.org/images/123547/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123547)


[![Duchesse d'enfant brisÃ©e (childâs chair)](http://api.artsmia.org/images/90349/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=90349)


[![Bear: Glittering](http://api.artsmia.org/images/117088/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=117088)


#5704-TableOneIndependentVariable-documentation-doesn't-agree-with-behavior (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (2219 of 2219 tests passed, 2 test warnings)

 * 1 test had: ERR diffs.
 * 1 test had: EIO diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Is this needed?


Either remove the first "the", or add "method" (or "function") before the comma :)


Can anybody please explain what is going on in QueryProperty class? I would like to understand what type of object is returned by `self.sa.session()` on line 125. Thank you.


Lets make it more flexible for the user. Show as many as the user wants. We can use a GET params for this combined with four buttons [5, 10, 20, 50]


the/method.


Why lazy loading this?


accepted


we may dont need it.i test now


You might need to manually patch this up.  `Unix Release` => `Unix_Release`


@arco you are my hero! :-) I can confirm that calls works fine, tested now!
@dkati take a look at my latest commit in this branch, mic volume it's ok for me, i've tested the rec app and a call, both are fine.. Haven't tried other things for the moment, tomorrow i'll merge your speaker change too!


TeamCity Mobile :: Android :: Android Develop (Nightly UAT) [Build 20](http://teamcity.hq.judo/viewLog.html?buildId=40683&buildTypeId=Mobile_AndroidDevelopNightlyUat) is now running


Does commit d4aeeedf5ba4f6fc help?


I think
```python
httpd = make_server(host, port, application)
```
is better than
```python
httpd = make_server('localhost', 8080, application)
```
because HOST and PORT can change


@arco I also have a question if we were to include

        ctl name="VEQ Enable" value="1"
        ctl name="ES325 2Mic Enable" value="1"

in the mixer_paths, but switching the acdb id temporarily to 34 can't be done from there, right? How would you be able to get around that?


121319197-Replace-a-field-name-of-Controlled-Zone-Name-in-two-objects-#5664 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1097 of 1097 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-33.97%25-green.svg)


restoreChatHistory这里的逻辑也不完善，没有处理scope.chatContent.length > 0 同时存在历史记录的情况。应该merge这2个Array。


@B--B  speaker volume is low i think.

my tests:
BT calls : perfect
music stream via bt : perfect
bt calls WHILE playing music from bt : perfect
loudspeaker on/off : works as it is supposed to

Now testing with reverted commit of es325 as @arco said so


Version is always 0?


121319197-Replace-a-field-name-of-Controlled-Zone-Name-in-two-objects-#5664 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Go ahead with your updates.


Why is this a reassign/out?.  That very much doesn't follow, and very very much does not match the successive lines.


@DarrenZDL , works fine for me.
Can you please provide the error message?

Is anyone else having trouble?


从群里单挑一个人聊天时，也会出现问题。scope 被重置了，需要更新引用。


compress_embedded_files (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


Yes, make sure everything is cleaned pls.


Sorry for missing that one earlier in #329


I'd probably call it OpenSslHelpers; but no biggie.


This needs a width


@chieakiba can you ignore `node_modules` when pushing to github?
In the future, things like that are usually not included. You may want to employ a `.gitignore` file.


These need to be floated left right and center


Move this to the calling function.  This method should only read out the version.


@shoqvalue never heard of it to be honest, so can't really comment. And it probably depends on your use-case. I would probably argue that LCB is super easy to deploy and pretty accessible for hacking on if you know some js.

There was a pretty big rewrite of the UI in React underway but I think people are busy with other things at the moment.


Don't see any other places using this removed GetBuildResults func, hence it should be safe to get rid of it. Looks good to me.


sss


That's just CheckValidOpenSslHandle...


CheckValidHandle?


compress_embedded_files (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


cleanup


esquece isto, estava me a dar warnings aqui mas era por causa dum ;


(If you expect null and expect the calling function to just handle that; add a comment saying so


isto idem aspas


Here should be resolved["name"], but not override the host!


TeamCity Mobile :: Android :: Android Develop (Nightly UAT) [Build 20](http://teamcity.hq.judo/viewLog.html?buildId=40683&buildTypeId=Mobile_AndroidDevelopNightlyUat) outcome was **SUCCESS**
Summary: Tests passed: 164 Build time: 00:09:44


oh my, I completely failed at RTFS. Carry on...


@fsfrazao, I figured it out. Thanks.


Add a comment here like
```C#
// origCertsPtr might be null.  But GetX509StackFieldCount will just return 0, so everything just works out.
```


Nit: Remove double-blank line


These byte variables could be declared `const`


Does `isDirectory` always traverse symlinks?  In fact, are symlinks handled at all by the FS proxy facility?  Either way, we should document the semantics.


Same comment as for `isDirectory`: what does this return if the path is a symlink that points to a nonexistent path?


constructedSet :)


I'd make this be a new variable, instead of a reassignment.  "contentInfo" was the outer sequence.  This is "encryptedContent".


@dookiedude No we can't temp switch to 34 using only mixer_paths afaik. If things still works without the ES325 patch, there's shouldn't be any need to. The klte and hlte devices doesn't, and they also has the ES325.


Hi Paul,

Will the license page be moved out of the wizard to first login?


`get to work around` => `work out`?


I'd add a line before this, like
// As we commented in the Windows implementation:


CheckValidHandle (or not...)


this requires ES template like :
`curl -XPUT http://_ESHOST_/_template/cuckoo -d {
  "template": "cuckoo-*",
  "settings": {
    "index.refresh_interval": "5s",
    "index.number_of_replicas": "0",
    "index.number_of_shards": "2"
  },
  "mappings": {
    "_default_": {
      "dynamic_templates": [
        {
          "strings": {
            "mapping": {
              "index": "analyzed",
              "omit_norms": true,
              "type": "string",
              "fields": {
                "raw": {
                  "index": "not_analyzed",
                  "ignore_above": 256,
                  "type": "string"
                }
              }
            },
            "match_mapping_type": "string",
            "match": "*"
          }
        },
        {
          "dates": {
            "mapping": {
              "format": "yyyy-MM-dd HH:mm:ssZ||yyyy-MM-dd",
              "type": "date"
            },
            "match": ".* time|time|@timestamp",
            "match_pattern": "regex"
          }
        },
        {
          "ips": {
            "mapping": {
              "type": "ip"
            },
            "match": ".* ip|ip",
            "match_pattern": "regex"
          }
        }
      ],
      "_all": {
        "enabled": true
      }
    }
  },
  "aliases": {}
}`


A redundant change? The parameter is already a `URL`.


line 280 and 283 above don't have the `>` character at the begining


seems like API calls upload is also buggy. It doesn't get pushed with results ? It gets pushed into index instead ?


Thanks! It certainly has been coming together. It's exciting that it's almost reached the feature set of v3.0.


why doesn't this include PATH_TRANSLATED in the output?


简化语法，修改部分不通顺的语句


I guess this is the result of the automatic bridging trying too hard? Should maybe expose these constant under a different name that doesn't get mutilated in Realm.Private.


AddOpeningAreaToIVRS (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2248 of 2248 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Ship it


Typo: unprotected


Is this disabling parallelizing build steps, or disabling compiling more than one thing at a time? The former is fine, the latter less so (especially since the change isn't xcode 8-only).


This should be within your `<main>`


@aziz is it for better appearance on OSX? we should put it in settings files I’ll do it

Windows 8.1:
![2016-06-14_05-21-02](https://cloud.githubusercontent.com/assets/1975298/16026369/e0c99606-31ef-11e6-80a4-ba041db9312c.png)


uh, i know you say that it only works on odd numbers, but as it's written j counts over all. so that's not entirely true.
Consider:
`isqrt=sqrt(i); for(j=3;j<=isqrt && i%j;j+=2);`


You understand ser vs estar.  I understand is vs are.  The attributes `are`, not is :)


#5516-Heat-recovery-from-multi-speed-dx-coil-does-not-appear-to-balance-with-water-heater-source-energy (rraustad) - Win64-Windows-7-VisualStudio-12: OK (2213 of 2213 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


"English please".  Your words, not mine!


Good question. How do the private npm modules get installed normally?

cc @JustinBeckwith thoughts?


I use a combination of date + an appended number for multiple releases on the same date. A date is a perfectly valid minor version number as long as it's in the right format.


Typo: sotred => sorted


AddOpeningAreaToIVRS (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


@khamukkamu Welcome to the most stupid workaround ever conceived. Be my guest.


Thanks.  I updated the instructions.  We'll see if that's clear as mud or not...
http://developer.rhino3d.com/guides/general/how_this_site_works/


NIt: blank line


Yes. I'm not really sure why it turned the string constants into values of `NSNotification.Name` type.


CheckInvalidHandle


You might want to add a comment that you're not searching the LM\My store because that's not supported on Unix.

I'm the one who doesn't support it, and I forgot that.


Hello Pavel,

Indeed.

Thank you.
BR


It should be the former. Without this flag set I was getting different build errors at different points (including errors that implied that code was being linked before the dependency it was being linked with was ready).


this method should belong to Player. So you would call player.play()


this should go into a Game object


Astuce :

var publicIP = process.env.EZMASTER_PUBLIC_IP || '127.0.0.1';


This should go into Game object


this should go into Game object


Probably just turning all externed const strings with names ending in Notification into NSNotification.Name. I think we can now override how it's bridged to Swift with one of the new features?


This should go into a UI class


this should go into a UI object


gh-pages


There _should_ be a way to override this feature, but nothing in the [proposal](https://github.com/apple/swift-evolution/blob/master/proposals/0033-import-objc-constants.md) indicates that it's possible :/.


But once a user has been deleted as a spammer, the associated post will be deleted and topic links are destroyed. Perhaps you can add a test case for the scenario you identified?

Furthermore, users.id and posts.id will never be null because those are primary keys.


The closest Windows equivalent is "An error occurred during encode or decode operation.".  But you can check that by causing one of the calls to CryptDecodeObject to fail :)


You shouldn't need this section; the Unix implementation doesn't need to build as a net46 fallback


Would be more consistent with the way the CLI is built to not use `quit()` (only the main CLI module uses it) error out and correctly capitalize the message:
```lua
error("you need to specify the node name to leave")
```


There is no `prefix` argument in your command according to the `lapp` string definition of it


Our normal style for inlined array declarations (etc) is a space of padding inside the braces


(Yay)


Newline plz