I think is better to use notify function, because it takes as parameter the changed row and the old row, you can do calculate the difference, and see what changed. I that way you remove the overhead of iterating over track_list dict.
You can change the function implementation at runtime:
idl.notify = custom_notify_handler


3.) malloc + free instead the vector<std::string>?


You only wanted to push the SharingButtonsViewController if the selection is in SharingButtons section, and keep SharingConnectionsViewController for the services section, right? Also, table cells should deselect when returning from a pushed controller, not on selection.


Fixes issue #13 too;
(also I meant to say CommonIdFetcher in the commit title)


I can build a 1.24 RC binary and upload it somewhere.


Line 47:

Same issues.  No documentation that will appear when a user types set_zp_range?.  Calls on columns of data that are defined outside of the function.  Function can't stand alone.  Doesn't specify data type of input or output.  Not comments to explain logical steps of function.


**HEAD commit has no message.**
*GitCommitBear, severity 1, section `commit`.* *Project wide.*


$tokenFile is bound to the scope of CheckToken, and not global.


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `docs`.* *Related to file `docs/Users/Tutorials/Tutorial.rst`, line 327.*

The issue can be fixed by applying the following patch:

```diff
--- a/docs/Users/Tutorials/Tutorial.rst
+++ b/docs/Users/Tutorials/Tutorial.rst
@@ -324,7 +324,7 @@
 -  The settings it uses (optional and required)

 There are two types of Bears which are there-
-- Global
+- Global
 - Local

 The Local Bear can analyze only one file at once. It can
```


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `docs`.* *Related to file `docs/Users/Tutorials/Writing_Bears.rst`, line 56.*

The issue can be fixed by applying the following patch:

```diff
--- a/docs/Users/Tutorials/Writing_Bears.rst
+++ b/docs/Users/Tutorials/Writing_Bears.rst
@@ -53,7 +53,7 @@
 The Bear so created is of the type LocalBear which can only read from a single file.

 To write a code which can read and extract facts from multiple files,
-A bear of type GlobalBear needs to be created.Here is a simple example
+A bear of type GlobalBear needs to be created.Here is a simple example
 for GlobalBear:

 .. code:: python
```


**Line is longer than allowed. (84 > 80)**
*LineLengthBear, severity 1, section `docs`.* *Related to file `docs/Users/Tutorials/Writing_Bears.rst`, line 53 column 81 to line 53 column 85*.


@Turbo87 you are correct, it should have been a `skip`


to clarify (from your message below), what do you mean by EMPTY. I was assuming you meant the empty() function:
``````
<?php if (!empty($pdrow['billing_note'])) { ?>


Um noch etwas in den Raum zu werfen: "Diese Maschine besitzt einen Kopierschutz, der noch nicht..."
Ist zwar etwas freier übersetzt, aber meiner Meinung nach doch recht eindeutig.

Ansonsten würde ich "Schutzmechanismen" bevorzugen.


Someone already [reported this](https://github.com/lepture/mistune/issues/80) a while ago.


Copy paste, though it doesn't matter since they're all in the same app. I will fix it though


This is incorrect renaming. `Class` means only `class` here. :)


PHP has kind of loose typing. It doesn't hurt to be specific with your conditionals, but yeah, it could be done with a '!'.


develop (lefticus) - i386-Windows-10-VisualStudio-14: OK (389 of 389 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 is incorrect as well.


good point


It's all good :D


This is not actually getting the preimage, but rather the hash


I'm guessing these are overridable in the host app, yeah? If I used this in a new app, and wanted sign_in_up to be "Log In/Log Up," I could just add the capturable.widget.sign_in_up key?


Gracias Carlos, perdón que vi ahora el comment.


s/the bear/they bear/


I will fix it Sorry

      From: Brady Miller <notifications@github.com>
 To: bradymiller/openemr <openemr@noreply.github.com>
Cc: Terry Hill <teryhill@yahoo.com>
 Sent: Tuesday, February 23, 2016 3:52 PM
 Subject: Re: [openemr] Billing Note change, take 2. (382a267)

to clarify (from your message below), what do you mean by EMPTY. I was assuming you meant the empty() function:<?php if (!empty($pdrow['billing_note'])) { ?>
—
Reply to this email directly or view it on GitHub.


Nope :-) hehe.

Make the m_strValue a real std::string and only use the c_str() at the very last occurence where you need it ... but this is most likely over the scope here and the c_str() is used to care for compatibility.


 ./clone_job.pl
Usage:
    clone_job.pl [OPTIONS] JOBID [KEY=[VALUE] ...]

    EXAMPLES:

    clone_job.pl --from https://openqa.opensuse.org 42

    clone_job.pl --from https://openqa.opensuse.org --host
    openqa.example.com 42

    clone_job.pl --from localhost --host localhost 42 MAKETESTSNAPSHOTS=1
    FOOBAR=

Options:
    --host HOST
        connect to specified host

    --from HOST
        get job from specified host

    --dir DIR
        specify directory where test assets are stored (default
        /var/lib/openqa/factory)

    --apikey <value>
        specify the public key needed for API authentication

    --apisecret <value>
        specify the secret key needed for API authentication

    --help, -h
        print help

* ./clone_job.pl
Usage:
    clone_job.pl [OPTIONS] JOBID [KEY=[VALUE] ...]

Options:
    --host HOST
        connect to specified host

    --from HOST
        get job from specified host

    --dir DIR
        specify directory where test assets are stored (default
        /var/lib/openqa/factory)

    --apikey <value>
        specify the public key needed for API authentication

    --apisecret <value>
        specify the secret key needed for API authentication

    --help, -h
        print help


https://stackoverflow.com/questions/15623569/rails-paginate-duplicates-entries

In the quick search some profiles were shown twice


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16896](https://teamcity.mesosphere.io/viewLog.html?buildId=142728&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:04:56


I wouldn't normally consider two lines to be DRY enough to build a whole function for it, but yeah, I can do that.


Wow, that's great!


Possible feature: have TCP-based syslog record the order in which messages are received.


For a title I was thinking "Shopping is hard, let's build a web app". The talk will touch on many of the things that are required to create a working web app.


Aber in der Emulation muss ich doch das Image/Abbild z.B. via Kommandozeile laden/als Parameter angeben und nicht irgendwo physikalisch einlegen, oder?


 - __information__: [src/chaiscript_stdlib.cpp:0](../blob/871ad10e0e5e65a160b2c8ae81fa695cc479cb07/src/chaiscript_stdlib.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 16 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/871ad10e0e5e65a160b2c8ae81fa695cc479cb07/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 677. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:683](../blob/871ad10e0e5e65a160b2c8ae81fa695cc479cb07/include/chaiscript/language/chaiscript_parser.hpp#L683) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/871ad10e0e5e65a160b2c8ae81fa695cc479cb07/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/871ad10e0e5e65a160b2c8ae81fa695cc479cb07/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 683. (multiCondition)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/871ad10e0e5e65a160b2c8ae81fa695cc479cb07/src/chaiscript_stdlib.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __information__: [src/main.cpp:0](../blob/871ad10e0e5e65a160b2c8ae81fa695cc479cb07/src/main.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 17 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __information__: [src/main.cpp:1](../blob/871ad10e0e5e65a160b2c8ae81fa695cc479cb07/src/main.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/871ad10e0e5e65a160b2c8ae81fa695cc479cb07/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/871ad10e0e5e65a160b2c8ae81fa695cc479cb07/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/871ad10e0e5e65a160b2c8ae81fa695cc479cb07/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/871ad10e0e5e65a160b2c8ae81fa695cc479cb07/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/871ad10e0e5e65a160b2c8ae81fa695cc479cb07/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


I will do a single folder then. Actually, I already did this, but not yet committed the changes.


This alias is not necessary because there were no such a property in 0.5.4.


ok so my "-libril" branches of device/vendor is what im working with. so far i have a working rild which is the main part for getting 6.0 but the libril doesnt work as far as i can tell. Ill do some more work on it because i know i had a semi-working one before and itd be nice to have as many lib/blobs opensourced as possible but yea for now rild is working as far as i can tell. I dont have service for this sim but it works as far as showing sim info in about device.(Which wouldnt happen if any of the ril/libril wasnt working). Now this is with a rom built with bionic hacks but obviously if what we have works then if it needs to be changed its we have the source so should be a matter of fixing it up to be compatible at this point, having something that works is a big start. note you will also need https://github.com/CyanogenMod/android_hardware_ril/commit/e3085f770de6e802aba09946b18547d70c054e9e depending on which cm version your building for.


I notice this commit uses relative keys, whereas the previous one uses absolute. Should we standardize on one or the other? What does the Devise gem do?


In Markdown, instead of `<hr>` you can do this:

```
- [ ]  All code checkins completed

---

- [ ]  Code review completed or waived
```


yes but it also might be a string, which would return TRUE every time it was not null.


I hadn't thought about using Set::Scalar here.  But I think shifting off a basic list is the least surprising thing to offer.  Reference counting works like you'd expect, and rewinding to replay prior data isn't something an iterator usually does.


This is a bit of a problem; if a timeout occurs we shouldn't discard ourselves from _THREADS. Unfortunately, join is singularly annoying in that it doesn't tell you whether a timeout occurred or not - you have to query Thread.is_alive after it returns to determine the result!


Sure. And if it was reported a while ago a pull request couldn't hurt, to speed up fixing :-)


Yes, thanks. This case is redundant. Removed the same.


(which is probably why I left timeout out of the overridden version - if we don't need it, why bother with the complexity)


Der Vorschlag gefällt mir fast am besten. Also wegen mir gerne mit
"Kopierschutz"

Am 23. Februar 2016 um 21:52 schrieb Lothar Serra Mari <
notifications@github.com>:

> Um noch etwas in den Raum zu werfen: "Diese Maschine besitzt einen
> Kopierschutz, der noch nicht..."
> Ist zwar etwas freier übersetzt, aber meiner Meinung nach doch recht
> eindeutig.
>
> Ansonsten würde ich "Schutzmechanismen" bevorzugen.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rootfather/mame/commit/ebf2ee3705279c7fc5ba30ecc708036c3ee99aab#commitcomment-16279737>
> .
>


`Top Level parent` -> `top level parent`


Also added confirmation page at the end of reservation wizard


Commenting it out helps during rebasing.  Otherwise the code that was removed reappears and there is nothing to indicate that it was explicitly removed


Without prefs the following tests throws NPE:
* upgradeExternalLinksMoveFromPdfToFile
* cleanupRenamePdfRenamesRelativeFile
* cleanupHtmlStripsHtmlTag
* upgradeExternalLinksMoveFromPsToFile

With prefs and without journalAbbreviationLoader only cleanupRenamePdfRenamesRelativeFile throws an NPE.

No idea why it happens as I do not really have changed anything which should (obviously) affect this...


Not sure if this is already documented somewhere but it might be nice to document the details of the value used as the preimage


Testing github API :)


package gmail.com.olga.romanko1988;

import java.util.ArrayList;
import java.util.List;

/**
 * Created by Olga Romanko on 22.02.2016.
 */

//created class Abiturient
public class Abiturient {
    private Integer id;
    private String Surname;
    private String Name;
    private String Patronymic;
    private String Address;
    private Integer Phone;
    private Integer Marks;
    private Integer Mark1;
    private Integer Mark2;
    private Integer Mark3;

    //constructor
   public Abiturient (Integer id, String Surname, String Name, String Patronymic, String Address, Integer Phone, Integer Marks, Integer Mark1,
                      Integer Mark2, Integer Mark3) {
       this.id=id;
       this.Surname=Surname;
       this.Name=Name;
       this.Patronymic=Patronymic;
       this.Address=Address;
       this.Phone=Phone;
       this.Marks=Marks;
       this.Mark1=Mark1;
       this.Mark2=Mark2;
       this.Mark3=Mark3;

    }
//getters+setters
    public Integer getId() {
        return id;
    }

    public void setId(Integer id) {
        this.id = id;
    }

    public String getSurname() {
        return Surname;
    }

    public void setSurname(String surname) {
        this.Surname = surname;
    }

    public String getName() {
        return Name;
    }

    public void setName(String name) {
        this.Name = name;
    }

    public String getPatronymic() {
        return Patronymic;
    }

    public void setPatronymic(String patronymic) {
        this.Patronymic = patronymic;
    }

    public String getAddress() {
        return Address;
    }

    public void setAddress(String address) {
        this.Address = address;
    }

    public Integer getPhone() {
        return Phone;
    }

    public void setPhone(Integer phone) {
        this.Phone = phone;
    }

    public Integer getMarks() {
        return Marks;
    }

    public void setMarks(Integer marks) {
        this.Marks = marks;
    }

    public Integer getMark1() {
        return Mark1;
    }

    public void setMark1(Integer mark1) {
        this.Mark1 = mark1;
    }

    public Integer getMark2() {
        return Mark2;
    }

    public void setMark2(Integer mark2) {
        this.Mark2 = mark2;
    }

    public Integer getMark3() {
        return Mark3;
    }

    public void setMark3(Integer mark3) {
        this.Mark3 = mark3;
    }

    @Override
    public String toString(){
    final StringBuilder sb = new StringBuilder("Abiturient ");
        sb.append("id = ").append(id);
        sb.append(", Surname = ").append(Surname);
        sb.append(", Name = ").append(Name);
        sb.append(", Patronymic =").append(Patronymic);
        sb.append(", Address= ").append(Address);
        sb.append(", Phone= ").append(Phone);
        sb.append(", Marks = ").append(Marks);
        sb.append(", Mark1 =").append(Mark1);
        sb.append(", Mark2=").append(Mark2);
        sb.append(", Mark3=").append(Mark3);
        return sb.toString();
    }

        public  static void main (String [] args) {
            List<Abiturient> abiturients = addAbiturients();
            for (Abiturient abiturient: abiturients){
                System.out.println(abiturient);
        }

private static List<Abiturient> addAbiturients {
        List<Abiturient> abiturients = new ArrayList<>();
        abiturients.add(new Abiturient()
                .setId(1)
                .setSurname("Ivanov")
                .setName("Ivan")
                .setPatronymic("Ivanivich")
                .setAddress("Kiev,Ukraine")
                .setPhone(12758)
                .setMarks(165)
                .setMark1(45)
                .setMark2(55)
                .setMark3(65)
        );

        abiturients.add(new Abiturient()
                .setId(2)
                .setSurname("Petrov")
                .setName("Petr")
                .setPatronymic("Petrovich")
                .setAddress("Rivne,Ukraine")
                .setPhone(78541)
                .setMarks(255)
                .setMark1(75)
                .setMark2(85)
                .setMark3(95)
        );

        abiturients.add(new Abiturient()
                .setId(3)
                .setSurname("Kuzko")
                .setName("Anna")
                .setPatronymic("Pavlovna")
                .setAddress("Lviv,Ukraine")
                .setPhone(12345)
                .setMarks(261)
                .setMark1(78)
                .setMark2(84)
                .setMark3(99)
        );

        abiturients.add(new Abiturient()
                .setId(4)
                .setSurname("Mishuna")
                .setName("Nadia")
                .setPatronymic("Nikolaevna")
                .setAddress("Lutsk,Ukraine")
                .setPhone(50514)
                .setMarks(85)
                .setMark1(25)
                .setMark2(15)
                .setMark3(45)
        );
        return abiturients;

    }}
}


The exploit is present for images too: `![text]("><script>alert`1`</script>)` (!! sign)


The super call admittedly doesn't do anything at the moment (given it's now `__new__` doing all the work), but that's not really a good reason to remove it?


You could make freezing message permanent (999 duration) then have a `ClearFreezeMessage` on the `RemoveFreezingIndicator` function, which does `Notifications:ClearBottom(pID)`

Also, you don't need to `ClearBottomFromAll()` in a single-player message (it will remove other peoples messages if you do it)


Should we take this opportunity to clean up long lines in the noscript messaging, or would that potentially confuse translators?


Squash multiple commits into one.


As long as this is being cleaned up, maybe put the last `</div>` on its own line, too?


So confrused!!


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.530.0](http://build01:8888/viewLog.html?buildId=14376&buildTypeId=ConfigInjector_CI) is now running


Absolutely :+1:


Is removing this ZH intentional in this commit? Or should we wait until we have a translation in place from Douglas?


Laurent, I think this line should be:

        'timezone': datetime_.tzinfo.tzname,


setTimeout is used to simulate the loading state (2 seconds) where previously the change in state lead to a jarring UI update.


The BCMA_CHIPCOMMON_MEM_RID is not defined

```
/usr/home/sobomax/projects/freebsd11.mips/sys/dev/bcma/bcma_chipcommon.c: In function 'bcma_chipcommon_attach':
/usr/home/sobomax/projects/freebsd11.mips/sys/dev/bcma/bcma_chipcommon.c:85: error: 'BCMA_CHIPCOMMON_MEM_RID' undeclared (first use in this function)
/usr/home/sobomax/projects/freebsd11.mips/sys/dev/bcma/bcma_chipcommon.c:85: error: (Each undeclared identifier is reported only once
/usr/home/sobomax/projects/freebsd11.mips/sys/dev/bcma/bcma_chipcommon.c:85: error: for each function it appears in.)
*** Error code 1

Stop.
make[2]: stopped in /usr/obj/mips.mips/usr/home/sobomax/projects/freebsd11.mips/sys/BCM471x
*** Error code 1
```


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.530.0](http://build01:8888/viewLog.html?buildId=14376&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 38 Build time: 00:00:34


Tip of the iceberg... lol


 - __information__: [src/chaiscript_stdlib.cpp:0](../blob/8d50160cd903f780f2aaa461f357c144f5bb2501/src/chaiscript_stdlib.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 16 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/8d50160cd903f780f2aaa461f357c144f5bb2501/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 677. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:683](../blob/8d50160cd903f780f2aaa461f357c144f5bb2501/include/chaiscript/language/chaiscript_parser.hpp#L683) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/8d50160cd903f780f2aaa461f357c144f5bb2501/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/8d50160cd903f780f2aaa461f357c144f5bb2501/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 683. (multiCondition)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/8d50160cd903f780f2aaa461f357c144f5bb2501/src/chaiscript_stdlib.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __information__: [src/main.cpp:0](../blob/8d50160cd903f780f2aaa461f357c144f5bb2501/src/main.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 17 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __information__: [src/main.cpp:1](../blob/8d50160cd903f780f2aaa461f357c144f5bb2501/src/main.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/8d50160cd903f780f2aaa461f357c144f5bb2501/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/8d50160cd903f780f2aaa461f357c144f5bb2501/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/8d50160cd903f780f2aaa461f357c144f5bb2501/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/8d50160cd903f780f2aaa461f357c144f5bb2501/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/8d50160cd903f780f2aaa461f357c144f5bb2501/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


Good catch on relative vs absolute. I thought that relative keys weren't working for me, but that was probably when I was experimenting with different files.

It looks like Devise uses relative (for example, in https://github.com/tigrish/devise-i18n/blob/master/app/views/devise/registrations/new.html.erb), so I'll add a fixup for my commit.


This is a new method, alias is not necessary.


Da würde ich beipflichten wollen.
Wir reden hier von der Bedienung des Emulators und da wird m.E. nichts in
den Emulator eingelegt sondern etwas geladen.

Am 23. Februar 2016 um 21:56 schrieb Lothar Serra Mari <
notifications@github.com>:

> Aber in der Emulation muss ich doch das Image/Abbild z.B. via
> Kommandozeile laden/als Parameter angeben und nicht irgendwo physikalisch
> einlegen, oder?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rootfather/mame/commit/ebf2ee3705279c7fc5ba30ecc708036c3ee99aab#commitcomment-16279830>
> .
>


Are you open to adding a function to override the `compactTextMarshaler` value? One of the reasons we have a long-lived fork of this repo is to support the idea of "redacted" fields that are omitted if protos are accidentally printed into logs, stack traces, etc. This recent change gets us very close to being able to unfork at least that part of the code.


Docs updated


This detailed string is what I would expect `__repr__` to return for a `Name` object. For `__str__` I would expect just `'Jenny'`. Fix `Name.__str__` and this patch should be good to go.


ok cool will do, in fact when i was going trough the list, i found a few cards that are not implemented properly so ill fix those all up also
https://github.com/magefree/mage/blob/master/Mage.Sets/src/mage/sets/legions/BerserkMurlodont.java


Oh wow. This is worse than expected. Good find. Somewhere later I will add a note to the other bug report and then create a pull request. You and your find will be mentioned, too.


Looks like obsolete comment. Should be removed.


Sure, thanks. Made the change as recommended with the check in DisplayString().


cursos para impartir


m_strValue's are already part of the addon interface struct wich will be passed as it is to open() a few lines below this discussion point. So this position is already nearly the last opportunity to do it.
But I have the feeling that I haven't really got your idea, sorry for that!


Hi @davivel! Even though I am not actively developing currently, I am still watching progress and very exited to see that finally uploads are becoming reliable.

Just a few words, why I introduced this KEY_WIFI_ONLY. The main reason is to keep program logic out of this FileUploader service: It should be told when (upload conditions so far: wifi_only, charging_only) to upload which file. It should not matter from where the file originates. This is the cleanest and most straightforward solution.

Otherwise you need to check later on for "is instant pictuture upload" and "is instant video upload" (and the appropriate logic (that is to which upload conditions this translates) is inside FileUploader). And when watched folders are coming "is upload for watched folder". Things get even more complicated if upload conditions can be configured per watched folder. Further, there might come an upload dialog which allows the user to delay the upload until the phone is being charged...
One last thing, storing the upload conditions inside the upload entry allows users to dynamically change instant upload when charging only condition, without affecting the upload conditions of already taken (but not yet uploaded) pictures.


Does this fix the following issue in today's build?

```
Caused by: java.lang.NullPointerException
        at org.openhab.binding.zwave.handler.ZWaveControllerHandler.initializeNetwork(ZWaveControllerHandler.java:85)[163:org.openhab.binding.zwave:2.0.0.201602230202]
        at org.openhab.binding.zwave.handler.ZWaveSerialHandler.initialize(ZWaveSerialHandler.java:96)[163:org.openhab.binding.zwave:2.0.0.201602230202]
        at org.eclipse.smarthome.core.thing.internal.ThingManager$7$1.call(ThingManager.java:639)[97:org.eclipse.smarthome.core.thing:0.8.0.201602231715]
        at org.eclipse.smarthome.core.thing.internal.ThingManager$7$1.call(ThingManager.java:1)[97:org.eclipse.smarthome.core.thing:0.8.0.201602231715]
        at org.eclipse.smarthome.core.common.SafeMethodCaller$CallableWrapper.call(SafeMethodCaller.java:170)[91:org.eclipse.smarthome.core:0.8.0.201602231715]
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)[:1.8.0]
        ... 3 more
```
If so I won't bother raising an issue for it.


New!


@keszybz No this is not the same thing. This is equivalent:
 * `const char *`
 * `char const *`

This: `const char const*` is just declaring twice the `const` on the char, there was no `const` on the pointer


@rjmccall, confirming that this is what you were suggesting yesterday?

(This is not yet in mainline; @cwillmor and I are still preparing a proper PR.)


Your commit is fine - you don't have much other chances to workaround this scope.

The real problem here is: we need to feed in what the interface wants. So we copy values, keep them in scope just that the interface can get its const char* values.

I did not present a better solution, but pointed out _why_ we have this issue.


Yes, agree. Changed to strict equality. Thanks.


My bad. I did something similar earlier that didn't work, which is why I introduced passing repository in getLayoutFromText. Using mock works here.


closes #619


I'm new to namespaces, but it compiles on my linux and travis, so I don't think it is needed here.  I think the reason it is not needed here is because class Zerberus and class Fluid both extend the class Ms::Synthesizer, so when dealing with names inside of the class, the "Ms::" (and the "using namespace Ms" which I deleted above) are not needed.  But please correct me if I'm mistaken.  I'm definately able to run musescore and change the fluid syntheizer soundfont.

Although the big problem I've yet to figure out is why I get a crass when changing any of the prferences folders using the file viewer...maybe do you have an idea on that?


don't want to increment if next() raises StopIteration


@sgress454 jk I also had to do https://github.com/treelinehq/machine-as-script/commit/3fd31023ec5e4e49823d5206407f224604001aca because I hadn't tested it yet.  But now it totally works.


congrats bro :+1:  yyeah its really good as start point and ı wonder
can we make this open-source rild to work with our old proprietary libril? if we can, then we can get rid of old non-pie rild and switch to open source one. and look at these (it can be interesting for ril stack for us)
https://github.com/CyanogenMod/android_device_samsung_galaxys2-common/commit/51b674be05aebdb3bd34460eece9f9770b734403
http://www.dmitry.gr/index.php?r=06.%20Thoughts&proj=05.%20Android%20M%20on%20Nexus%20S (dimitry-ril part)


Fixed.


It runs on startup (the delay is `TimeSpan.Zero`).


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16897](https://teamcity.mesosphere.io/viewLog.html?buildId=142730&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:03:38


Would this read better if the leading space was inside the JavaScript? Like

```
error += ' <%= t '.description'} %>';
```


You might want to check the boundingbox of the mummy to spawn for any collisions so that they dont spawn in walls


Another incorrect renaming.


ok, the only thing I was thinking about: is c_str() really safe? so is it really assured that the Cstring holds a continuous memeory block and c_str() returns this allocated block? Or could c_str() also be temporary?


One more incorrect renaming.


Would like to see some good descriptive words here. Looks good otherwise


Check the whole file for such incorrect comments starting from `"class_name that`


So the better way to handle would be to bring in this commit to pygobject. https://git.gnome.org/browse/pygobject/commit/?id=4164e7088be446fcab65cef642c6853aad3b6a69

It's also not on a stable release yet, so I'll add it to our patch series and submit an obs request along with this work.


Awesome!  I'll also update appveyor.yml to use GHC 7.10.3, which should avoid the inconsistent install plans.


new school: target_compile_definitions(process PRIVATE -DPLATFORM_OVERRIDE)


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/__init__.py`, line 5.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/__init__.py
+++ b/coalib/__init__.py
@@ -3,6 +3,7 @@

 __version__ = Constants.VERSION

+
 def assert_supported_version():  # pragma: no cover
     if not sys.version_info > (3, 2):
         print("coala supports only python 3.3 or later.")
```


**No newline between shortlog and body at HEAD.**
*GitCommitBear, severity 1, section `commit`.* *Project wide.*


turns out it was 'c'. doh.


No - it’s not related, but consider this reported and I’ll add some checking in this area which should fix the problem. If it persists in the next version, raise an issue tomorrow.


since mp3s don't store the actual notes themselves, we'd have to auto-generate mp3s on the fly when people make any change to their feldman document, right?


I'd just do an `else` here, to make sure that you're ALWAYS raising exception if dropping queryset


I wonder if there's a better name for this key than "description." Maybe "instructions" is more appropriate?


Also, did you get a chance to click the demo link?


Yeah, you're right. I hadn't tested that, but just gave it a try, and it works.

I just realized, should this all be nested under `devise`, with `capturable` as the level below it? Or does it make sense as is?


Looks like it was a simple coding error. It is inconsistent to specify a negative constant as "**unsigned** long".
Surprisingly, this does not issue any warning on a 32 bit machine. This may be the reason why this warning made it till today.


If these imports are not required, please remove the whole `if` statement.


ewe


thats what i was just saying....the open source libril isnt working atm. So the only way atm is to use proprietary libril with open source rild.

However in addition to that i had a working libril before so i just have to figure out which iteration i used and what changed in order to update it to start working again. And if i can get to that point again then we wont need to shim or anything as we can just change the source code. The only thing i dont think well be getting rid of any time soon is the libsec-ril thats a whole other beast.


I'm not sure this is correct. What this says is that if you're not using `_id` as your id field, you can overwrite it.  Is that desired behavior? Personally, I don't think so. Now the behavior of passing an "id" parameter in the body changes depending on a different, non-obvious config setting.


`*MouseInput()` should be renamed as well since these methods are already renamed.


All fine, see: http://www.cplusplus.com/reference/string/string/c_str/


 - __information__: [src/chaiscript_stdlib.cpp:0](../blob/feb7775d21420670fd7a3bd227dad54fe35a7ac9/src/chaiscript_stdlib.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 16 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/feb7775d21420670fd7a3bd227dad54fe35a7ac9/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 677. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:683](../blob/feb7775d21420670fd7a3bd227dad54fe35a7ac9/include/chaiscript/language/chaiscript_parser.hpp#L683) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/feb7775d21420670fd7a3bd227dad54fe35a7ac9/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/feb7775d21420670fd7a3bd227dad54fe35a7ac9/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 683. (multiCondition)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/feb7775d21420670fd7a3bd227dad54fe35a7ac9/src/chaiscript_stdlib.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __information__: [src/main.cpp:0](../blob/feb7775d21420670fd7a3bd227dad54fe35a7ac9/src/main.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 17 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __information__: [src/main.cpp:1](../blob/feb7775d21420670fd7a3bd227dad54fe35a7ac9/src/main.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/feb7775d21420670fd7a3bd227dad54fe35a7ac9/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/feb7775d21420670fd7a3bd227dad54fe35a7ac9/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/feb7775d21420670fd7a3bd227dad54fe35a7ac9/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/feb7775d21420670fd7a3bd227dad54fe35a7ac9/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/feb7775d21420670fd7a3bd227dad54fe35a7ac9/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


develop (lefticus) - Win64-Windows-10-VisualStudio-14: OK (389 of 389 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)


According to Wikipedia, it "prevent(s) copying of objects at function calls or with the copy-initialization syntax".  https://en.wikipedia.org/wiki/Copy_constructor_(C%2B%2B)#Explicit_copy_constructor.


Incorrect renaming.


ok.


To be clear, in my PR, I always set `data[this.id] = id;` even if it was already defined. Now, when you're using `_id`, you can't overwrite the unique id of the object, but when you're using `id` you can.


See if that's better. EMPTY wss a Visual Fox Pro command like ELSE 8^)


Reminder: typo here, `pipeling`


LOLZ - Ash is always to blame


Created a Dummy Dimmer, level range is from 1 to 100. Setting to 0% sends a 1 and switches dimmer off. Setting to 100% sends a 100 but after refresh changed to 99% in webinterface and database.


I haven't looked at this code in a while, so bear with me—won't this make it so that if you have this pref set, the plugin won't work at all (even if your selection is non-empty)? Or will it do something else?
Thanks!


Usually binding descriptors is done in `__init__`, not `__call__`.  I'm not clear on why you're using the latter here, if there's some reason for it I'm not seeing, maybe add a comment or docstring explaining it?


Why are there two files...?


Na ja, man lädt es in der Emulation ja eigentlich auch nicht. Wenn ich in den "File manager" gehe und für flop1 ein Disk-Image angebe, dann kann ich in der Emulation die Dateien darauf nutzen, aber ich als Anwender weiß nicht, dass der Emulator dieses Image lädt oder was er genau damit anstellt. Es ist eher sowas wie ein "Einhängen" oder "mount".

Zugegeben, "einhängen" ist etwas zu plastisch.


This is the _other_ demo which does exactly what we want: https://ryoyakawai.github.io/smfplayer/


wauw.... thats a huge change lord... great job!


I'm afraid I've never really looked at threading in Python much, so I'm not quite sure what you're saying here.
Should I simply revert `join` back to how it was?


Incorrect renaming. This is a sample app window title.


we should add an == instead


`--index` must match _exactly_, and precludes any other "filter" based options.  You can have multiple `--prefix` and they work in conjunction with other "filter" based options.  It's not obvious from the docs - I had to use the source for curator along with trial and error.


I checked in a change that just changes the condition to

````Xml
Condition="'$(TargetGroup)' != 'net46'"
````


That's a good point. You could look at another Devise plugin, such as Invitable, to see how they handle it. There should be several examples to look at from the Devise repo and wiki on Github.


Wobei "einhängen" gar nicht mal schlecht ist, das kann man ja auch mit rein
virtuellen Dingen wie einem Dateisystem machen. "Einhängen" klingt für mich
weniger plastisch als "einlegen".

Am 23. Februar 2016 um 22:12 schrieb Michael Zapf <notifications@github.com>
:

> Na ja, man lädt es in der Emulation ja eigentlich auch nicht. Wenn ich in
> den "File manager" gehe und für flop1 ein Disk-Image angebe, dann kann ich
> in der Emulation die Dateien darauf nutzen, aber ich als Anwender weiß
> nicht, dass der Emulator dieses Image lädt oder was er genau damit
> anstellt. Es ist eher sowas wie ein "Einhängen" oder "mount".
>
> Zugegeben, "einhängen" ist etwas zu plastisch.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rootfather/mame/commit/ebf2ee3705279c7fc5ba30ecc708036c3ee99aab#commitcomment-16280247>
> .
>


真逼屌


If you make the id an arbitrary field, you can change that arbitrary field no? For example, if the id is the email address you would still want people to be able to change their email address.
Not being able to change `_id` is really just a MongoDB restriction (afaik you can change it in almost any other database).


> Why are there two files...?

@phistuck  See: https://github.com/MicrosoftEdge/Status/issues/357.


Also, i'm going to fix the lines more than 80 chars too


Overloading `__dir__` is rarely a good idea in my experience.  It makes debugging difficult by breaking introspection, particularly if something goes wrong in `__dir__` itself, and I don't see the necessity here.  I'd rather make this an ordinary method, maybe called `dir`, and do Object.special_attributes.dir(), or something similar.


 - __information__: [src/chaiscript_stdlib.cpp:0](../blob/f36b1fc5eba6b9eb03301537c1e6aec00bc73509/src/chaiscript_stdlib.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 16 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/f36b1fc5eba6b9eb03301537c1e6aec00bc73509/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 677. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:683](../blob/f36b1fc5eba6b9eb03301537c1e6aec00bc73509/include/chaiscript/language/chaiscript_parser.hpp#L683) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/f36b1fc5eba6b9eb03301537c1e6aec00bc73509/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/f36b1fc5eba6b9eb03301537c1e6aec00bc73509/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 683. (multiCondition)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/f36b1fc5eba6b9eb03301537c1e6aec00bc73509/src/chaiscript_stdlib.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __information__: [src/main.cpp:0](../blob/f36b1fc5eba6b9eb03301537c1e6aec00bc73509/src/main.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 17 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __information__: [src/main.cpp:1](../blob/f36b1fc5eba6b9eb03301537c1e6aec00bc73509/src/main.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/f36b1fc5eba6b9eb03301537c1e6aec00bc73509/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/f36b1fc5eba6b9eb03301537c1e6aec00bc73509/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/f36b1fc5eba6b9eb03301537c1e6aec00bc73509/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/f36b1fc5eba6b9eb03301537c1e6aec00bc73509/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/f36b1fc5eba6b9eb03301537c1e6aec00bc73509/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


:clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap: :clap:


AFAIK, this was done by CS-Fixer


develop (lefticus) - i386-Windows-10-VisualStudio-12: OK (388 of 389 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.74%25-yellow.svg)


This needs some kind of `__repr__` and `__str__` for debugging, to make it easy to see what special attributes a given object has defined.


#
 2.  Save the run_analysis.R script.


Looks like [Invitable](https://github.com/scambra/devise_invitable/blob/master/config/locales/en.yml) goes with `devise` at the top level, like core Devise does. I'll make that change along with updating my commit to use relative keys.


Fixed in https://github.com/IQSS/dataverse/commit/26d4ddf9afc1d79a1bbda416d79f03cd2eb5f564


I was expecting to wait until Douglas respond the translation email, but if on Wednesday we don't have the translations, we can remove them.


When I was researching Dimmer issues there was a call in the JavaScript you mentioned that added +1, for some reason it looked like internally everything had an offset and then it was corrected at the UI..

I think with all the restructuring of switches lately this has got muddied up, I wonder why it was like that originally because it was causing issues on its own.


I was thinking it might prevent confusion like https://github.com/RPi-Distro/python-gpiozero/pull/217#issuecomment-186705446
Perhaps it's just me, but it feels weird doing `super(MockPWMPin, self).__init__()` and not passing through the `number` parameter. This code is the first time I've ever used `__new__`  ;-)
But happy to add this line back if you think it makes more sense to.


lgtm


Woops, forgot to remove the "Daily", it'll be gone on the next commit


Woops, this is diverging from the ckan...maybe I messed up my patch. It should be `'timezone': datetime_.tzname()`


@Eoler sorry about this, the changes are available, just not picked up since there has not been a new version created. It is possible to still obtain the changes by installing the plugin again over the top (do NOT uninstall the plugin, this will result in data loss).

1. Navigate to System > Updates > Install Plugins
1. Type in the search: rainlab.user
1. Select the RainLab.User plugin (it should already be installed, DO NOT REMOVE IT FIRST)
1. This will download the latest files from the server for you


Do not use inline styling :)


please rename dateOFmanufacture to dateOfManufacture
first letter is lowercase, every first letter of other words are UpperCase


1. Effectivement, il y a un espace de trop.
2. Non, HTML5 n'aime pas trop les custom elements  en balise simple (iirc)


Cool, thanks a lot. :)

I appreciate that it's a non-trivial and carefully written script. Documenting what it does (not how it does it) might go a long way towards more people adopting it and helping you maintain it. Maybe listing out which files are touched, or even just pasting forum links would be of great value.


Can You create one more class Manufacturer and put the object of this class as this property?


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/__init__.py`, line 5.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/__init__.py
+++ b/coalib/__init__.py
@@ -3,6 +3,7 @@

 __version__ = Constants.VERSION

+
 def assert_supported_version():  # pragma: no cover
     if not sys.version_info > (3, 2):
         print("coala supports only python 3.3 or later.")
```


Also i was thinking of resuming the exploration (once model gets updated) from the point we last touched, it makes sense if you think about it. We start at random ONLY the first time.

Maybe i've reached a compromise with the GP, i'll post some pics soon of the reconstructed mug


alright. thanks


My comment still is this really shouldn't be here... cause it's public.


I would think it should be ZFS_PROP_USEROBJQUOTA and ZFS_PROP_GROUPOBJQUOTA here.


Ah oui, c'est mal ça.


You have two properties for one purpose: numberProductOrder and number. Please chose one and remove another.
And maybe it is better to name it numberOfProducts?


Tyhle __str__ nejsou pro veřejnost nikde vidět, jsou pro naši interní orientaci (/admin je používá pro popisky spojených objektů do rozbalovacího selectboxu). Zkus to prosímtě zkrátit a hlavně.. ta metoda musí vracet str(), schází tam return.


Looks good. Please fix above-mentioned things


+1 sounds good to me!


Using blocky to set the level on a Hue dimmer uses the functions in SQLHelper.cpp, range from blocky is 0 - 16. Doing the same on a dummy dimmer results in a range of 0 - 101.

Looks like the webinterface should use the same range ?


Ensure "Cmd-" is consistently initial cap.
Added: Canvas topic.
Removed: Pen topic.
Changed: Arithmetic, Character, Date, Fonts, Forms, Introduction, and Strings topics.


typo here: geBasictInstantiationType -> getBasicInstantiationType


excelente


the file should have the name Instrument.java


develop (lefticus) - Win64-Windows-10-VisualStudio-12: OK (388 of 389 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.74%25-yellow.svg)


Please rename numberOFstring to numberOfStrings


> If you use "AddDefault" method, the namespace name is "default"

That's what I mean by "magic" :) Would rather prefer to rely on a constant or method to provide me that information and not remember that because it was added using `AddDefault` the name therefore is `default`.


This line fails for me on Windows (with Python 2.7.5) with the following error:

> [1/28] Update build-version.inc in the Spirv-tools build directory (if necessary).
> FAILED: cmd.exe /C "cd /D E:\repo\SPIRV-Tools\build && E:\repo\SPIRV-Tools\utils\update_build_version.py E:/repo/SPIRV-Tools"
> Traceback (most recent call last):
>   File "E:\repo\SPIRV-Tools\utils\update_build_version.py", line 52, in <module>
>     main()
>   File "E:\repo\SPIRV-Tools\utils\update_build_version.py", line 46, in main
>     new_content = ('"spirv-tools ' + describe(sys.argv[1]).replace('"', '\\"') + '\\n"\n')
>   File "E:\repo\SPIRV-Tools\utils\update_build_version.py", line 37, in describe
>     return subprocess.check_output(["git", "describe"]).rstrip()
> AttributeError: 'module' object has no attribute 'check_output'
> ninja: build stopped: subcommand failed.

What you are trying to do with this command is admirable, but you can do the same thing by using a cmake_custom_command in conjunction with ${CMAKE_COMMAND} -P <some cmake script>.cmake.

I'd be happy to contribute a patch for this behaviour.


Comentario de prueba


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16900](https://teamcity.mesosphere.io/viewLog.html?buildId=142740&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:09


Вопросы:
* _WritePendingException_ конечно так не исправляется,ваше исправление в худшем случае приведёт к Stack Overflow. Одно из корректных решений - развязать рассылку сообщений через очередь
* Поясните, каким образом в вашем коде вы решили проблему прихода части сообщений? Разве реализация _putMessage()_ позволит заполнить _messageStream_ в случае прихода половины?
```
        while (messageStream.size() >= 4 && hasFullMessage()) {
            putMessage();
        }
```


test


Try this code:

instead of

    ![Screenshot - LOGO](logo.png)

@varlesh


Ooh good catch. I'll have to figure that out.


@oraixearl hahahahaha..i can imagine


Whoops, this was already installed in my node_modules, so I guess I didn't notice that removing it would bork a fresh install.


Hi j00zek, with this change not compile.

` gdi/picload.cpp: In function ‘void gif_load(Cfilepara*, bool)’:
gdi/picload.cpp:481:39: error: too few arguments to function ‘GifFileType* DGifOpenFileName(const char*, int*)’
  gft = DGifOpenFileName(filepara->file);
                                       ^

In file included from gdi/picload.cpp:12:0:
/media/devel/ipbox/HD/E2/9000HD/tufsbox/cdkroot/usr/include/gif_lib.h:178:14: note: declared here
 GifFileType *DGifOpenFileName(const char *GifFileName, int *Error);
              ^

make[5]: *** [gdi/picload.o] Errore 1`


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#LiJJp5a_S5y5RebQkMh_tg


This threw me for a loop. It looks like DESTDIR is being defined in test-prep, and this ifdef check is for that definition. However really the DESTDIR is coming from the .travis.yml file; when running `make test` locally we're getting the else branch (which is desired, it's just confusing).

The travis line is `make test DESTDIR=$HOME/prefix/ SHOW_INTERACTIVE_LOG=1`. Is the DESTDIR variable still necessary? With this nice new testing architecture it shouldn't need to install at all.


I had another NPE thrown from the HarmonyHub binding that I built last week, I'm wondering if recent ESH commits have broken things.


Better try if adding `utillinuxCurses` to `propagatedBuildInputs`, like is done for `coreutils` and `bc`, fixes your problem, else use `substituteInPlace` like in the lines above your's.

And you could fix your commit title to reference the NixOS package instead of the command name and rather reference the command itself in the message.


Hmm... it looks like adding `devise` at the top level breaks things, because the scope coming from our paths doesn't have devise in it, it has capturable.

For example, if I use a relative key in app/views/capturable/_widget.html.erb and add devise at the top level in the locales file, I get:
> translation missing: zh.capturable.widget.sign_out

I think that means we should leave it as is.


@mattdangerw Sounds good, thanks for tracking that down.


This is only used on 2.7, correct?  There should probably be a comment/docstring to that effect.


Clarification: Does this imply that contributors should create an issue for each PR?


 - __information__: [src/chaiscript_stdlib.cpp:0](../blob/abc30ba57352bc6ec929aa1bff2321d7bbd9fce9/src/chaiscript_stdlib.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 16 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/abc30ba57352bc6ec929aa1bff2321d7bbd9fce9/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 677. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:683](../blob/abc30ba57352bc6ec929aa1bff2321d7bbd9fce9/include/chaiscript/language/chaiscript_parser.hpp#L683) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/abc30ba57352bc6ec929aa1bff2321d7bbd9fce9/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/abc30ba57352bc6ec929aa1bff2321d7bbd9fce9/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 683. (multiCondition)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/abc30ba57352bc6ec929aa1bff2321d7bbd9fce9/src/chaiscript_stdlib.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __information__: [src/main.cpp:0](../blob/abc30ba57352bc6ec929aa1bff2321d7bbd9fce9/src/main.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 17 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __information__: [src/main.cpp:1](../blob/abc30ba57352bc6ec929aa1bff2321d7bbd9fce9/src/main.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/abc30ba57352bc6ec929aa1bff2321d7bbd9fce9/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/abc30ba57352bc6ec929aa1bff2321d7bbd9fce9/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/abc30ba57352bc6ec929aa1bff2321d7bbd9fce9/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/abc30ba57352bc6ec929aa1bff2321d7bbd9fce9/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/abc30ba57352bc6ec929aa1bff2321d7bbd9fce9/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


Sorry, I let this-one trough.


Works for me.


nitpick: please be consistent over ordering of new/old entities.


Add condition here.


I did update my settings, indentations should be two spaces from now on.
Thanks for reminding!

On Tue, Feb 23, 2016 at 10:22 AM, Desiree Cox <dcox@zendesk.com> wrote:

> Hey! Thanks. I did notice that. I keep changing it manually in the files.
> I think I just have to adjust my settings in Sublime.
>
> On Tue, Feb 23, 2016 at 7:33 AM, Eliot Sykes <notifications@github.com>
> wrote:
>
>> @dcox <https://github.com/dcox> minor, but indents appear to be set at 4
>> spaces or tabs for some of these files. Lets review the indent/soft space
>> settings in the editor soon to stop these creeping in.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/dcox/bloccit/commit/463b30e047db8ae4e664796b1c40c8e209b26c2b#commitcomment-16270948>
>> .
>>
>
>
>
> --
> Desiree Cox
> Product Data Analyst
> Zendesk
>

--
Desiree Cox
Product Data Analyst
Zendesk


Is there a reason this doesn't inherit from FunctionModel?


Tiens, du coup je me demande si le lien est supposé être dans la transaction-short ou en-dehors ; pour l'instant ça fait étrange


Ještě jsem zapoměl - aby ten model (tabulka) byla přístupná v administraci, je nutné ji do ní zaregistrovat. Viz.: https://github.com/jizdoteka/jizdoteka-web/blob/master/apps/web/admin.py#L8


Can you add `go_check` to `.PHONY` target at line 56.


Shouldn't the denominator actually be the sixth power of the norm? Upstairs there's the component of the position vector which is not normalized, so the sixth power would be the normalizing factor.


Should:

On MACOS): [arduino Package Contents]\contents\Jave\libraries\makeblock\src

be

On MACOS): [arduino Package Contents]\contents\Java\libraries\makeblock\src


This method name has been changed in the commit
https://github.com/AppFolioOnboarding/Li_Image_Share/commit/114d0057ace5bdc27d00314a4c046314c3d705d8

Li Zhang

Software Engineer | AppFolio Inc.
P: (614) 648-2368
li.zhang@appfolio.comwww.appfolioinc.com

On Tue, Feb 23, 2016 at 11:42 AM, rmacklin <notifications@github.com> wrote:

> I think we could move the page.has_selector? into the private method too.
> Then this would be something like:
>
> assert label_is_present?('image_url')
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AppFolioOnboarding/Li_Image_Share/commit/b8f44162ae666678b78e8aae6a9a7e94e486f03b#commitcomment-16277790>
> .
>


because


t-thanks


This assertion has been added in the commit
https://github.com/AppFolioOnboarding/Li_Image_Share/commit/4a2360066afed11839e21472447574a607aeae62

>


pull betwen modal and master


I learned a lot from this GitHub issue on the HTML5 WebMIDI API: https://github.com/WebAudio/web-midi-api/issues/45


Ok, will do. Thanks!


You are correct, for some reason I though if you wrapped it in `ObjectId(id)` it would work.

That said, I'm personally not a fan of this behavior. Does feathers send proper redirects when primary keys are changed?


Ah oui, tout à fait. J'ai changé ça plus tard.


Author is DDuarte, ported over from TC. Mostly I did cleanups thrown in with this commit. As long as it fixes silly issue with lava/slime :+1:


@joshuajabbour Ya my thinking was the same as @daffl. If you choose to use a custom id you can do whatever you want.

I mean I can't really think of a scenario where you would want to update your primary `id` so I'm totally fine removing that functionality so someone doesn't accidentally fuck up their data. It really should be a new record if you want to change your primary id.


This breaks some styling on chromium
![screen01](https://cloud.githubusercontent.com/assets/2412457/13267468/ad0c5f8e-da7d-11e5-951b-f567d2985d06.png)
The dropdown menu is always shown.


Maybe we want the left most button to be left justified and the rest centered?


@joshuajabbour nope. That would be left up to the developer. Probably in an `after` hook.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16901](https://teamcity.mesosphere.io/viewLog.html?buildId=142735&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:20


No.  But if the PR fixes an issue, it should be mentioned in this way so that the issue is automatically closed when the PR is merged.


I guess it depends on whether you want to verify that `make install` "works". It's probably superfluous but the DESTDIR hack allows `make test` to verify that the `configure --prefix` stuff works.


No. I should add back "if applicable". I deleted it to keep the notes sort, but it's apparent that shortness isn't helping :smile:


Diky :)


You mean to intercept what the default `String()` method on proto messages will do? I think it's a bit unlikely given how global and potentially chaotic that could become. There's lots of code that depends on `proto.CompactText` behaving as it currently does.


Oops, nope.  I'm just a little worried about what the bar is for what goes in "core" since extra stuff means bloat. It might make sense to make Core .h files only and make everything else explicit, does that seem reasonable?


why did you choose 'memcache' and not 'memcached' ;) ?


Thanks. Changing to 6.


Let's keep it consistent and rename "dn-" to "obj-" as well.


@tkelman, I had just tried this today and was amused to see you also had the same idea yesterday. Did it work for you? I have no idea how to interpret the errors on [my run](https://travis-ci.org/phobon/RobotOS.jl/builds/111297036).


Should be cherry-picked into 2.x branch


I see, so in Travis, we're doing a real install into some prefix, symlinking test_root to that prefix, and then running the tests against that. Do I understand correctly?


No problem. Thanks that's good info.
I'm happy to help test for SW if you need any testers.

Learning the background of the files so maybe one day I can help you there :)


saw this post


There are still two requests here.


Also, this guy's keeps commenting the threads (to a relatively unresponsive bunch of people whom he calls "powers-the-be") and he's actually solved the whole issue of WebMIDI -> WebAudio here, I think: http://james-ingram-act-two.de/open-source/WebMIDISynthHost/host.html


@ekryski So that's the problem then with this being a hidden toggle of functionality. Now I need to know that the behavior silently changes in the background. And that other non-related config change now means I have to explicitly introduce checks to make sure data isn't manipulated.

Personally, I'm of the mindset that IDs should not be changed in REST APIs, since that creates the URLs. It can be done, but it should be extra explicit, because a bunch of other decisions need to be made (do you redirect? yes. should you allow the original ID to be reused? etc)


Вопросы:
* Закоммичен результат сборки: MessageBody, генерированный класс также является результатом сборки, или вы готовы ответить и по его коду в том числе. Наличие закоммиченного файла может затруднять сборку на других платформах, где генерация происходит при помощи других версий protobuf
* Поясните логику строк:
```
        serverThread.join();
        serverThread.start();
```
* Поясните, какием образм вы защищены от _WritePendingException_ в _write_ у _ChatWebSocketWorker_?
* Поясните, почему в следующем коде вы решили игнорировать _InterruptedException_?
```
        while (true) {
            try {
                if (!commands.isEmpty()) {
                    Command command = commands.take();
                    request(command.getWorker(), getRequestText(command.getCommand()));
                }
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }
```


This assertion should be outside of the block as it's not essential to the assert_difference.


Fixes #2


An (im)perfect example. ;)


@joshuajabbour @daffl ok then we all agree :+1: I'll issue a new patch release that reverts that behavior so ids can't be overridden.


You could keep the arrows and instead of `this`, use `module.exports`, which contains a reference to the function :0


Change this to
`$temp .= "<tr><td class='postblock' align='left'>" . S_NAME . "</td><td align='left'><input type='text' name='name' size='28' placeholder='" . S_ANONAME . "'></td></tr>";`


No, I'm kind of mystified too. @shionlau came by and asked me if I had any ideas on https://github.com/phobon/RobotOS.jl/issues/10 so I was trying this out. Looks like his issue was a misunderstanding on the test setup, not sure what's wrong on Travis though. If it becomes pressing, I think you can contact Travis' support address and ask for 24 hour ssh access into a worker node to reproduce more easily.


Not sure if @daffl agrees in his comment above :)

Either way, I can work around this because with a before hook to enforce IDs never being changed. A note should probably be added to the readme though detailing the behavior in relation to the `id` config setting.


I tried this out and it breaks all the relative paths in CSS style sheets. Deal is if you specify localBasePath, you must also specify remoteSkinPath, which as far as I can tell has to be the literal name of your install directory. This means making this change in order to make the install directory case-insensitive is impossible for case sensitive file systems. It would end up having to be specified as either 'Monaco' or 'monaco' and nothing is gained except consistency with the WikiMedia Foundation's skins, which all use Uppercase name style.


Yep, but just for us. I don't see it likely that our "redacted" tag and behavior gets upstreamed, be it ever-so-useful (we've tried, albeit not too hard, from the Java direction), so this seemed like a nice place to intercept.

If I find the time, I think I'll still rewrite our changes to use a replacement `compactTextMarshaler` - at least it'll keep our patch of the proto plugin to a one-line `SetCompactTextMarshaler` method.


Too many late nights - cheers


Ya @daffl and I chatted in slack about it. We're all good. :smile:


Thanks :)


Looks like here OBJQUOTA missing as well.

Can we move this assignment of offset earlier as a part of successful dmu_objset_userobjspace_present check? It will help deduplicate the code and eliminate the source of errors.

```
if (type = ...) {
   if (!dmu_objset_userobjspace_present(..))
      return (SET_ERROR(ENOTSUP));

   offset = ...;
}
```


This references issues #3343, where an image download request can crash if it doesn't have a valid URL. Whether this should be handled by the framework or by the implementer is a design decision. It appears that NSAssert is not used for cases like this in the framework, so we are failing silently instead. And given the rest of the behavior in the method, it is suggested the failure block should not be called (or a task created in the first place given invalid inputs). I hope this commit aligns with the solution the authors of AFNetworking would suggest for this scenario.


Thx ) fixed


Wow, I even got rid of the extraneous white space, which is totally something that I would do.


@joshuajabbour on another note. I had seen some of the work you were doing on https://github.com/ferryjs/ferry. For v3 we are looking to be engine agnostic. Maybe you'd be interested in helping out?? :smile:


I was just giving my two cents. @ekryski is changing it and @joshuajabbour needs it so I should really be the last person arguing over it ;)


<img width="462" alt="screen shot 2016-02-23 at 4 46 23 pm" src="https://cloud.githubusercontent.com/assets/210240/13267756/05011ab2-da4d-11e5-8955-2e131ea94606.png">
++ Looks good


Nit: Do we (still) need this?


Make sure this gets sanitized in `form()`. Letting users put strings directly into a query is stupid anyway.


Same as below. I think we can introduce a macro or an inline function to check if `type` is obj-related.


Hopefully you changed the token?


Correct.


Ok just checked and you are right. But is weird doesn't even appear an error. You think is cpu freq  I need to update? I'll check your sources! Thanks!


I improved my build inspired by your lines without the need of `sudo` by adding `stunnel` to the `$PATH`. Check out these resources:

* https://github.com/mp911de/lettuce/commit/bd44d38b4c3d677c3e5a10337c2bd667aa752f34
* https://travis-ci.org/mp911de/lettuce/builds/111325852


Although the main use of lambda functions is to have a different `this` context so I wouldn't worry :p


Nit: Do we (still) need this comment?


Yeah, we should update this on the server side, but it's not an easy change to apply. I bet we will keep this code for long.


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/__init__.py`, line 5.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/__init__.py
+++ b/coalib/__init__.py
@@ -3,6 +3,7 @@

 __version__ = Constants.VERSION

+
 def assert_supported_version():  # pragma: no cover
     if not sys.version_info > (3, 2):
         print("coala supports only python 3.3 or later.")
```


you can `lazy override val settings = super.settings ++ Seq(` here and it'll be automatically loaded by all projects, including the root (which here, is using a different version of scala to the projects it aggregates)


apparently `marked` published new APIs between 0.3.1 and 0.3.4 that caused errors if we didn't implement the above renderer types. IMO since they are new features that caused errors in our code they shouldn't have used semver patch, so i took out the ~ in our package.json


Would we really want to roll back the EQ entry if the write fails?  Doesn't this just mean that the pipe/eventfd is full and the consumer isn't reading from the fd quickly enough?  Seems like an argument for moving to a model where sread always drains the whole fd.


I think there is a bug if `dependencies` is `null` or `undefined`.


radio_context_tbl_[radio_id]->set_bs_id(bs_id);


You should make the decision that best works with feathers (e.g. if you're going to disallow changing primary keys, all db adapters should enforce that imo). Again, I can work around this in a before hook, so if it isn't changed then I think a doc update would solve the problem for me.


Oh you fancy huh!


Looks good.


@ekryski haha, yes I was going to pitch adding engine portability for feathers someday (I want to use restify :) We had high ambitions for ferry, but after finding feathers I'm happily moving on...


Po mergnuti s tvojou vetvou nam program crashne na tomto riadku pri nacitavani grafu cez tlacidlo Load function calls. Vyzera to, ze sa 2x vola createNewGraph.
Mas na to riesenie, ci sa mame pokusit to fixnut? :) Zatial len ako docasne riesenie je zakomentovanie celej podmienky..
thx


Let's add a comment here. Otherwise this line looks rather surprising.


You're Awesome : D


Do you think it's worth adding a small section to INSTALLING.md that describes the extra dependencies for running all the tests?  I don't think it's a huge deal but I remember that last time we added one (lxml I think?) it tripped some people up.


I think it's OK, maybe there is no need for the back-quotes around empty, but I'm nitpicking.


Sorry about the failure.  It looked like it was available in 2.7:
https://docs.python.org/2/library/subprocess.html
but I guess that doesn't cover platform portability.

I'd accept a pull request as you suggest.


You could also default to `'%key%.%env%.json'` if `env` is set :)


lnln


develop (lefticus) - i386-Windows-10-VisualStudio-14: OK (389 of 389 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)


Feel free to file an issue so we don't lose track.


@joshuajabbour regarding, changing ids. No. The other database adapters currently do not support that.

regarding helping we'd definitely love some support there. We've soft launched 2.0 this week and will be doing a more formal announcement later this week or next.


Ah.. I read it wrong.  I thought it was something like this ( a && a.b && a.b.c === something).  In this case, then the lodash.set would make sense.  Just ignore my comment.


Fonctionnel à moitié en fait


Holy fuck, it's fucking noclip mode


well in most unit tests deps by default is undefined and they works as expected.
I can add test for case when dependecies == null but i'm sure it will works ok because it converts all falsy values to an emtpy array.


facepalm


FillComponent may be what you are after:

  http://www.vtk.org/doc/nightly/html/classvtkDataArray.html#ad29ee5000e57bdb6cf854b2de4b3f804


Here a simple:
if delegate == nil {
   return rc
}
could be better, no need for guard since the result of assignment is not used


Aha, is all good. Was driving me nuts! I was using the Monogame client specifically for the fact that it doesn't require XNA framework on the players part. I originally forked the 1/28 stuff and I know nothing about Git. I tinkered around with stuff in 1/28 but never understood why 1/29 wouldn't work.
Went through and tested out each commit and, after a couple of WEEKS, I realized what was causing the freeze. xD
If I find anything else out, I'll make a pull request. As soon as I figure out how to merge the updates. xD
Thank you for all that you've done! A custom client is simply the best lol.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16905](https://teamcity.mesosphere.io/viewLog.html?buildId=142785&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:04:23


will this change cause `ENOSYS` or segfaults for users who erroneously call `fi_eq_sread`?


Yeah, I think it'd be best for you to stick with your local modifications.


Note that this assignment statement can change `mCheatBank[mCurrentIndex]` from `false` to `true` if the user has newly cheated, but it can also change `mCheatBank[mCurrentIndex]` from `true` to `false` if the user didn't look at the answer on this most recent visit to the `CheatActivity` but had previously looked at the same answer. That doesn't seem right to me. It seems that once we have recorded a `true` into this array, we ought never turn it back to `false`. I think the simplest fix would be to replace `=` by `|=`.


Noted.


ok @joshuajabbour new patch release v2.1.2 removes that ability.


"Precise" is the opposite sense of "Indeterminate", so this should be `!requiredResourceCountIsIndeterminate`...


Not too big of a deal right now, mostly just because ROS has barely changed in ~3 years. But yeah, the switch should probably be done in the near future. Maybe by the time ROS K is released.


Don't we end up with `define([null], f)` in all these cases?


...and this should be `else if (status.isRequiredResourceCountPrecise())`.


The naming for this var I found a bit hard to follow. How about something like "isPotentialObjectPattern" or "couldBeObjectPattern"? A little hard to name this :)


I think the there was a presoftmax_scale parameter, which was later removed or added from one of the scripts.


Especially with 16.04 coming out soon, I think very few users will be on 12.04 any more. 14.04 will still be widely used for another year or two though.


I was thinking same. Done.


The choice of `expr == 0` instead of `!expr` was a deliberate choice by whoever wrote it originally (probably Reese).  While I find both forms readable, I do often have a slight preference for `== 0` and this seems like needless style churn.  Ditto for all the little wibbles moving `static ssize_t` to the same line as the function name in the definitions.  It's probably not worth it for you to go back and update all these commits to undo these churn-inducing changes, but it would be good to resist the temptation to continue them in future PRs.


nope,

```js
dependencies = dependencies || [];
```

will replace all these values: ```"", null, undefined, false``` with an empty array


It's a Symfony standard, I just ran PHP-CS-Fixer and Symfony is the default one.


Thanks!


As @kinncj said and it's also in the commit message.

PHP-CS-Fixer uses Symfony as a default standard, IMO, doesn't hurt.


@vpavic I would love that :)


If you just have a requirements.txt file like this - augmenting tox probably(?) isn't needed, I bet it will just use that file automatically.


I meant to merge this up into the previous commit.


Actually this is the correct if you see that the file is `TICKTables`, which is how it's used on https://github.com/gabrielrcouto/php-terminal-gameboy-emulator/blob/c116b7b/src/Core.php#L411-L412


I can confirm that this does solve #957 on archlinux/X11


Found this problem when testing create profile from config item.


Should "Something happened" on [line 34 of login_ajax](https://github.com/oreillymedia/devise-capturable/commit/dc5e697d3e9cea251c81b91155547c1cbdcaecd7#diff-dfef55a338ecae6f29d638cf7d01af5eR34) also be translated? I think that message is really never displayed, but in case it ever is...


for rendering of the table layout see my page on branch documentation.

I know there are two commits there, i will rebase, i made all stuff from gui now.


I will never get used to not using semicolons.


Ho you're right, I need some sleep ! ;)


good on ya clem,


^ typo ('tt')


:beers:


I would toss this whole commit, other than maybe the bits related to `FI_SUCCESS`.


But, I have to agree with you, `TickTables` is how it should be named.


You can find the code in EntityLivingBase#canSpawnHere (or something similar)


Hey thanks.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16906](https://teamcity.mesosphere.io/viewLog.html?buildId=142806&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:03:28


Ah okay - join is just a method that waits for a thread to terminate, i.e. if thread 1 waits for thread 2 to terminate we can say thread 1's "joined" thread 2 (if you think about a graph of their runtimes it makes a certain sense, or you can just think of it as a "wait" method like any other IPC device - mutexes/events/whatever).

The usual `Thread.join` method (which we're inheriting from here) takes a timeout so you can wait for a thread to terminate, but if it doesn't terminate within a particular time your calling thread can continue running. Unfortunately, unlike other IPC wait methods (like Event.wait etc.), `join` doesn't return True/False to indicate whether the thread *actually* terminated - if you specified a timeout you have to query `Thread.is_alive` afterward (see https://docs.python.org/2/library/threading.html#threading.Thread.join for details).

Thankfully, using a timeout with join is actually a fairly rare use-case (assuming you write your threading code well :) so in this case I just simplified the join method in the sub-class by getting rid of the timeout parameter. If you want to keep it in there it'll need refining - but it's probably not worth it if/until we find a use for a timeout on `GPIOThread.join`.


The problem is that the pdf cleanup uses layouts to determine the new file name using `Util.getLinkedFileName`. Just add a repository parameter to the constructor of` ̀RenamePdfCleanup` and then pass it to getLinkedFileName. In the tests you just create a mocked repository. It also explains why the test failed since you changed the method above.

(By the way, this is a perfect example why dependency injection is superior to global objects. They clearly display they dependencies they have and don't require spooky statements in setUp() :smile: )


@rwinch - great, I'll put together a PR during the next week or so.


I think this change should be undone.  The purpose of using typescript is the strong typing and compile time errors about things that won't be valid at runtime.  Combining the definitions of the main and render environments bypasses compile-time checks and allows attempts to use render-only APIs in the background scripts and background-only APIs in the foreground scripts.  Perhaps splitting out into electron.d.ts, electron-main.d.ts, and electron-render.d.ts (the first being the common definitions for both environments) it would resolve the issue?


> Doesn't this just mean that the pipe/eventfd is full and the consumer isn't reading from the fd quickly enough? Seems like an argument for moving to a model where sread always drains the whole fd.

Well, I'd think that the write could fail for other reasons too. That isn't the only failure case, is it?


Thanks! And yes, I'll keep an eye out here and pitch in. I'm really happy with feathers so far...


Hi Paul,

Which part of control loops does support the GPS glitch behaviour now? I did some digging in the ArduPlane directory and just can't find any references to "glitch", "offset" or anything similar related to the GPS misbehaviour.

By the way, I'm a big fun of your work on the EKF.

Regards,
Przemek


develop (lefticus) - Win64-Windows-10-VisualStudio-14: OK (389 of 389 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)


Fixed


No, it's already set to `fi_no_eq_sread` in the struct initialization above.


@vpavic Awesome! If you are feeling ambitious I'd love to see a PR that sets up Eclipse on import with the correct settings too. Something similar to what Spring Framework does https://github.com/spring-projects/spring-framework/tree/master/src/eclipse


Good catch—yes, I think if we're translating the token error message, we should take care of this one, too.


Maint**e**iner is typo.
Maint**a**iner is correct.

desi**ng**er is typo.
desi**gn**er is correct.

Thanks for including me on dev list.

@varlesh


Po mergnuti s tvojou vetvou nam program crashne na tomto riadku pri nacitavani grafu cez tlacidlo Load function calls. Vyzera to, ze sa 2x vola createNewGraph.
Mas na to riesenie, ci sa mame pokusit to fixnut? :) Zatial len ako docasne riesenie je zakomentovanie celej podmienky..
thx


:D


I understand for now you are piggybacking on deployment.deploy_cfme

We will want to modify this to check "deployment.deploy_openshift".
That can be a separate PR where you:
 - Add deploy_openshift to deployment model, add migration
 - Update this deploy.rb
 - Create a script to kick off deployment from CLI and verify it's working with OpenShift task.

Please write yourself a note this is  needed so you dont forget.


Jeezus tap dancing christmas, sigh, ignore the stuff from script/ my clean seems to be fubar =/


Yeah, agreed.

Fixup pushed.


Actually it is used for Python 3 as well, since we don't have a distinction right now between bound and unbound methods.


@XhmikosR oops, added :)


Ref: https://github.com/Automattic/wp-calypso/issues/3518


Great, thanks! I'm going to email Douglas for these translations soon.


I see that point, but imagine another formatter needing another "global"
object etc. There will be quite a few arguments passed around which are
only used in very rare cases...

How to mock the repository?


Typo here ... should be something_happened


A quick google search turned up this:
http://www.paulirish.com/2010/the-protocol-relative-url/

Not sure that is the definitive answer, but it suggests that you would be correct that it would be appropriate to always request over https.


I tried to avoid using classnames. statinamic doesn't have classnames as dep. The would increase bundle size (assuming that user doesn't use classnames)


@jazzzz, fixed


oginal > orginal?


I was wrong, this line is OK, since we are reusing DMU_{USER,GROUP}USED_OBJECTS to store object-related (or `dnode` in old notation) quotas.


Could you move them right after the php one? So they will be all regrouped at the top and new lib can be added at the bottom


fixed!


Thx you!
Ou )) Sorry for this! I bad know English :pensive:


Should we check for these curl function when curl is enabled?


Tidy isn't a requirement any more (even in readability)


Is it still used ?


@rwinch IntelliJ IDEA user here so that one might be a bit outside of my regular domain :)
I'll look into it if and when I get some time, unless someone beats me to it.


Thanks. That seems like a nice thing to test. I also just realized that DESTDIR [has an established meaning](https://www.gnu.org/prep/standards/html_node/DESTDIR.html) so that changes my view a bit.

I also just discovered that command line variables override the variable defined in the Makefile, so the DESTDIR assignment in the Makefile won't do anything in Travis. That's a nice leveraging of that feature.

This is really nice! I think all that it needs are some expository comments:
1. Above the `DESTDIR = ...` assignment, point out a DESTDIR specified on the command line takes precedence, since that's not obvious (at least not to me)
2. Above the `test-prep` assignment, explain that when DESTDIR is set, we expect fish to already be installed at that path and want to test the installed contents.

Those comments will definitely help me a month from now when I've forgotten how this stuff all works :)


Couple of comments here:
* Start this section with `### Deleting no longer follows symlinks.`
* You don't need to say "This is a breaking behavior change where in" Just say "Pervious versions of Gradle would follow-symlinks during deletions."
* No initial "to" in "to project.delete() to follow symlinks,"
* Add "delete" in "attempt to **delete** files outside of Gradle's"
* Make the NPM example a parenthetical phrase `(e.g. NPM installed in a user-writeable location.)`


> Ditto for all the little wibbles moving static ssize_t to the same line as the function name in the definitions. It's probably not worth it for you to go back and update all these commits to undo these churn-inducing changes, but it would be good to resist the temptation to continue them in future PRs.

All of the new code I'm adding is conforming to the linux kernel style guide and being enforced by the `checkpatch.pl` script distributed in the kernel repo. I renamed/moved this function and as such the declaration has to conform to the style guide. The usnic_direct source tree has a different style guide that I'll comply with when in that source tree, but when in the libfabric source tree I'm going to conform to the kernel style guide. I find value in unifying the style. It makes it more readable.

> While I find both forms readable, I do often have a slight preference for == 0 and this seems like needless style churn.

Alright, I will avoid changing existing conditionals unless I change them. I personally almost always try to use pointers as truthy themselves, and flag comparisons (`&` or `|`) as truthy.


The primary purpose is to navigate by flying, not to go through walls.


develop (lefticus) - i386-Windows-10-VisualStudio-12: OK (388 of 389 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.74%25-yellow.svg)


## Task 1

Previously, you wrote a program that used a recursive function to compute the sum of the integers from 0 to 1000 (inclusive).  This time, I want you to accomplish the same thing, but using a `while` loop. Compute the sum of 0 to 1000 (inclusive) in a function using `while`. Return the sum of these numbers to the main loop and print the result. Write the program to include a `main()` function and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program computes sum using a `while` loop in it's own function (1 pt)
- [ ] Program correctly computes and displays sum of 0 to 1000, inclusive (1 pt)

## Task 2

Above, you wrote a program that used a `while` loop to compute the sum of the integers from 0 to 1000 (inclusive).  This time, I want you to accomplish the same thing, but using a `for` loop. Compute the sum of 0 to 1000 (inclusive) in a function using `for`. Return the sum of these numbers to the main loop and print the result. Write the program to include a `main()` function and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program computes sum using a `for` loop in it's own function (1 pt)
- [ ] Program correctly computes and displays sum of 0 to 1000, inclusive (1 pt)

## Task 3

In Python, we tend to use `for` loops over different types of iterators much more than `while` loops - they're more flexible and iterators are **everywhere** in Python.  Write a program that includes 5 different functions, each of which should demonstrate the use of `for` over **different types** of iterators. Your program should include these five functions, a `main()` function that calls all the other functions, and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program includes `main()` function and "ifmain" statement (1 pt)
- [ ] Program contains 5, correct subfunctions that use `for` over **different types** of iterators (5 pt)
- [ ] Program's `main()` function calls all subfunctions (1 pt)
- [ ] Program runs and correctly demonstrates use of `for` (1 pt)


Spurious whitespace change on this if line


Yeah I added it at a later commit and then rebased the whole branch D: So if you check out #255 now it's in there :tada:


We don't actually want to exclude `joda-time`, regardless of what leiningen says. It will break unless something else happens to pull it in (which may be happening or it may only break the DynamoDB protocol). It's possible that the latest version of Datomic fixes this, but upgrading to it is not as simple as usual because you have to pull in your own AWS library (which I think is the source of this internal version conflict in the first place).


If the value you get back from the `savedInstanceState` is `true`, that implies the answer was displayed before the device was rotated. Because users don't usually expect information to disappear just because they rotate their device, I'd suggest redisplaying it under this circumstance.


It's an internal Quake command, not something that players should see in the menu.


I'd update this string since we aren't really doing a yum install in this task.


lgtm


unless I misunderstand semver, minor, and revision updates shouldn't break compatibility

so if the major version is the same, it should just work


but 'privacy' not 'Privacy' **does** end up on screen...


It worked! I was able to download the branched file into avid and open with all the same edits. The only thing is AVID knows it's a copy so you can't have both the original and copy open at the same time. However, since it will be different systems. Just need to to change bin file name to reflect edit or version or whatever is designated.


Why? The eventual goal is to converge on the usnic source tree being consistent and kernel style guide compliant.


Testing says no


Nevermind.. i was blind.
Joining boat (if botton or right click): 1
Leaving boat: 3
Left click: attacking


this looks good, good reuse of existing class


Yep, no problem - I just went ahead and fixed the compiled dist in master here so I could push a package for someone that was giving a demo this week. Overwrite as you see fit


Can we make this a debug-level log message and wrap `electorate` in `pr-str`?


Yes, minimizing the complexity and FunctionModel wasn't designed to be used by UnboundMethodModel, so they might be some underlying inconsistencies that I'm not keen to explore (just did a small test and it crashes because the signature of UnboundMethod is different than what's expected by the FunctionModel).


Spurious whitespace changes.

I agree the 5-space indentation on these lines is weird. However, it's coming from upstream, so if we "fix" it in GPDB, we'll just get unnecessary merge conflicts in the future as we merge with new upstream releases. (in theory anyway; I don't think there's been much change to these lines in later PostgreSQL releases)


Indeed I see the general argument for passing arguments as opposed to
global variables. What I think is annoying here is that one out of
40(?) formatters need a specific data structure leading to that it
always needs to be passed around. In combination with that the
argument is still passed in multiple places from the same global
variable that we wanted to avoid to start with. Would make more sense
to remove that formatter...


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16908](https://teamcity.mesosphere.io/viewLog.html?buildId=142818&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 236 Build time: 00:03:43


another spurious whitespace change


That won't work here's the failing scenario

Try to display an interface with a description and a native vlan and you're screwed

that [1] is assume stuff, what you want it the last interface_data item


Hi Paul,

Thanks for the pull request!

Re mysqli, this will go away by switching CodeIgniter to the 'mysqli' driver in system/config/database.php $db['default']['dbdriver'] = "mysqli"; ... this update is made in the repository, too.

What are the others you mention?

Just want to check in on these to see if we can get them out of the codebase.


same comment


be aware this will change, openshift will get it's values from the webui for memory/disk/cpus


Ah oui, ça c'est la raison pour laquelle je voulais un account par défaut pour l'événement.

Dans le workflow du khube, il y a clairement un bénéficiaire par event.


you 're a nigger


mock(JournalAbbreviationRepository.class) does not work?


The `-to` refers to `timeout`? Could we expand it `-timeout`? Or alternatively add a docstring to these explaining what they do? It's pretty apparent what they do from the code, but I started out trying to figure out "to what?". :)


Might be simpler to just have these fields in the struct, even when !USE_SSL.


:) Also this is built on Web Languages. So you use html,css,js,php in assets folder ;)


Since Git doesn't read the meta data in a .avb file and no access to video files. It will only show file as "changed" or "modified" but won;'t track actual edits. Need to use the comments to log changes or make notes on edits


I had forgotten we now use YAML for translations. Nope.


>  it worked fined with node.js 0.10.x, but following this change this will no longer be the case as far as I can see - unless Object.assign is provided through some es6 shim.

I appreciate that you're keeping an eye on this stuff :)

Johnny-Five will continue to support 0.10.x for a very long time (as long as necessary, I guess?). You can keep tabs on this by watching travis-ci as well, here's the report from this commit: https://travis-ci.org/rwaldron/johnny-five/builds/110457209


failing builds :) will investigate soon