The first and the second block of includes should be together, sorted, and should be after the local includes.


Comments on https://github.com/datashaman/putio.py/pull/2


![regexp-friends](https://cloud.githubusercontent.com/assets/1858316/13343157/963ee61e-dbfe-11e5-894c-f163dd5cb297.jpg)

Ah. Disregard about the second group. I take it valid usernames match `/^[a-z0-9]$/i`?


Shouldn't this return `true` if we reach this part of the code?


Comments to be found on https://github.com/datashaman/putio.py/pull/2


Me mandaste un Framework completo :P WTF?


The pythonic way is to write `if chans:` instead of `if len(chans) == 0:`.


First caret doesn’t seem to matter:

![first-carat](https://cloud.githubusercontent.com/assets/1858316/13343187/09487cb0-dbff-11e5-8d8f-3c14ae27b9c5.jpg)


change Name


I've changed my mind about 500 being included in this list. I suspect it will most likely *not* be a recoverable error, and should immediately fail.


si Juan, recien veo que hice cagada con el push. maniana vuelvo a subir todo, el topic0y el topic1. Disculpa el quilombo.


no problem. aprovecha y lee el código, se aprende bocha leyendo código bien hecho.


I believe you are correct.  I think you will need an extra check for all zeros.

From: Tatiana Goldina [mailto:notifications@github.com]
Sent: Thursday, February 25, 2016 10:15 PM
To: Aero-Paitil/ftc_app <ftc_app@noreply.github.com>
Subject: Re: [ftc_app] 1) Implement the new installed left beacon color sensor which works with right beacon color sensor to determine the color on beacon. (fd82a6a)

*	if (blueR >= redR && redL >= blueL) { //blue Right, red Left

Is it correct to have >= rather than > here? If blueR, redR, blueL, and redR are all 0, it means that no color can be detected, not that the color on the right is blue.

—
Reply to this email directly or view it on GitHub <https://github.com/Aero-Paitil/ftc_app/commit/fd82a6a72497219b77199fcea6dbf5f8eb26d694#commitcomment-16344233> .  <https://github.com/notifications/beacon/AIrB32Z2vQm1wJW7V5-gA5qybgNKATXjks5pn8jTgaJpZM4HjdRM.gif>


remove this comment


indent this whoel method correctly


Agreed, not secure, and not intended to be.
Essentially, we're looking to (make it easier to) avoid [Pokémon generation abuse](http://www.smogon.com/ingame/rng/bw_rng_intro), not protect credit cards.


remove comment


remove numbers below and this one...


de Una. Te avisd maniana cuando lo tenga subido.


Here in similar places below I suggest that you use a white space before the second and subsequent parameters, i.e. `callLater(MIN_REFRESH_RATE['s'], self.startTakingTemps, c)`. Please see examples in [PEP8](https://www.python.org/dev/peps/pep-0008/).

Otherwise, looks good to me.


yeah just remove all comments.


Shouldn't this be

```
<command>vcregress bincheck</command> or <command>vcregress recoverycheck</command>
```

?


I like this ;)


Ah, I just [commented on tr-design](https://github.com/w3c/tr-design/issues/52#issuecomment-189099322) about this, @marcoscaceres, @halindrome.

I think this automatic approach would only work if we stuck perfectly to natural years. But as our very first cycle of tr-design development shows, that may not happen, regardless of what [the process dictates](https://github.com/w3c/tr-design/blob/gh-pages/README.md#principles)…


This looks a little bit horrible. Platform shouldn't have to access thread states in `ds2::Target::ThreadBase`.

I see that `GetThreadState` is never used, and I also see that `FreeBSD::Platform` also defines such a function, so I assume you did it because `FreeBSD` does it?

I'm thinking it might be better to not even have a `GetThreadState` function in here.


Apple use MacOSX in all the system header, and LLDB use macosx as osType :) I'm ok to change, but from the code it make more sense to have MacOSX, only the build system (CMake) use Darwin. As you want :)


You might wanna do `DS2BUG("not implemented")` instead.


That's a good point.  However, the policy page says clearly that it can
only change on 1 January ;-)

On Thu, Feb 25, 2016 at 8:40 PM, tripu <notifications@github.com> wrote:

> Ah, I just commented on tr-design
> <https://github.com/w3c/tr-design/issues/52#issuecomment-189099322> about
> this, @marcoscaceres <https://github.com/marcoscaceres>, @halindrome
> <https://github.com/halindrome>.
>
> I think this automatic approach would only work if we stuck perfectly to
> natural years. But as our very first cycle of tr-design development shows,
> that may not happen, regardless of what the process dictates
> <https://github.com/w3c/tr-design/blob/gh-pages/README.md#principles>…
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/w3c/respec/commit/cceb8ada6e96c9f64dcad7f40d16e1802beb8ffa#commitcomment-16344447>
> .
>

--
Shane McCarron
halindrome@gmail.com


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


No commented code plz.


It's actually testing hot standby since you set `hot_standby = on` by default and query the standby below.

Tests for warm standby would be useful but separate.


```bash
Apple LLVM version 7.0.2 (clang-700.1.81)
Target: x86_64-apple-darwin14.5.0
Thread model: posix
```

Yes it doesn't compile, after a quick search, it sound like Apple don't want to implement it, at least it's what I've found. Are you compiling with this keyword on your mac ?


a [cascading] standby


Wait but isn't `ApplicationTestCase` more specific than `AndroidTestCase`...? Does it work?


Nope.


if (isCold(...))
  continue;


I think these commits are somewhat out of date and we should just delete this pull request.


latobarita: retry


Changes API


Even if they're equivalent I usually use `x is None`, this is what PEP8 says to do, it reads better as plain English and is faster:
```python
In [4]: timeit.timeit("1 is None", number=100000000)
Out[4]: 3.022216796875

In [5]: timeit.timeit("1 == None", number=100000000)
Out[5]: 5.446609973907471

In [6]: timeit.timeit("1 is None", number=100000000)
Out[6]: 3.0111279487609863

In [7]: timeit.timeit("1 == None", number=100000000)
Out[7]: 5.4324119091033936
```


What is this?


Oh I see.
> (caddr_t)1 to indicate that execution is to pick up where it left off
Maybe add a tiny comment there?


After I updated the client to v1.2.0, deadlock have not happened yet.
The problem seems to have solved completely.
Thank you very much!


Hi,

The latest stage0 snapshot knows about `armv7-unknown-linux-gnueabihf` target and therefore, as of next nightly, it's possible to switch the whole build to that:

`./configure --build=armv7-unknown-linux-gnueabihf` , etc.

The `mk` part of the `arm-unknown-linux-gnueabihf` target was recently changed to explicitly pass `-march=armv6`, so the latter `sed` command from this commit should lead to compilation failure (`-march=armv7-a` is required by gcc and not `armv7`).

Your intervention is required either way.


@davidhadrianus89 @novikesumaningtyas terkait update di Vagrantfile.example, harap run berikut untuk mengupdate vagrant yg sudah dibuat:
* vagrant halt #matikan VM vagrant dulu
* copy semua isi Vagrantfile.example ke Vagrantfile
* vagrant up
* ssh ke vagrant
* ls #sekarang folder source ada di /home/vagrant/opendims, tidak di /home/vagrant/.virtualenvs/src lagi
* rm /home/vagrant/.virtualenvs/opendims/src
* workon opendims
* cd opendims
* pip install -r requirements.txt #install package2 baru
* nodeenv -p --prebuilt #install node (agak lama kl internet pelan)
* workon opendims
* npm install -g bower #install bower
* done


sned == send


has_many :comments? mas comments não é um model e sim um campo a mais na sua tabela


This seems bad to me. You're returning a pointer to a temporary. Looks like FreeBSD code has the same issue.


This is excellent, I like the way you've broken up the sections and your titles are good.

+1


1) what is the issue being fixed here?
2) how does the change fix it?


`return ..()`


Most of this front end will have to be re written since the card placements and animations will differ. Feel free to remove the html and css page from your repo.


aha, yeah, you're right. WTF is this bs...

On Thu, Feb 25, 2016 at 11:43 PM, kokarez <notifications@github.com> wrote:

> Apple LLVM version 7.0.2 (clang-700.1.81)
> Target: x86_64-apple-darwin14.5.0
> Thread model: posix
>
> Yes it doesn't compile, after a quick search, it sound like Apple don't
> want to implement it, at least it's what I've found. Are you compiling with
> this keyword on your mac ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kokarez/ds2/commit/d13d77743af45a05b187f3c1a45e4cc13c63fe6e#commitcomment-16344476>
> .
>

--
Stephane Sezer


この辺，Optional を外したのはなぜですか？


I would have been happy to do that this year and deploy January 1. (That's why I've been fastidiously asking for branches and strict cutoff dates, and reminding people that we should not be making changes days before the date we intend to go live ;)


This function needs to be updated now that we are using it differently.
1) it doesn't write to xpdUser/export, only to the glbl.archive_dir
2) it doesn't clean the export tree before bundling (we will no long offer this big bundle for users to export)


Funny enough, when you don't specify `preload` attribute, most browsers will default to `preload=auto`. So by adding `<audio>` on the homepage, every visitor was preloading every one of our episodes. This was regardless of whether or not they actually listened to it. This resulted in a 5x bandwidth bill! Yikes.


That's a good point. The way I see it, Darwin is the kernel, MacOSX is the
OS. If we were to build this for iPhone, the source would be the same, the
kernel would be the same, but the OS would be different. No?

On Thu, Feb 25, 2016 at 11:41 PM, kokarez <notifications@github.com> wrote:

> Apple use MacOSX in all the system header, and LLDB use macosx as osType
> :) I'm ok to change, but from the code it make more sense to have MacOSX,
> only the build system (CMake) use Darwin. As you want :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kokarez/ds2/commit/d13d77743af45a05b187f3c1a45e4cc13c63fe6e#commitcomment-16344456>
> .
>

--
Stephane Sezer


Is there a test for JavaScriptReverseRouter.create?


Probably Container interface should stay in container.go


You will get no arguments from me.  I am but a humble servant. <* tugs
forelock *>

On Thu, Feb 25, 2016 at 8:53 PM, tripu <notifications@github.com> wrote:

> I would have been happy to do that this year and deploy January 1. (That's
> why I've been fastidiously asking for branches and strict cutoff dates, and
> reminding people that we should not be making changes days before the date
> we intend to go live ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/w3c/respec/commit/cceb8ada6e96c9f64dcad7f40d16e1802beb8ffa#commitcomment-16344567>
> .
>

--
Shane McCarron
halindrome@gmail.com


ditto this, Process interface should stay in process.go since we should be able to implement it


We should be able to use the card class and parts of the game class. But most of the remove/move/etc won't be applicable here. I also think we'll end up needing to rewrite the routes, but we'll see what maven allows us to auto generate.


I had to move it due to specs.Process to ensure it pulled in windowsspecs, not opencontainers/specs. Part of the gross hack I mentioned.


I'm not sure why you moved implementation details into the header.


Cool.


Ditto on my comment about specs.Process


@caocampb The docstring for inputs needs some attention.  The lon and lat variables should be floats since the netCDF variable declaration has them both set to a double precision floating point number.

Also please change variable 't' to something more descriptive.  It is a good habit to write longer variable names that communicate the purpose and meaning of the variable.  In this current version of the function it's focused on temperatures so temps or temperatures would make good sense.  But this will break down when someone is passing in a Numpy Array of precipitation data.  Toward a more generic and reusable design a name like inputData or even inputArray will help with code readability.


oh I see, this has a dependency on specs.Process...

maybe a way around this is to say type ProcessSpec specs.Process in process_unix.go and type ProcessSpec int in process_windows.go until we can support opencontainers


Unnecessary file


no need for the default.


@athanatos My bad, got it, its needed..


> If https://yourextractor.com/ shows more than one obviously infringing video, reject.

Just more than one, that is all? Time to kick out support for YouTube..
> Otherwise, use our good judgement; usually include.

Sounds like a sensible approach.
> We most definitely put websites into categories. It would be far more damaging - both from a moral as well as a legal standpoint - [..]

Morals. That what you put on the table when you run out of real arguments.

:wink:


Old DB, not in use anymore, will upload current DB being used


@caocampb It is always tempting to concatenate file paths within python, but this causes issues when using the code on *nix OS vs. Windows.  The best way to ensure consistent file pathing is to use the os.path module from the python standard library.  See https://pymotw.com/2/ospath/#building-paths  for a complete example.


Config-Access


Old DB, will update with new DB


Do you really need this vs the IsSpecialType extension?


I'd love to not have that weird symbol. Explicit >=, < are more obvious.


Optional型の変数に！をつけるのがあまりよくないと聞いて勘違いしておりました、、、


Maybe resolvers rather than source langs


what about string or object?


I am so sure I do not know what I am doing


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


間違った変更です


really its work properly thanks.


Thanks for starting this project!


Maybe...


Ha-ha, sorry for commenting on this before. Great job!!! :)


Nah..Yesterday i tried compiling webview,end up with errors,so i sleep. I will try today maybe.


Who wants to constantly type `collections` though. Lists are a commonly used type.


I don't like the idea of either.


The crucial thing with getting the supply point from a location is that the location should control the linkage (as is done in [`linked_supply_point`](https://github.com/dimagi/commcare-hq/blob/master/corehq/apps/locations/models.py#L401-L408)).

If you archive a location or change it to no longer track stock, we don't actually delete the supply point case, because in general we don't ever delete cases, so we just close it and remove the `location.supply_point_id` property.  Always going from location to supply-point assures that the location object is the source of truth.


Is this in use?  Can we change it to `location.linked_supply_point` as in the couch version?


You can use Icon.createWithBitmap() to draw a better icon


![internalnaming](https://cloud.githubusercontent.com/assets/10700823/13343770/b6e904a2-dc04-11e5-8e38-59b5369b4b1c.png)
Example Image, In some cases the tooltip can't even fit on the screen


Please use /res/values/strings.xml -- don't hard code you strings


Cool, but you probably shouldn't push directly to dev.


Here is to fetch link name using ```netlink.LinkByName ``` with preferred interface ```iface```


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


Do you really need the .svn folder?


Fetching all route list using ```netlink.RouteList ``` of the interface mentioned above


(Unlit)Redstone Lamp should be a LightSource


Add that one file (only) after cloning the Chromium repo, not before like you did


I think that sentence could be clarified.
Is it "Returns a map from each possible pair of annotations to their glb."


@caocampb I appreciate that you gave credit and a link to the solution you found. :+1:


This wants to read `memcached`.


Since we might have more than one result list and/or may have ```nil``` result, check the result has ip address and returns very first ip address


:heart: this, and that's an awesome commit message!


No, if you are talking of jenkins log? ,then no,i have already rearranged everything in a proper way.Not to worry. Don't think about it.


@olivierlacan could we pull that commit message into the README? Would you like a PR to do that?


@latobarita: retry


Add index_en.html, the English version of index.html


@rjmccall, is this more or less what you were thinking? One thing I'm running into right now is a bus error in deallocating deinits of _CollectionBox. The crash log tells me:

```
  * frame #0: 0x00000001007511b8
    frame #1: 0x000000010017c14a libswiftCore.dylib`Swift._CollectionBox.__deallocating_deinit + 58
    frame #2: 0x0000000100127101 libswiftCore.dylib`generic specialization <protocol<>, Swift._TeeStream<Swift.String, Swift._Stdout> with Swift._TeeStream<Swift.String, Swift._Stdout> : Swift.OutputStreamType in Swift> of Swift._print_unlocked <A, B where B: Swift.OutputStreamType> (A, inout B) -> () + 2065
    frame #3: 0x00000001001d7b5e libswiftCore.dylib`function signature specialization <Arg[0] = Owned To Guaranteed, Arg[1] = Owned To Guaranteed and Exploded, Arg[2] = Owned To Guaranteed and Exploded> of generic specialization <Swift._TeeStream<Swift.String, Swift._Stdout> with Swift._TeeStream<Swift.String, Swift._Stdout> : Swift.OutputStreamType in Swift> of Swift._print <A where A: Swift.OutputStreamType> (Swift.Array<protocol<>>, separator : Swift.String, terminator : Swift.String, toStream : inout A) -> () + 798
    frame #4: 0x00000001001d84f2 libswiftCore.dylib`function signature specialization <Arg[0] = Owned To Guaranteed, Arg[1] = Owned To Guaranteed and Exploded, Arg[2] = Owned To Guaranteed and Exploded> of Swift.print (Swift.Array<protocol<>>, separator : Swift.String, terminator : Swift.String) -> () + 194
    frame #5: 0x000000010012aebd libswiftCore.dylib`Swift.print (Swift.Array<protocol<>>, separator : Swift.String, terminator : Swift.String) -> () + 29
    frame #6: 0x0000000100001221 test`C.foo(self=0x0000000100d00aa0) -> () + 481 at test.swift:9
    frame #7: 0x0000000100000e9c test`main + 140 at test.swift:13
    frame #8: 0x00007fff865b85ad libdyld.dylib`start + 1
```

for code:
```swift
class C<T> {
  let x: T
  init(x: T) {
    self.x = x
  }
  func foo() {
    print(1)
    print("Hello")
    print(self)
  }
}

C(x: C(x: 1)).foo()
```

The print of `1` and `"Hello"` work fine. Does anything in the above stick out to you?


not working


Haha damn, the wacky world of HTTP. Man I need to learn more about all
that, been leaning too much on my old school sysadmin LAMP stack
background. One of these days I'll bang through the Tangled Web.

On Thu, Feb 25, 2016 at 8:54 PM, Osman Surkatty <notifications@github.com>
wrote:

> Funny enough, when you don't specify preload attribute, most browsers
> will default to preload=auto. So by adding <audio> on the homepage, every
> visitor was preloading every one of our episodes. This was regardless of
> whether or not they actually listened to it. This resulted in a 5x
> bandwidth bill! Yikes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/braxtone/soundsecurity/commit/c206db24a8d01ea72594a3748ca46a3ca4f33a1b#commitcomment-16344571>
> .
>


That's very odd that the case_id changed.  What signaling are you referring to?  (I [recently got rid of](https://github.com/dimagi/commcare-hq/pull/9946) a lot of the signaling that was happening between the location and supply point)


I went ahead and did the PR. See https://github.com/nats-io/ruby-nats/pull/94


This was a PITA :frowning:  For some reason, when I used ``pydoc.locate`` to dynamically locate the filter classes, they weren't returning True for ``issubclass(cls, filter_class)``.  Any ideas would be appreciated.


@olivierlacan lol, basically took no longer to create the PR than to post a comment asking if you wanted one, so here: https://github.com/dependent/dependencies/pull/1 :)


saw approval from graydon
at https://github.com/stellar/stellar-core/commit/c15747750b248cb7e75e306370daecd032ae8a05


merging stellar/stellar-core/dm/find_pg = c1574775 into auto


stellar/stellar-core/dm/find_pg = c1574775 merged ok, testing candidate = 623aea70


This change, which made it into JBoss EAP 6.4.4, gives me the following exception when using SOAP with attachments:
```
java.lang.ClassCastException: javax.mail.internet.InternetHeaders cannot be cast to java.util.Map
    at org.apache.cxf.binding.soap.interceptor.SoapActionInInterceptor.getSoapAction(SoapActionInInterceptor.java:86)
    at org.apache.cxf.binding.soap.interceptor.SoapActionInInterceptor.handleMessage(SoapActionInInterceptor.java:121)
    at org.apache.cxf.binding.soap.interceptor.SoapActionInInterceptor.handleMessage(SoapActionInInterceptor.java:45)
    at org.apache.cxf.phase.PhaseInterceptorChain.doIntercept(PhaseInterceptorChain.java:272)
    at org.apache.cxf.transport.ChainInitiationObserver.onMessage(ChainInitiationObserver.java:121)
    at org.apache.cxf.transport.http.AbstractHTTPDestination.invoke(AbstractHTTPDestination.java:249)
    at org.jboss.wsf.stack.cxf.RequestHandlerImpl.handleHttpRequest(RequestHandlerImpl.java:97)
```

I guess the CastUtils.cast construct was there for a reason.


That make the `assert(size);` superfluous though. If you care to drop that line, please fix the formatting on line 254 while at it.


Can you copy the docstring from `get_supply_point_by_location_id` above?  And/or make a private method or something?  It'd be good to make sure this doesn't accidentally get used instead of `location.linked_supply_point()`.


The syntax looks strange. What does it do? Why not `self .deleteRowAtIndexPath(indexPath)`?


asd


I remember to fix that, but most likely in an other commit. I will bring back the fix here.


I made a mistake


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


stellar/stellar-core/dm/find_pg = c1574775 merged ok, testing candidate = 069c0b5d


sounds like a solid plan.
-brady


please remove VideoConfigAVCIntraPeriod since it is useless. intraPeriod can be set by VideoParamsCommon and idrInterval set by VideoParamsAVC.


It shouldn't, it's already been checked if texture type is GL_TEXTURE_RECTANGLE [here](https://github.com/igv/mpv/blob/19ca9aabbb44de3b8c6508b1162518592fc53365/video/out/opengl/video.c#L884)


Yes it is :) This function will be used, but also moved in the mach files. I will remove this from this commit and change the impl in the mach file to return the `system state` and convert that in the Thead code.


wrong file


See issue thread https://github.com/craigsapp/midifile/issues/20 which discusses this addition to the code and gives a short program to test the new code.


Hello,
My name is Tim Hurley and i am an amateur games programmer. I have written code for a few different programs, including Flash and Unreal Engine 4. Because of this I have experience in Action-script 3 and Blueprints. I have also looked into Unity as a game engine, and therefore have looked at Javascript.

Please find attached files to examples of my work.


The problem with `OnHold` being treated as an error is that `eos-autoupdater` will not update its timestamp, so in 15 min when `eos-autoupdater.timer` expires, another update attempt will be made, when we actually want the same behavior as when the system is already up-to-date, which is (by default) to try it again in 14 days. I wasn't aware of the side effect of ostree pulling all branches from the config file. In my tests it didn't seem to pull anything, but I might have missed something.

On your second comment, we're not really dereferencing a NULL pointer, as printf (and g_strdup_printf) will resolve the %s to `(null)`. But it is simply a matter of skipping these lines if `branch` is NULL.


Yes, apparently I changed this in my compiling machine but it never made to the final commit, sorry and thanks for the catch.


There are Travis failures when using the non-redundant form.

To be able to `bundle exec rake checks`, bundler needed those files in the :test group, it seemed.

> On 26 Feb 2016, at 00:53, Petr Korolev <notifications@github.com> wrote:
>
> I believe, if we specify dependencies in gemspec - we don't need to duplicate them here, isn't it?
>
> —
> Reply to this email directly or view it on GitHub.
>


saw approval from graydon
at https://github.com/stellar/stellar-core/commit/c15747750b248cb7e75e306370daecd032ae8a05


merging stellar/stellar-core/dm/find_pg = c1574775 into auto


stellar/stellar-core/dm/find_pg = c1574775 merged ok, testing candidate = 2cfcf962


Wasn't sure if that meant the unset was going to be added?


Fixed with my next kernel version.

Thanks again for reporting.


Do you really want to allow empty columns?


keyPeriod equal to intraPeriod() * (m_videoParamAVC.idrInterval + 1), so why we need this parameter to be set? Or maybe we can keep keyPeriod and remove idrInterval.


`</div>` could also be sorted in a constant `DIV_TAG_CLOSING`.


Not immediately apparent why this is imported. Where/what is it used for?


m <- NULL
        set <- function(y) {
                x <<- y
                m <<- NULL
        }
        get <- function() x
        setmean <- function(mean) m <<- mean
        getmean <- function() m
        list(set = set, get = get,
             setmean = setmean,
             getmean = getmean)
}


Thanks for putting this into a method! Cleans things up!


Hm. It might be even a hardware limitation, although I think it is rather unlikely.
OPX is very similar to OPO, and there I was able to reach higher charge rates.

Think it requires some additional investigations then. But maybe you have done them already and came to the conclusion it is not advicable?


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


Missing newline.


Look Good To Me. Thanks!


No, wait, this is already all skiped. Did you check the latest version of `eos-updater-poll.c`? Please see these lines:
```
  if (!branch) /* this means OnHold=true */
    {
      g_task_return_pointer (task, NULL, NULL);
      goto cleanup;
    }
```
I think we're already good here, put please tell me otherwise.


Probaly


return true


Do you want to allow empty columns?


Just pushed a fix for this one.


新建主页项目


@rleonid add a simple TODO, changed 3 -> 2.


whats on line 172? EOS?


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


Thank you for fixing this. :3 It's essential when you want to put a challenge door to your ship. ^w^


Hi

how can I pass the **--insecure** command to openstack-install


You press 'G'.

Message.sender = "Joseph";

On 02/25/2016 08:12 PM, ParkerMc01 wrote:
>
> How do you go to a planet now then?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TEAMModding/FutureCraft/commit/6363c0dcbd22073c1462df29a2fc16abb925a15e#commitcomment-16343172>.
>


FYI @mfischer-zd no more slowdown when connection to the same servers multiple times


возможности:
отображение времени
установка времени выключения Windows (в текстовой файл)
отображение времени выключения Windows


If you guys fix bugs in your own branch I'd appreciate if you took the time to submit a `PR`. Thanks.


ディープリンクについてでいいよ


Because this is in the core wouldn't it always be true? So wouldn't the addon plugins, like Text Limiter, need this clause instead?


Also run the command I provided instead of:
gclient sync --nohooks


Reference: 1.1. Getting Started - The first C Program
The C PROGRAMMING LANGUAGE, Second Edition, Brian W. Kernighan & Dennis M. Ritchie


Just like I stated for rwmb_the_value, because this is in the core wouldn't it always be true? So wouldn't the addon plugins, like Text Limiter, need this clause instead?


直しました！


Why is the constant prefixed with the class? We never did that somewhere else, did we?


Fixed logical error with this functor.


Initial Commit


q


Due to one of the (innumerable) limitations of the Java language, there is no way to access the members of an anonymous class not already provided by the interface it implements. So I had to resort to making these throw-away named classes that will only get instantiated once.


All the Weighter class instances are explicitly instantiated down here. They are defined above.


I have already checked out branch No need of this


> Ah. Disregard about the second group. I take it valid usernames match

right right.  however, then _again_, I have a subsequent PR to enforce that usernames are restricted to a non-hyphen pattern on the User model itself, which seems like the better place to enforce that username logic. i agree, the second group can be simplified.  i will update it.  given that its not critical, if youd be kind enough, let me settle that in my next PR (just the hassle of committing, branch change, new commit, branch change, re-merge, etc).

> First caret doesn’t seem to matter:

it may be a low likelihood, but this is the case i was after
<img width="285" alt="screen shot 2016-02-25 at 10 56 57 pm" src="https://cloud.githubusercontent.com/assets/1003261/13344439/29b3e8ae-dc13-11e5-8065-3e9d76906b70.png">


Thanks @maxpoulin64 and @JocelynDelalande!


add by wangxy


@EnTeQuAk Shown as `You can <ahref="%(url)s">edit additional settings</a> if you'dlike.` in https://pontoon.mozilla.org/zh-CN/amo/LC_MESSAGES/django.po/.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17367](https://teamcity.mesosphere.io/viewLog.html?buildId=146757&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 Build time: 00:02:00


[NOR2X1.zip](https://github.com/jimmery/Absolute-Threshold/files/147724/NOR2X1.zip)
Fixed output


Yes this comme from the man :) It's same for ST_STEP. I will add a comment.


some tests failed:
failure: https://travis-ci.org/stellar/stellar-core/builds/111932671


Wouldn't it be easier to not pass the hotkey at all?


Yes this code come from freebsd, I didn't check verify this part :) I will try to see what I can do.


Removed static instance counter because: (1) it's inflexible and will assign lectures numbers in the order instantiated, which is not generally desirable (eg user or dev might want to instantiate lectures from last to first); (2) static variables are evil; (3) a lecture number member variable isn't all that useful in the first place. The container that stores them will most logically sort them by date.


This iterator is virtually identical to a ListIterator, so I removed this.


Added capacity constructor to NRList to parallel constructor in Roster.


Lol ok 👍


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


When was the last time someone told you you were amazing?


This is a clever method!


Same story than PTrace :) apple want to do better!


CSR fix.


Added remove function to NRList. Currently, the way this works would suggest that all feedback given by the removed student must be purged, or else another added student could get assigned the same code as the former student. This will cause the new student to accidentally get dumped with all of the former student's feedback.


Attempting to replace all functions accepting arrays with functions accepting Lists, because they allow greater genericity.


https://nixos.org/nixpkgs/manual/#sec-pkgs-overridePackages

> Warning: this function is expensive and must not be used from within the nixpkgs repository.

Please revert this.


The ProtoApp class simply packages together all the other classes we've made so far. It combines them together in a logical interface and possesses functions that require the use of all these classes. For example, Roster alone cannot accept unprocessed feedback csv files without using NRList to encode their netIDs. Student cannot remove duplicate entries without a collection of Lectures to identify duplicates. Etc. I find this design to be better in multiple ways over relying on static methods and variables.


Oops, you're right. This line should be in addons instead of the core plugin.


I'm not a fan of the colors for the text, it's very hard to read and covers the logos.


Correct me if I'm wrong, but it seems like this modifies the object exported by `properties`. Wouldn't this cause problem if some other module were to also import `properties` and expect it to work normally?

So perhaps it would be safer to say:

```javascript
return sync.await(properties.parse(source, sync.defer()));
```
(Or alternatively just use the async-api without synchronize.)


I'd call this `decodeUnscaledValue`


Can confirm is working on go1.6 linux/amd64. Thanks.


And this one, `encodeUnscaledValue`


LGTM.


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


@olivierlacan awesome. I thought about taking a stab at adding sections to aid readability but figured that would evolve once the content was there. This looks like a great start.


only win8 and the above guests have the pkgmgr problem ?
like Win7, Win2008r2


Oh, now that I read documentation of `synchronize` properly it seems that it preserves the original semantics as well. So it shouldn't break other uses.

Still, feels a bit fragile to modify objects exported by others: I can come up with a few ways how it could break things, even if they are not very likely.

Anyway, this covers my needs perfectly. Thank you very much!


Reference: 1.2 Variables and Arithmetic Expressions - Fahrenheit-Celsius table
The C PROGRAMMING LANGUAGE, Second Edition, Brian W. Kernighan & Dennis M. Ritchie


Why do we need the precision here? Isn't the unscaled value + the scale sufficient to reconstruct the decimal value?


Added FIELD_NAME_CNN


添田です。
文字列は、ダブルまたはシングルクォーテーションで括る必要があります。
このままでは文字として扱えません。


Tip: you add [ci skip] somewhere in commit message, that commit will not be processed by CI.


:+1:

What's with this `#` syntax for prototype methods? Is that a standard or are we cribbing from Ruby? I've used `::` elsewhere, which is valid CoffeeScript syntax. If the JS world has a standard other than the horridly ambiguous `.` or the verbose `.prototype.` I'd love to know what it is.


Should this raise an exception, instead of being defensive?


The boundary force checks are not working on OS X.


From what I can read, they are differences (lib, usage, etc...), but also
have some stuff which look to be the same (mach).

I think the day where someone will do the port of iOS, we will need to
found what part of the code are the same and create a Darwin folder with
the same usage than the POSIX one.

So at this point I'm ok to move all under Darwin since we don't use fancy
lib or UI.

 --
Kemerit plijadur! - Corentin DERBOIS


Move this in SystemEnvironment? This could be problematic on slow WAN links, and some users may need to bump the timeout.


I don't see any usages of this class


ah yeah it does stink:

firefox:

chrome:

safari:

reverting back

> On Feb 26, 2016, at 12:10 AM, Kevin Olson <notifications@github.com> wrote:
>
> I'm not a fan of the colors for the text, it's very hard to read and covers the logos.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/kolson133/agile_sp_2016_BlueTeam/commit/c62bd4556d9b10af98a49430bf152e065ec4a94a#commitcomment-16345255>.
>


If they're not used by current functionality then I'd say lets not have them. The `tabs` package doesn't need them?


Call this variable just `result`


Sorry, didn't prepare it yesterday. Will try to do it soon.


firefox:

chrome:

safari:


/Users/apple/Library/Developer/Xcode/DerivedData/Yep-gouotshhrmygmlfbwvfancwfrsyn/Build/Products/Debug-iphoneos/Yep.app/Yep": No such file or directory


添田です。
ここは、入力を数字にしてほしかったです。課題内でも1なら2ならと記載してあったと思いますので。


since the portAudio v19 api isn't compatible with older versions a call to  ```Pa_GetVersion()``` might be considered here. As it doesn't exist in older versions this should result in the try_compile failing?


Really? Why?


Please remove this block


Awesome!!! :)


added two TODO comments for you to check my work


添田です。
回答は問題ありません。ただ、stiringではなくstringかもしれませんね。


Add a comment explaining why we need to fill the top 8 bytes. Without the understanding that LONG_DECIMAL_LENGTH is 2 * SIZE_OF_LONG, and that we're trying to widen from a long to 16 bytes, it's not clear what's going on.


add version note


添田です。
想定外の処理が無いですね。。。


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


I styled both views - just the Expression field is still a bit weird


Why is this method necessary? I only see it being called in tests.


添田です。
default自体用意しない方が良いですね。なぜなら、何もする必要が無いからです。


Dejando un comentario


添田です。
ここはAを30個表示してほしいのではなく、変数の中にAが30個入った状態にしていただきたいのです。
処理を考えてみてください。


添田です。
ここを文字列にする必要はないですよ。


need to decide about some styling..
user can subscribe to "client". i had to do some css to existing. some
media queries got damaged though.

subscribed clients can be reflected as a star/heart above/on left of the
client sensors. see below(level 1 our side)
can be also as a bottom border under the relevant "Client".

[image: Inline image 1]

On Fri, Feb 26, 2016 at 1:19 AM, Jenya Y. <notifications@github.com> wrote:

> I'll review it tomorrow.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Enttoi/enttoi-ui-web/commit/03d21e859ca127d0e732d8f5e61c582b9d0d3f9d#commitcomment-16340650>
> .
>


instead of fprintf use ```cst_errmsg``' instead of ```fprintf``` to ensure portability (even if it's only palmos that seem to have problems with this)


This is personal preference and not a mandatory change. but having code on the same line as the the if statement makes the code hard to read.


And maybe this interface change will involve hevc encoder, could you help to change it also if necessary.


最好做一些删改操作


Yes I will


This is hard to read. Is this just trying to write the `unscaledValue` in the bottom 8 bytes of the array (in big-endian order)? If so, this would be easier to read:

```java
Slice result = Slices.allocate(LONG_DECIMAL_LENGTH);
if (unscaledValue < 0) {
    result.fill((byte) 0xff);
}
result.setLong(SIZE_OF_LONG, Long.reverseBytes(long));
```

or even
```java
Slice result = Slices.allocate(LONG_DECIMAL_LENGTH);
if (unscaledValue < 0) {
    result.setLong(0, -1L);
}
result.setLong(SIZE_OF_LONG, Long.reverseBytes(long));
```


It's a [heredoc](https://en.wikipedia.org/wiki/Here_document). EOS means "end of string", but you can use anything if you want.


Shouldn't we put this into a ``finally`` block? Just in case.


生成不同样式的二维码和条形码。目前最后一个二维码，无法扫描获得信息。

商品条码：主要包括厂商识别代码、商品项目代码和校验码，对应的位数分配方式有两种：7-5-1和8-4-1。 厂商识别代码由中国物品编码中心统一向申请厂商分配（根据国际上的规定，中国的厂商识别码的开头3位数必须在690和695之间）。商品项目代码（690、691开头的厂商识别代码，商品项目代码是5位；692~695开头的厂商识别代码，商品项目代码为4位）由厂商自行分配。这样来看，如果你买的东西是中国公司的产品，但条码开头三位不是在690~695之间，那一定是假货。如果无法查询到厂商识别码对应的公司，那也一定假货，因为正经公司都会到中国物品编码中心备案的（可以到 http://www.ancc.org.cn/service/tools_09/tools.aspx 查询）。
[GS1已分配给国家(地区)编码组织的前缀码](http://www.qinms.com/webapp/barcode/gs1bianma.html)

（补充）关于编码信息：
Code128A字符集 包括大写字母、数字、常用标点符号和一些控制符。
Code128B字符集 包括大小写字母、数字、常用标点符号。
Code128C字符集 为纯数字序列。
Code128Auto 是将上述三种字符集最佳优化组合。
EAN128条码生成 是由UPC/EAN指定代表意义规则的128码，编码方式同code128条码。
Code39条码生成字符集包括数字 、大写字母以及- . $ / + % * 空格等字符,其中"*"只用于标记开始和结束。
Code93条码生成是 full ASCII 模式，可使用ASCII全部128个字符。
库德巴码（Codabar）条码生成，字符集包括数字和- $ : /. + 以及ABCD等字符，其中ABCD只用于开始或者结尾，作为标识符使用。
交叉25码（Interleaved 2 of 5）条码生成，常用于物流管理，字符集仅为数字且个数为偶数,为奇数将自动在前面加"0"。
Code11条码 只允许11种字元，分别是0-9和"-",为降低检查错误率，可使用两位的检验码。
MSI条形码 必须是纯的数字0-9，带有一位检验码。
EAN13商品条码是纯数字，而且位数是12位，在编码过后外加一位校验码，组成13位数字。
EAN8商品条码是纯数字，而且位数是7位，在编码过后外加一位校验码，组成8位数字。
UPC-A条码商品条码是纯数字，而且位数是11位，在编码过后外加一位校验码，组成12位数字,主要在美国和加拿大使用。
UPC-E条码商品条码是纯数字，是由UPC-A缩减而成，位数是7位，而且首位必须为0，在编码过后外加一位校验码，组成8位数字。


We should be able to get two consecutive longs from the block if we wanted to avoid creating this intermediate `Slice`. So, something like:

```java
long top = block.getLong(position, 0, SIZE_OF_LONG);
long bottom = block.getLong(position, SIZE_OF_LONG, SIZE_OF_LONG);
return new SqlDecimal(decodeUnscaledValue(top, bottom), ...);
```


OUTPUT

0	-17
20	-6
40	4
60	15
80	26
100	37
120	48
140	60
160	71
180	82
200	93
220	104
240	115
260	126
280	137
300	148


0 never great than max, so BUG_ON useless.


生成不同样式的二维码和条形码。目前最后一个二维码，无法扫描获得信息。

商品条码：主要包括厂商识别代码、商品项目代码和校验码，对应的位数分配方式有两种：7-5-1和8-4-1。 厂商识别代码由中国物品编码中心统一向申请厂商分配（根据国际上的规定，中国的厂商识别码的开头3位数必须在690和695之间）。商品项目代码（690、691开头的厂商识别代码，商品项目代码是5位；692~695开头的厂商识别代码，商品项目代码为4位）由厂商自行分配。这样来看，如果你买的东西是中国公司的产品，但条码开头三位不是在690~695之间，那一定是假货。如果无法查询到厂商识别码对应的公司，那也一定假货，因为正经公司都会到中国物品编码中心备案的（可以到 http://www.ancc.org.cn/service/tools_09/tools.aspx 查询）。
GS1已分配给国家(地区)编码组织的前缀码

（补充）关于编码信息：
Code128A字符集 包括大写字母、数字、常用标点符号和一些控制符。
Code128B字符集 包括大小写字母、数字、常用标点符号。
Code128C字符集 为纯数字序列。
Code128Auto 是将上述三种字符集最佳优化组合。
EAN128条码生成 是由UPC/EAN指定代表意义规则的128码，编码方式同code128条码。
Code39条码生成字符集包括数字 、大写字母以及- . $ / + % * 空格等字符,其中"*"只用于标记开始和结束。
Code93条码生成是 full ASCII 模式，可使用ASCII全部128个字符。
库德巴码（Codabar）条码生成，字符集包括数字和- $ : /. + 以及ABCD等字符，其中ABCD只用于开始或者结尾，作为标识符使用。
交叉25码（Interleaved 2 of 5）条码生成，常用于物流管理，字符集仅为数字且个数为偶数,为奇数将自动在前面加"0"。
Code11条码 只允许11种字元，分别是0-9和"-",为降低检查错误率，可使用两位的检验码。
MSI条形码 必须是纯的数字0-9，带有一位检验码。
EAN13商品条码是纯数字，而且位数是12位，在编码过后外加一位校验码，组成13位数字。
EAN8商品条码是纯数字，而且位数是7位，在编码过后外加一位校验码，组成8位数字。
UPC-A条码商品条码是纯数字，而且位数是11位，在编码过后外加一位校验码，组成12位数字,主要在美国和加拿大使用。
UPC-E条码商品条码是纯数字，是由UPC-A缩减而成，位数是7位，而且首位必须为0，在编码过后外加一位校验码，组成8位数字。


Will a portAudio error map correctly to normal errnos? should we take care to display the error message inside the portaudio before returning so any important info reaches the user?


I'd call this method `encodeScaledValue`.

BTW, if this method is needed for tests, make it package-private.


If tracking stock gets reenabled or it gets unarchived do we then create a new case?


FYI, I cross-linked so that his method won't get lost: https://github.com/dotnet/orleans/issues/1497#issuecomment-189132159.


I did so in the next commit.  I actually updated the file before this one, but for some reason git extensions didn't recognize that.


I don't think we need this at this time?


With AWS we are going to stored data to SQS  with node intially i have created front end html page
from where you can posting your data to your node js server then further node wil stored that with connected SQS url

steps to follow
1.install npm aws-sdk
2.in your project folder add config.json which will include your secretAccessKey,accessKeyId and region region
3 add the file with require(../config.json)
 and then your node general way to run your code
node file_name


Either is fine.  I thought my way was easier to read.


This doesn't seem to be used anywhere in this commit.


fix #12


QRcode


Haha now we both did it.


Oups, I'm sorry, rebase problem again... I'll fix it in v2


I like this one! A good use case for metaclasses.


ここにも[]を追加


Why you should hire me


make sure to also quit it, please


If you do a `meteor show reactioncommerce:reaction-launchdock` you'll see that 0.2.1 is a local version, not yet published.  This will cause package warnings and not  install/run for fresh clones of the development branch that haven't done a reaction pull first.  Packages that aren't in the reaction repo, we should publish releases for when we increment the version in dependencies.

```

Recent versions:
  0.1.3  January 18th, 2016
  0.1.4  January 20th, 2016    installed
  0.1.5  February 11th, 2016   installed
  0.2.0  February 17th, 2016   installed
  0.2.1  February 22nd, 2016
```


OUTPUT
  0  -17.8
 20   -6.7
 40    4.4
 60   15.6
 80   26.7
100   37.8
120   48.9
140   60.0
160   71.1
180   82.2
200   93.3
220  104.4
240  115.6
260  126.7
280  137.8
300  148.9


Grammar issue introduced here, will fix/squash after merge.


添田です。
良いですね。これもまた一つの答えです。
ただし、もう少し簡単に終わらせる事もできます。重要なのは、checkedを付けるか付けないかなので、
INPUT全体の表示を切り替えるのではなく、checkedだけ切り替える様にできたらより良いです。


What are the constraints on how big or small `bytes` can be? It's big-endian, too, no?


I'll take a look at styles later, but I have two things that I wonder about:
* Shouldn't I supposed to be auto-unsibscribed once I got notified?
* Did you look into browser Notification API? This would be a great way to deliver notifications.

P.S. can you revert changes in `web.config` and `config.js`?


Hey @amaitland ,

you're right but I didn't because that functionality is specific to our app. In our app we don't manipulate any resource on the OnBeforeResourceLoad and OnResourceLoadComplete but still we manipulate resources at some other point of the resourcehandler lifecycle. So if the popup browser is disposed at those moments (which seems to be caused be a race condition in CEF between the resource loading and the browser disposal) we don't really need to throw an exception.


Oops. ok thanks for pointing out that typo. Fixed :+1:


添田です。
保存場所の表示ではなく、実際に保存したファイルを表示して欲しいです。


添田です。
このelseの処理は正しく無いと思います。$_COOKIEは書き換えても意味がありません。
もし、クッキーの中を書き換えたい場合、setcookieを利用してください。


I'm confused about this encoding. If, say, `bytes` has size 2 with the following contents: `[0x80, 0x00]`, is the number that it represents 32768 or -32768 (i.e., is it interpreted as `0x0000_0000_0000_8000` or `0xFFFF_FFFF_FFFF_8000`)?


you can consult the boot in the base, also replace the adbd in the boot with the base or m7


No, we look up the old closed case and reopen it [here](https://github.com/dimagi/commcare-hq/blob/442531eef47f0d1b13cb24086cff8583a6c06a31/corehq/apps/commtrack/models.py#L504-L515).  This uses the `get_supply_point_by_location_id` dbaccessor, so the closed case maintains the link between the two.  I'm not 100% happy with that system, but that's what we've got right now.


canola spelled wrong


Please remove this change.


@vendethiel I don't use both :) only IRC and some slack rooms rarely, and twitter XD


还有几点问题，这个这个是 public 的配置项，我觉得应该慎重一些：

- 下划线开头在一般指代私有字段，私有字段一般认为是可以随意重构掉的，使用私有字段一般被认为是 hack 的行为，这里是否合适。
- API 是缩写，应该大些。
- to replace AV.serverURL


Wow, I'm surprised this works :) Maybe just leave the ``http()`` as a private method?


Is there a reason why you put `imageElement` and `imageSrc` in the highest scope?


if the base(36) changed this will break the extraction.
also packed obfuscator is a used by many of sites that are not supported directly(supported by Generic Extractor).


Awesome commit!


The problem is that, surface->view is also tied to surface buffer attach callbacks. Which cause view to act accordingly. This will be problematic since view has no concept of multiple views. The last surface buffer acks will modify the state of view. E.g. see the surface_attach function in this file.


Ah you're right, sorry!


btw... what you suggested maybe redution but maybe complication


I'm generalizing this.

> packed obfuscator is a used by many of sites

Could you point out some example besides Vidzi, Videomega and iQiyi?


To make the frontend more responsive in updating the robot state, we have to decrease the interval for the logging anyway. Maybe 2 seconds will be right number here.


@rebootcode
Very appreciate of your contribution.
The reason I postpone this pull request is that the `web` middleware group was initiative first in laravel 5.2.
This means accepting this PR, we may lose backward compatibility.

What do you think?


при смене ориентации экрана как себя ведет данный кейс, нет вылета с критом?


第一点我认为要么去掉下划线，区分用户配置与程序内部全局使用的变量，要么给能配置的那些配置加上 getter setter。


Anything that's in build-essentials should not be depended on unless a specific version is required

https://lintian.debian.org/tags/build-depends-on-essential-package-without-using-version.html

As for the other removals I couldn't see anything that actually needed git or subversion to build (once you have the source of course). I wasn't 100% sure about libtool-bin vs libtool I was hoping to get some feedback about that from a debain sponsor/mentor.


tab was not replaced by spaces in my editor :-)


I dislike the magic value `4` here, can we have a descriptive name instead please? Same for further down with the `>=` case.


改动两个地方请注意查收(test)


Fix typo in @string/answer1


Hmm, I see you applied some custom changes, which you probably should have submitted a `PR` for late last year.

> So if the popup browser is disposed at those moments (which seems to be caused be a race condition in CEF between the resource loading and the browser disposal) we don't really need to throw an exception.

This is not specific to your app. Would have saved me time if someone had submitted a `PR`.

Anyway, I've already fixed this in `master` a bit over a week ago. I expect you'll have some merge conflicts next time you update.


Fahrenheit - Celsius
     0        -17.8
    20         -6.7
    40          4.4
    60         15.6
    80         26.7
   100         37.8
   120         48.9
   140         60.0
   160         71.1
   180         82.2
   200         93.3
   220        104.4
   240        115.6
   260        126.7
   280        137.8
   300        148.9


было бы хорошо сдавать домашки пуллреквестами, а не коммитами в мастер --- их так проще комментировать и следить за исправлением замечаний


That's any editor worth a damn has word complete, and also why we are going
to have `use x as y` and maybe `use collections::List`?

On Fri, 26 Feb 2016 06:12 Liam <notifications@github.com> wrote:

> Who wants to constantly type collections though. Lists are a commonly
> used type.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ark-lang/ark/commit/2952a0935545829dad9402a08b20cfe1f34894e0#commitcomment-16344710>
> .
>


OUTPUT
  0	   -17
 20	    -6
 40	     4
 60	    15
 80	    26
100	    37
120	    48
140	    60
160	    71
180	    82
200	    93
220	   104
240	   115
260	   126
280	   137
300	   148


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


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


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


Fix @string/welcome1 typo in values-sv


This change looks like it cost the metadata provider references to the value editor types.


два пальца вверх за ссылки в коде на ответы


我应用内用 Gson，但又必须集成你们的 FastJson，可否不要写死。


For some reason it was on the page spinning indefinitely. I checked the other pages and saw that this image was only present in items controller. There is still a spinner in the searchbox tho.

Maybe the css got messed up and it showed on a different position now? Anyway once we upgrade to jquery-ui we'll probably use those animations anyway.


There is something odd about this function. In line 66 above you resign in a loop variable listFuture defined in line 48.
Is this necessary?


This fails, returning:
      1) AdvertisementsController GET #create redirects to the new post
     Failure/Error: expect(response).to redirect_to Advertisement.last
       Expected response to be a <redirect>, but was <200>


Ok seems like it shouldn't be there, let me get rid of that


Is this really wanted in 2.7.0 ?
ZF2 requires zend-servicemanager ^2.5
So this component (requiring 3.0.3) can't be installed.


Uploaded!


Looking through the errors on the CI build output, there are some issues relating to the interactions design assembly around the resolution of value editor types. Looks like the removal of sn info in the assembly reference broke this.


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


求助，安装好之后后台和前台访问都不显示中文，浏览器的语言为中文的，换了几个浏览器依旧不行。


@tristil It's from existing `rand(max : Int)`. I don't know, it's better to change or not.


Is this part of the deprecation? Careful with your commits, I think you may be adding too much stuff in here.


LGTM


i seen it in many sites that use jwplayer.
the one that i remember for now is openload.


My biggest gripe is the bright jobs background & white text: the contrast is not big enough to make it glance-friendly:

<img width="222" alt="screen shot 2016-02-26 at 07 11 46" src="https://cloud.githubusercontent.com/assets/3342/13345629/428f1bc4-dc58-11e5-94c9-3b9a6cd24477.png">

The previous default theme was slightly better, but suffered from the same problem: the muted jobs background & the white text. The white text is perfect, I wouldn't have anything else, the problem for me are the box backgrounds.


Methods in this class appears to be generic enough that they and some other should perhaps be part of separate product and not of CK. This way others might use it as well.


Think of it this way. libtool-bin contains the ```libtool``` command. This is needed for crosstool-ng to work properly.
Without tar, crosstool-ng will fail ```./configure ...```, because it is required. Same with bzip2 and gzip.

Without subversion, and with most of the optional package requirements, a Kconfig option is set in crosstool-ng, so that if an optional dependency (such as subversion) is not available during the build, then support is said to not be available. Regardless if you install subversion later or not.

Then a question has to happen with external packages and optional dependencies. If the package is built with all of the optional dependencies, and one of the optional dependencies is not available, does crosstool-ng fail the way we expect?

Another reason I try to stay away from packaging crosstool-ng with the distribution.


I'm not clear what you are doing here.
UTF-8 *is* the file format, it is not dependant on QT version (?)


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


the variable is here, i just wanted to provide a base


q'q'q


fixing #7


_config 是不希望用户设置的，所以不补充 getter setter，其他的修正了。


We are no longer support Simplified Chinese.


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


Should't methods in this class Futures be part of separate package so that they can be reused elsewhere.


Uploaded!


... into the infrastructure of computational science.


Try saying `datas` in your head for a while. It'll grow on you.


Алексей делайте pullRequest не надо мерджить самостоятельно - иначе у нас в CRM летит статистика)))


Hello Hoang,
Can you give me password of rar file?


All I can think of saying datas is "muh datas" lol

On Fri, Feb 26, 2016, 08:25 Liam <notifications@github.com> wrote:

> Try saying datas in your head for a while. It'll grow on you.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ark-lang/ark/commit/2952a0935545829dad9402a08b20cfe1f34894e0#commitcomment-16346031>
> .
>


Hi @geeeeeeeeek, 你觉得可以加 ESLint 进来么? 纯 ~~qiɑ́ng~~ 个 ~~pò~~ 人 ~~zhèng~~ 习惯请见谅......


Hi Mark,
The default behaviour in Qt5 for the QString char* constructor is to convert from utf8:
http://doc.qt.io/qt-5/qstring.html#QString-7
I should add a comment to that effect.
Although I wasn't sure what the requirement/plan was for maintaining QT4 support so #ifdefs in.


Fixed at last forced [commit](https://github.com/crystal-lang/crystal/commit/3f5a77f23ef404e287cbdbbe58e4e9f19d7c17f9#diff-ab21012862ff4bc662382874cb0c0485R93), thanks.


On Thu, Feb 25, 2016 at 04:45:27PM -0800, rswier wrote:
>    Perhaps you can get the Perl assembler to be just good enough to
>    assemble "as" (page 32 of 05-1-4.pdf). Modifying the "as" where needed
>    to meet the needs of the simple assembler or even hand assemble parts
>    of it. Run the result on the original "as" and compare the binaries to
>    verify the bootstrap.

Good idea to have two different tools that can validate each other.
First, though, I have to get the a7out simulator to the point where
it can run as :-)

Cheers, Warren


Hey, that makes it memorable.


This should be wptuser, right?


`_input` and `_gradOutput` need to be added in clearState


Could you give an concrete URL? I didn't find any ```eval()``` pattern in the first few links on http://www.openloadmovie.com/. And by the way, most links on its homepage looks copyright infriging.


It just feels like a misnomer, I dunno man. I could probably live with it
but it currently makes me feel a lil' iffy

On Fri, Feb 26, 2016, 08:29 Liam <notifications@github.com> wrote:

> Hey, that makes it memorable.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ark-lang/ark/commit/2952a0935545829dad9402a08b20cfe1f34894e0#commitcomment-16346085>
> .
>


Fixed at last forced [commit](https://github.com/AlexWayfer/crystal/commit/3f5a77f23ef404e287cbdbbe58e4e9f19d7c17f9#diff-ab21012862ff4bc662382874cb0c0485R93), thanks.


OK. I'll modify the pointer focus then to keep track of the focused view independently.


Don't move the method. It makes it harder to see what differences exist between the old version and new version.


IRC is fine, I'm often on freenode... Tell me if you wanna create a #bikini (or something else) room.


there is an issue for this site in https://github.com/rg3/youtube-dl/issues/8489 it contain example urls.


Please help review the code, thanks a lot~ @Zhiqiang Zhang


The commit message should say "Implement *implicit* coercions between decimals". When I first started looking at it, it wasn't clear how it was different from the previous commit that says "implement casts between decimals"


Also it shouldn't be more complicated, but I'd need to reinstall Idris and try again a few things.


I think that it looks better!!!


@acemaster Haha I was about to write this script. The dhclient doesn't always work man :/ We need to put it in a while loop XD


Maybe you should mention that there several other European alphabets supported, but not unit-tested? Cause you have test method for Greek and several letters like this ones äöüßéàç that, I assume, covers the France, German and several others.


Please help review the code, thanks a lot~ @zqzhang


Other option could be to check the places where surface calls to view and special case for subsurfaces, from muscle memory surface_attach should be only place (?), but worth checking.


Nice first attempt, but a better implementation would take any number of arguments


* partner_demo2 тоже к base нужно привязывать?
* сделай разные ean13


Application is namespace scoped, use NamespaceKeyFunc() instead.


one moment :)


OUTPUT

Fahrenheit - Celsius
     0        -17.8
    20         -6.7
    40          4.4
    60         15.6
    80         26.7
   100         37.8
   120         48.9
   140         60.0
   160         71.1
   180         82.2
   200         93.3
   220        104.4
   240        115.6
   260        126.7
   280        137.8
   300        148.9
Celsius - Fahrenheit
     0         32.0
     4         39.2
     8         46.4
    12         53.6
    16         60.8
    20         68.0
    24         75.2
    28         82.4
    32         89.6
    36         96.8
    40        104.0
    44        111.2
    48        118.4
    52        125.6
    56        132.8
    60        140.0
    64        147.2
    68        154.4
    72        161.6
    76        168.8
    80        176.0
    84        183.2
    88        190.4
    92        197.6
    96        204.8
   100        212.0


Did you remove the netceiver intentionally from the README?
Cheers


Check if your vim uses the `.clang-format` style file

    let g:clang_format#detect_style_file=1
    let g:clang_format#command="clang-format-3.6"

but then again there were already issues in rtd that only happened over at @MoKob


Reference: 1.2 Variables and Arithmetic Expressions - Fahrenheit-Celsius table
The C PROGRAMMING LANGUAGE, Second Edition, Brian W. Kernighan & Dennis M. Ritchie


Reference: 1.2 Variables and Arithmetic Expressions - Fahrenheit-Celsius table
Exercise 1-3. Modify the temperature conversion program to print a heading above the table.
The C PROGRAMMING LANGUAGE, Second Edition, Brian W. Kernighan & Dennis M. Ritchie


OK,


Thx


Option class has the synchronized method, now sync word could be removed


also from https://github.com/rg3/youtube-dl/issues/7606#issuecomment-162135535. i remember that i checked videowood.


Reference: 1.2 Variables and Arithmetic Expressions - Fahrenheit-Celsius table
Exercise 1-4. Write a program to print the corresponding Celsius to Fahrenheit table.
The C PROGRAMMING LANGUAGE, Second Edition, Brian W. Kernighan & Dennis M. Ritchie


yum install curl


Hi,

you can do conversion programmatically e.g. with Java: https://github.com/Tombio/Trafalert/blob/master/Server/src/main/java/com/studiowannabe/trafalert/control/CoordinateConverter.java or you can use some online tool like karttapaikka: http://kansalaisen.karttapaikka.fi/koordinaatit/koordinaatit.html?e=375082&n=6676328&scale=8000&width=600&height=600&tool=pienenna&lang=en

Btw. next time please ask questions at our group: https://groups.google.com/forum/#!forum/roaddigitrafficfi so that others with same problem can see the resolution.

- Digitraffic


unzip master.zip


Yes that sounds good. And if the user provides it we should skip adding it.


hello this is test


* 63040 is for AI demanding liege title
* I added another TODO in here for for powerful vassal takeover
* consider what emf_rr could/should do with the new traits (robust/feeble/shrewd/dull, and in last patch, erudite/stubborn)


Yep, for consistency I probably should use `.id` - just a habit. If associations are all configured correctly however, then rails is able to understand `assigned_user: user_admin` directly, without needing to bother with the `_id` or `.id` parts. I'll make the more general change of removing all `id` references and double check the associations instead.


why are you adding those new dependencies?


That was really annoying to fix. Silly copy-pasta bug >.>


This probably shouldn't be an error. We only bash `.Library` if we were able to create symlinks / junction points to libraries. Since junction points are an NTFS feature this could break packrat on FAT32.


So we switched to C++11, right?


why


It'd be cool to show using process.env here. Just as a best practice of not checking in your credentials.


Nice find! That bug goes right back to the initial commit. We've all been in there and missed it many times.


qwe


qq


--debug


:+1: Cool. I hope this gets included in the next release :)


Yes, that is what I was talking about doing for all the post-1.0 sections in the PR comment...


You might consider putting a `,` at the end of this line. Then when new codes are added this line won't have to change in the git diff.


hahahaha :+1:


Yep, because - currently - e.G. `imageElement is used in `getImageSrc()`, `unbindImageClickListeners()` and `bindImageClickListeners()`, so, sadly, it has to be available in highest scope.

`imageSrc` is wrong in my pull request, it's only used in  `loadImage()`, so it can be private in this function. Without `var` its available in global scope outside of your script.


This will not work here, because CODE_GPS_L1CA and so on are enum elements.


Also. there is an high chance that the following have to be applied:
https://github.com/freebsd/freebsd/commit/130dafc3086e3c5dbde01a9ff799fa24b3fa0569
https://github.com/freebsd/freebsd/commit/1c17300d1dcdf3d060b9ffb3d59ac06094bd936f
https://github.com/freebsd/freebsd/commit/ad65b26786a9160789d37051277ea5c34c98113b
All need to be checked and tested. (But I don't have a 10GbE system to test on).

There may be others as well. See:
https://github.com/FreeBSD/FreeBSD/commits/master/sys/dev/ixgbe

Seems FreeBSD developers do a lot of changes to Intel drivers to fix bugs.


I'm on #bikini :P good


What about adding a comment like `\\ start number of pulses at 2`, or something to that effect? Might keep someone from accidentally "fixing" it somewhere down the road.


What does it do? I mean, where would be that test file defined?
btw thanks this for sample app!


Great, okay. I guess my preferred approach is to refer to the `db/schema.rb` file when I need to, but I can understand where you're coming from (I have had the urge to document the schema like this in the past, before forming the "check-schema-file habit"). Glad to hear at least it's automated by the `annotate` gem; I hadn't noticed that. Will remove this commit.


Yep, great idea. Will update now.


possible issue: `shrewd` is opposites with `imbecile`/`slow`/`genius`, and `dull` with `imbecile`/`quick`/`genius`. There are some places in events where vanilla adds one of these traits without checking for these congenital opposites. What happens in that case? Does the trait fail to add, or does something different and bad happen?


Don't worry too much and aesthetics. We can work on the look and fee once we get it working fine. I notice you removed the `hashService` in the controller. Why?


Selenium standalone


if(...)   ===>  if (...)


if(...)  ===> if (...)


Is the double slash intentional?


@MBoustani review plz


Does this require that the externalLocation is in classpath?


fasdfasdfasd


@gregolsky Your way of calculating the method body length worked and was not too slow. But I think using the API directly from sourceFile object is a little more clear and does not require us to maintain a regular expression. However, it does count empty lines, which I think is OK.


Logging here would be nice:

       logger.info("Using external configuration file: {}", externalLocation);


Thanks for that. Seems current ```decode_packed_codes``` is not sufficient for openload.co.


This patch allows the spellings of notes to be preserved in a MIDI file.  The bottom two bits of note-on attack velocities are used to encode the spelling of notes.  For example if the key number is 60, then these three values for the bottom two bits of the attack velocity will indicate which of the three possible ways of spelling 60 is correct:
```
    1:   60 is D-double-flat
    2:   60 is C natural
    3:   60 is B-sharp
```

If the spelling code is 0, then the spelling is technically undefined and a spelling based on the musical key (from a earlier key-signature meta message) should be chosen.  In this implementation the code-2 spelling is selected if a code-0 is encountered.  The MIDI file must be encoded with MIDIPlus spelling codes in order for the MidiMessage::getSpelling() to return valid information (there is no standard way to indicate this).

This pitch-spelling encoding system is based on the MIDIPlus system described in Chapter 5 of [Beyond MIDI: A Handbook of Musical Codes](http://beyondmidi.ccarh.org/beyondmidi-600dpi.pdf).

Here is a test program which checks the spelling preservation for all pitch names with up to double flat/sharps (plus the special F-triple sharp case) within the middle-C octave:

```cpp
#include "MidiFile.h"
#include <iostream>

using namespace std;

void printSpellings(MidiFile& midifile);
void printDiatonicNote(int base12, int spell, int base7, int accidental);

int main(void) {
   MidiFile outfile;

   int track   = 0;
   int tick    = 0;
   int channel = 0;
   int index;

   index = outfile.addNoteOn(track, tick, channel, 58, 99);
   outfile[0][index].setSpelling(0, -2);   // Cbb
   index = outfile.addNoteOn(track, tick, channel, 59, 99);
   outfile[0][index].setSpelling(0, -1);   // Cb
   index = outfile.addNoteOn(track, tick, channel, 60, 99);
   outfile[0][index].setSpelling(0,  0);   // C
   index = outfile.addNoteOn(track, tick, channel, 61, 99);
   outfile[0][index].setSpelling(0, +1);   // C#
   index = outfile.addNoteOn(track, tick, channel, 62, 99);
   outfile[0][index].setSpelling(0, +2);   // C##

   index = outfile.addNoteOn(track, tick, channel, 60, 99);
   outfile[0][index].setSpelling(1, -2);   // Dbb
   index = outfile.addNoteOn(track, tick, channel, 61, 99);
   outfile[0][index].setSpelling(1, -1);   // Db
   index = outfile.addNoteOn(track, tick, channel, 62, 99);
   outfile[0][index].setSpelling(1,  0);   // D
   index = outfile.addNoteOn(track, tick, channel, 63, 99);
   outfile[0][index].setSpelling(1, +1);   // D#
   index = outfile.addNoteOn(track, tick, channel, 64, 99);
   outfile[0][index].setSpelling(1, +2);   // D##

   index = outfile.addNoteOn(track, tick, channel, 62, 99);
   outfile[0][index].setSpelling(2, -2);   // Ebb
   index = outfile.addNoteOn(track, tick, channel, 63, 99);
   outfile[0][index].setSpelling(2, -1);   // Eb
   index = outfile.addNoteOn(track, tick, channel, 64, 99);
   outfile[0][index].setSpelling(2,  0);   // E
   index = outfile.addNoteOn(track, tick, channel, 65, 99);
   outfile[0][index].setSpelling(2, +1);   // E#
   index = outfile.addNoteOn(track, tick, channel, 66, 99);
   outfile[0][index].setSpelling(2, +2);   // E##

   index = outfile.addNoteOn(track, tick, channel, 63, 99);
   outfile[0][index].setSpelling(3, -2);   // Fbb
   index = outfile.addNoteOn(track, tick, channel, 64, 99);
   outfile[0][index].setSpelling(3, -1);   // Fb
   index = outfile.addNoteOn(track, tick, channel, 65, 99);
   outfile[0][index].setSpelling(3,  0);   // F
   index = outfile.addNoteOn(track, tick, channel, 66, 99);
   outfile[0][index].setSpelling(3, +1);   // F#
   index = outfile.addNoteOn(track, tick, channel, 67, 99);
   outfile[0][index].setSpelling(3, +2);   // F##
   index = outfile.addNoteOn(track, tick, channel, 68, 99);
   outfile[0][index].setSpelling(3, +3);   // F###

   index = outfile.addNoteOn(track, tick, channel, 65, 99);
   outfile[0][index].setSpelling(4, -2);   // Gbb
   index = outfile.addNoteOn(track, tick, channel, 66, 99);
   outfile[0][index].setSpelling(4, -1);   // Gb
   index = outfile.addNoteOn(track, tick, channel, 67, 99);
   outfile[0][index].setSpelling(4,  0);   // G
   index = outfile.addNoteOn(track, tick, channel, 68, 99);
   outfile[0][index].setSpelling(4, +1);   // G#
   index = outfile.addNoteOn(track, tick, channel, 69, 99);
   outfile[0][index].setSpelling(4, +2);   // G##

   index = outfile.addNoteOn(track, tick, channel, 67, 99);
   outfile[0][index].setSpelling(5, -2);   // Abb
   index = outfile.addNoteOn(track, tick, channel, 68, 99);
   outfile[0][index].setSpelling(5, -1);   // Ab
   index = outfile.addNoteOn(track, tick, channel, 69, 99);
   outfile[0][index].setSpelling(5,  0);   // A
   index = outfile.addNoteOn(track, tick, channel, 70, 99);
   outfile[0][index].setSpelling(5, +1);   // A#
   index = outfile.addNoteOn(track, tick, channel, 71, 99);
   outfile[0][index].setSpelling(5, +2);   // A##

   index = outfile.addNoteOn(track, tick, channel, 69, 99);
   outfile[0][index].setSpelling(6, -2);   // Bbb
   index = outfile.addNoteOn(track, tick, channel, 70, 99);
   outfile[0][index].setSpelling(6, -1);   // Bb
   index = outfile.addNoteOn(track, tick, channel, 71, 99);
   outfile[0][index].setSpelling(6,  0);   // B
   index = outfile.addNoteOn(track, tick, channel, 72, 99);
   outfile[0][index].setSpelling(6, +1);   // B#
   index = outfile.addNoteOn(track, tick, channel, 73, 99);
   outfile[0][index].setSpelling(6, +2);   // B##

   printSpellings(outfile);

   return 0;
}

void printSpellings(MidiFile& midifile) {
   int base7, accidental;

   for (int i=0; i<midifile[0].size(); i++) {
      if (!midifile[0][i].isNoteOn()) {
         continue;
      }
      midifile[0][i].getSpelling(base7, accidental);
      printDiatonicNote(midifile[0][i].getKeyNumber(),
            midifile[0][i].getVelocity() & 0x03,
            base7, accidental);
   }
}

void printDiatonicNote(int base12, int spell, int base7, int accidental) {
   int octave = base7 / 7;
   int base7pc = base7 % 7;
   cout     << "[" << base12 << ", " << spell << "]"
        << " => (" << base7  << ", " << accidental << ")\t";
   switch (base7pc) {
      case 0: cout << "C"; break;
      case 1: cout << "D"; break;
      case 2: cout << "E"; break;
      case 3: cout << "F"; break;
      case 4: cout << "G"; break;
      case 5: cout << "A"; break;
      case 6: cout << "B"; break;
   }
   int i;
   if (accidental < 0) {
      for (i=accidental; i<0; i++) {
         cout << 'b';
      }
   } else if (accidental > 0) {
      for (i=0; i<accidental; i++) {
         cout << '#';
      }
   }
   cout << octave;
   cout << endl;
}
```

Here is the output text from the program, which lists all 36 pitches in the 5th octave (middle C-flat-flat to the next B-sharp-sharp):

```
[58, 1] => (35, -2)	Cbb5
[59, 1] => (35, -1)	Cb5
[60, 2] => (35, 0)	C5
[61, 2] => (35, 1)	C#5
[62, 3] => (35, 2)	C##5
[60, 1] => (36, -2)	Dbb5
[61, 1] => (36, -1)	Db5
[62, 2] => (36, 0)	D5
[63, 3] => (36, 1)	D#5
[64, 3] => (36, 2)	D##5
[62, 1] => (37, -2)	Ebb5
[63, 2] => (37, -1)	Eb5
[64, 2] => (37, 0)	E5
[65, 3] => (37, 1)	E#5
[66, 3] => (37, 2)	E##5
[63, 1] => (38, -2)	Fbb5
[64, 1] => (38, -1)	Fb5
[65, 2] => (38, 0)	F5
[66, 2] => (38, 1)	F#5
[67, 3] => (38, 2)	F##5
[68, 3] => (38, 3)	F###5
[65, 1] => (39, -2)	Gbb5
[66, 1] => (39, -1)	Gb5
[67, 2] => (39, 0)	G5
[68, 2] => (39, 1)	G#5
[69, 3] => (39, 2)	G##5
[67, 1] => (40, -2)	Abb5
[68, 1] => (40, -1)	Ab5
[69, 2] => (40, 0)	A5
[70, 3] => (40, 1)	A#5
[71, 3] => (40, 2)	A##5
[69, 1] => (41, -2)	Bbb5
[70, 2] => (41, -1)	Bb5
[71, 2] => (41, 0)	B5
[72, 3] => (41, 1)	B#5
[73, 3] => (41, 2)	B##5
```

The numbers in square brackets, [], are the MIDI key number and the 2-bit spelling code from the velocity parameter.  The numbers in parentheses are the diatonic pitch number (diatonic pitch class plus octave information) and the accidental (-2 = double-flat, -1 = flat, 0 = natural, +1 = sharp, +2 = double-sharp).  And the alphabetic pitch name representation of the numbers in parentheses is given at the end of each line (diatonic pitch class, accidental alteration, and then octave).


Fix issue #88


But what was happening before that change?
The return type is a pointer to wxChar, so no way an actual string was passed out.
My reasoning was, that I was just adding an explicit conversion where the same conversion was done implicitly before.


@makalaaneesh Reading the whole file is the standard procedure before commenting :wink:


Don't


Yes ... `~/.bashrc` isn't used by them


as this is in require-dev I don't think this is the case. No? And that should not prevent from being installed I think. But maybe I am wrong not 100% sure about this. And yes commit message and diff seems a bit strange :D


Also, please cache this directory (Check lines 2-8)


Reduced to:

```swift
class C{}
print(C())
```

```
* thread #1: tid = 0x21fcf2, 0x00000001007d2030, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=2, address=0x1007d2030)
  * frame #0: 0x00000001007d2030
    frame #1: 0x000000010005f939 libswiftCore.dylib`::_swift_release_dealloc(object=0x0000000100c01f70) + 41 at HeapObject.cpp:434
    frame #2: 0x000000010005f4b2 libswiftCore.dylib`_swift_release_(object=0x0000000100c01f70) + 66 at HeapObject.cpp:296
    frame #3: 0x000000010005f464 libswiftCore.dylib`swift_release(object=0x0000000100c01f70) + 36 at HeapObject.cpp:292
    frame #4: 0x0000000100225fcd libswiftCore.dylib`_adHocPrint_unlocked<A, B where ...> (value=0x0000000100c00510, mirror=Swift.Mirror @ 0x00007fff5fbff0c0, isDebugPrint=false) -> () + 5645 at OutputStream.swift:173
    frame #5: 0x0000000100227611 libswiftCore.dylib`_print_unlocked<A, B where ...> (value=0x00007fff5fbff670, target=) -> () + 2305 at OutputStream.swift:206
    frame #6: 0x000000010022b1d3 libswiftCore.dylib`_print<A where ...> (items=1 value, separator=" ", terminator="\n", output=) -> () + 627 at Print.swift:134
    frame #7: 0x000000010022adf3 libswiftCore.dylib`print(items=1 value, separator=" ", terminator="\n") -> () + 515 at Print.swift:39
    frame #8: 0x00000001000012e3 test`main + 163 at test.swift:2
```

Taking `inline` off of `asFullMetadata`, the killer is:

```
    frame #0: 0x000000010005ea37 libswiftCore.dylib`::_swift_release_dealloc(object=0x0000000100b06db0) + 23 at HeapObject.cpp:434
   431 	// Declared extern "C" LLVM_LIBRARY_VISIBILITY above.
   432 	void _swift_release_dealloc(HeapObject *object) {
   433 	  auto metadata = reinterpret_cast<const LiveHeapMetadata *>(object->metadata);
-> 434 	  asFullMetadata(metadata)->destroy(object);
   435 	}
   436
   437 	#if SWIFT_OBJC_INTEROP
```
In `asFullMetadata`, it seems that the metadata pointer is already aligned as a `FullMetadata`:

```
   902 	/// Given a canonical metadata pointer, produce the adjusted metadata pointer.
   903 	template <class T>
   904 	static FullMetadata<T> *asFullMetadata(T *metadata) {
-> 905 	  return (FullMetadata<T>*) (((typename T::HeaderType*) metadata) - 1);
   906 	}
   907 	template <class T>
   908 	static const FullMetadata<T> *asFullMetadata(const T *metadata) {
(lldb) memory read -f A -c 10 metadata
0x1007d6048: 0x0000000100061290 libswiftCore.dylib`(anonymous namespace)::destroyGenericBox(swift::HeapObject*) at HeapObject.cpp:179
0x1007d6050: 0x0000000000000000
0x1007d6058: 0x0000000000000041
0x1007d6060: 0x0000000100000010 test`_mh_execute_header + 16
0x1007d6068: 0x00000001007d1030
```

Kind of looks like a `FullMetadata<LiveGenericBoxHeapMetadata>`, maybe from a `BoxCacheEntry` (the value witnesses pointer is `nullptr`)?


Well, thank you Jürgen ! .qgs file size is going to be reduced with QGIS 2.14 and it is very good !


Add an if condition to check if the `dart-sdk` folder already exists (as we would be cacheing it)
PS : "Dart commands" seems more appropriate for the comment


We use RST in documentation. So, you need to use two backticks for code. (1 backtick is markdown)


Should we also add an else condition in case the event transferable is not an instance of the `WrapperTransferable` ? I think we need to inform the UI (user) that the drop is not possible, I guess this is an error on our side then if, otherwise the user won't recognize what happend when he dropped it but don't get feedback.


OK. Removed for H264 and HEVC encoder.

Best Regards,
Xiaoxia Zhong

From: lizhong1008 [mailto:notifications@github.com]
Sent: Friday, February 26, 2016 1:36 PM
To: ZhongXiaoxia/libyami
Cc: Zhong, Xiaoxia
Subject: Re: [libyami] h264enc: Modify IDR interval and key period parameters setting. (8782b63)

please remove VideoConfigAVCIntraPeriod since it is useless. intraPeriod can be set by VideoParamsCommon and idrInterval set by VideoParamsAVC.

—
Reply to this email directly or view it on GitHub<https://github.com/ZhongXiaoxia/libyami/commit/8782b63e4b1c243360b40cfe40ed6726b8a41ad3#commitcomment-16344894>.


loadedPath变量也应该清空：
public void resetDict(){
    	_dict = new DictSegment((char) 0);
    	freqs.clear();
    	loadedPath.clear();
    }


Yes, I agree. That's my first rule and I wasn't aware that I can get this info from `sourceFile`. Also the clearer the better :)
I was counting only non-empty lines, since I was suspecting that there will be people who try to cheat the rule, by removing all empty lines from offending functions, however I believe that if someone wants to cheat, then will find another way (like e.g. putting everything in one line). This rule will work perfectly with line length limiting rule.


Removed keyPeriod from interface. Added m_keyPeriod as member variable for H264 and HEVC encoder class.

Best Regards,
Xiaoxia Zhong

From: lizhong1008 [mailto:notifications@github.com]
Sent: Friday, February 26, 2016 1:42 PM
To: ZhongXiaoxia/libyami
Cc: Zhong, Xiaoxia
Subject: Re: [libyami] h264enc: Modify IDR interval and key period parameters setting. (8782b63)

keyPeriod equal to intraPeriod() * (m_videoParamAVC.idrInterval + 1), so why we need this parameter to be set? Or maybe we can keep keyPeriod and remove idrInterval.

—
Reply to this email directly or view it on GitHub<https://github.com/ZhongXiaoxia/libyami/commit/8782b63e4b1c243360b40cfe40ed6726b8a41ad3#commitcomment-16344957>.


what is happening if we have the period type is the month and duration greater than 1 or my product is a downloadable product?


Why is this a peer dependency? It won't work without `autoprefixer`, so shouldn't it be a dependency?


@acemaster My bad. My bad :-1:


OK. Same changes made both for H264 and HEVC encoder. Patch submitted.

Best Regards,
Xiaoxia Zhong

From: lizhong1008 [mailto:notifications@github.com]
Sent: Friday, February 26, 2016 2:35 PM
To: ZhongXiaoxia/libyami
Cc: Zhong, Xiaoxia
Subject: Re: [libyami] h264enc: Modify IDR interval and key period parameters setting. (8782b63)

And maybe this interface change will involve hevc encoder, could you help to change it also if necessary.

—
Reply to this email directly or view it on GitHub<https://github.com/ZhongXiaoxia/libyami/commit/8782b63e4b1c243360b40cfe40ed6726b8a41ad3#commitcomment-16345483>.


Not a bad idea. Will do this and finish the targets when I'm back at a PC.


remove parameter


xx


Nothing is impossible in python. I spend some time in argparse, wrote special handler to ignore "-s" clash only to discover it's not actually clashing. In v2 I'll enable booth and keep one unittest using it. We'll see how older argparse handles that, maybe the handler comes handy...


Shouldn't this be /run/keys in stead of /var/run?


also: really dunno what's up with the `vassal_request_events.txt` part with `new_character`.


Is that an explanation of what we must do in order to use the ADC ? In that case could we add it to the code so that it is done automatically ?


The diff is aligned badly but this is undoing the way I compatched these files and redoing it a better way


The more of these I see, the more I realise that Go gets this right and disallows implicit conversions.


czs


asdf


Right, this doesn't prevent installation, at least prevent to run the test suite.


I think for all external js libraries we should use `bower` if is it possible.
https://github.com/thesabbir/jquery-nestable/blob/master/bower.json


Ah ok! Not sure that really speeds anything up then .. did you profile it and find it made a difference?


I got this when trying to run the program (any ideas what happened?):

Executing C:\Users\Noah\Desktop\RocketDog-AddCutScene\dist\run279851548\RocketDog.jar using platform C:\Program Files\Java\jdk1.8.0_45\jre/bin/java
Exception in Application init method
java.lang.reflect.InvocationTargetException
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:497)
	at com.sun.javafx.application.LauncherImpl.launchApplicationWithArgs(LauncherImpl.java:389)
	at com.sun.javafx.application.LauncherImpl.launchApplication(LauncherImpl.java:328)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:497)
	at sun.launcher.LauncherHelper$FXHelper.main(LauncherHelper.java:767)
Caused by: java.lang.RuntimeException: Exception in Application init method
	at com.sun.javafx.application.LauncherImpl.launchApplication1(LauncherImpl.java:912)
	at com.sun.javafx.application.LauncherImpl.lambda$launchApplication$152(LauncherImpl.java:182)
	at com.sun.javafx.application.LauncherImpl$$Lambda$50/1343441044.run(Unknown Source)
	at java.lang.Thread.run(Thread.java:745)
Caused by: MediaException: MEDIA_UNAVAILABLE : c:\Users\Dubs\Documents\NetBeansProjects\RocketDog\src\splashReduced.mp4 (The system cannot find the path specified)
	at javafx.scene.media.Media.<init>(Media.java:405)
	at edu.uco.sdd.rocketdog.model.CutSceneIntro.<init>(CutSceneIntro.java:34)
	at edu.uco.sdd.rocketdog.model.SceneFactory.getLevel(SceneFactory.java:32)
	at edu.uco.sdd.rocketdog.controller.RocketDogGame.init(RocketDogGame.java:55)
	at com.sun.javafx.application.LauncherImpl.launchApplication1(LauncherImpl.java:841)
	... 3 more
Exception running application edu.uco.sdd.rocketdog.controller.RocketDogGame
Java Result: 1


SimpleUI is a command-line based UI for this application that uses Unix-like commands to navigate through the set of all created courses and the lectures and students therefor. There is currently no linkage to any graphing scripts, but those can be added later. Still in progress. Right now only allows addition and removal of courses,


This looks like it's undoing my previous commit:

https://github.com/Twinside/svg-tree/commit/5dc890f270a9a31d6182527931701ba3ee8f85dc

I'm not sure what was the bug. Why aren't all definitions be given an id? In any case, I absolutely need an id for markers.


It's not very wise to add custom attributes to `aiohttp.web.Request` -- the class is designed to be non-customizable, your approach may lead to compatibility problems in future.

I suggest `request['conn'] = await Connection.create(...)`


Does this commented code serve a purpose? Same below.


Make sure to load your JS files just before the closing </body> tag. This way your DOM will have rendered before you try to bind any 'click' (or other) events in your JS. As it is now, you would have had to wrap the $(document).ready() function around any event binding to make sure the DOM was fully loaded and the events were bound properly.


What are you doing here?
Pls check PHP doc, and what `$charge->getPublicData()` return.


@ketanbhatt We can remove print statement here I guess. :tongue:


Yeah, I've learned that more recently, and forgot to update this.

Thanks!


tyygh


Ok, thanks.


use `$this->initPaymentwallConfig()`


欢迎提意见


The manual claims that `overridePackages` is expensive, and IIRC `overrideDerivation` is said to be better avoided in defaults, but I've heard nothing of the like about `.override` and we use it a lot in all-packages.nix.


I think a better way to handle events like clicks in this case would have been to bind an event handler to click events instead of hard-coding in an 'onclick' handler into the HTML. For instance, you could have done something like:

toDoList.on('click', 'button', function() {
	var item = $(this).closest('li')[0];
	completeList.prepend(item);
});

completeList.on('click', 'button', function() {
	var item = $(this).closest('li')[0];
	toDoList.prepend(item);
});

It's important to note that in both cases, we're selecting the parent elements (toDoList and completeList, respectively), and then passing in a second parameter after the 'click' event listener. The second parameter is 'button', because it is the button click event we want to listen to. Passing in this second parameter will bind the click event listener to dynamically-generated elements, which is the case when we're creating new list and button elements for each new item added to the list. Using the .closest('li') method on $(this) would have been fine. However, you'd also want to place the button element inside of the 'li' element so that when you move it to the completed list, the entire list/button item is moved together. I know this is probably complicated, so let me know if you want to go over it together.


You're right, thanks for the heads up!


@maethu ok, good to know. We need to document that as well then.


波哥这个是不是得用select+ticker，for+ticker跟sleep是一个效果吧


On the weekend, I'll again have access to my device, which means I'll be able to test the new branch. If everything works fine (or at least as fine as before), I'll merge it with master. JFYI.


关于框架：是我在 2015 年 5月写的
          用来熟悉 php 框架及加载流程
框架特点：代码的高可移植性 因为我有许多项目都用这个做测试的
          比如magento zencart wordpress 的
关于作者：ajian-tuzi  一个从很小就开始接触编程的人
作者博客：http://www.weicot.com
交流方式：QQ 1050653098
		 weicots@gmail.com  or ajiang-tuzi@outlook.com

WeicotFramework 的目录结构

├─app 应用目录
│  ├─code  代码
│  │  ├─Acma 记账应用
│  │  │  ├─Block      视图块  关于视图的和块缓存
│  │  │  ├─Controller 控制器  路由及分发 调用 不可重复利用代码 集合
│  │  │  ├─DB         数据库  所有数据库操作集合
│  │  │  └─Model      模型集合   业物逻辑
│  │  ├─Base 基础
│  │  │  ├─Block
│  │  │  ├─Controller
│  │  │  ├─DB
│  │  │  └─Model
│  │  ├─Bill 订单管理应用
│  │  │  ├─Block
│  │  │  ├─Controller
│  │  │  ├─DB
│  │  │  └─Model
│  │  ├─Csv csv 处理应用
│  │  │  ├─Block
│  │  │  ├─Controller
│  │  │  ├─DB
│  │  │  └─Model
│  │  ├─Doc doc 处理应用
│  │  │  ├─Block
│  │  │  ├─Controller
│  │  │  ├─DB
│  │  │  └─Model
│  │  ├─Http 。。。
│  │  │  ├─Block
│  │  │  ├─Controller
│  │  │  ├─DB
│  │  │  └─Model
│  │  ├─Mail 。。。
│  │  │  ├─Block
│  │  │  ├─Controller
│  │  │  ├─DB
│  │  │  └─Model
│  │  └─Test 。。。。
│  │      ├─block
│  │      ├─controller
│  │      ├─db
│  │      └─model
│  └─etc 配置文件
├─lib 依赖库
│  └─Core  核心加载文件
├─media  媒体
├─NOTE 笔记
├─skin  样式及图片
│  ├─css
│  ├─img
│  └─js
├─test
│  └─note
├─tmp
└─tools  工具箱
    └─lib
        ├─canadapostapi
        │  ├─docs
        │  │  └─tracking
        │  ├─SOAP
        │  │  ├─tracking
        │  │  │  ├─GetDeliveryConfirmationCertificate
        │  │  │  ├─GetSignatureImage
        │  │  │  ├─GetTrackingDetails
        │  │  │  └─GetTrackingSummary
        │  │  └─wsdl
        │  └─third-party
        │      └─cert
        └─postmaster-php-master
            ├─lib
            │  └─Postmaster
            └─tests
                └─Postmaster


please use ```cst_errmsg``` instead of ```fprintf``` for portability (even if platforms with portaudio generally won't have a problem with ```fprintf```) and style.


Klopt dit?


Do you need the join to HazardType?


Do you maybe want to log the exceptions?


@legoboy0215 http://gidf.com?q=disassemble+minecraft


When you only re-raise the exception, you would not need the try-catch.


+1. I also like how Pony makes order of precedence explicit. I don't like that many languages make it all to easy to slip up.


Концептуально все вроде нормально. Тогда жду, пока поправишь форматирование и перепушишь с форсом, удалив лишние коммиты.


In this case you could also query with `DBSession.query(...).first()` which returns `None` instead of raising an exception:
http://docs.sqlalchemy.org/en/latest/orm/query.html#sqlalchemy.orm.query.Query.first


님 연락좀여


Hi, passed pawn evaluation is known to be time-control dependant, unfortunately. Just like king safety.
But let's wait and see ...


귀신이 와서 커밋하는건가....


合并


What do you think about this:

```ocaml
type tcpp_const =
   | CppInt of int32
   | CppFloat of string
   | CppBool of bool

and tcpp_expr_expr =
   | CppConst of tcpp_const
   | CppSwitch of tcppexpr * (tcpp_const list * tcppexpr) list * tcppexpr option
```

This would automatically avoid String-switches and whatever else might come up from weird Dynamic code.


sorry don't get it what test suite, because https://travis-ci.org/zendframework/zend-modulemanager runs just fine with both 3.0 and 2.7 versions. Also I think all the required packages already supports 3.0 it wouldn't be any difference if it was `^2.7.5 || ^3.0.3` or just `^3.0.3`


Not sure.. actually its light level 0


I dnt understand this


Attach files by dragging & dropping,  Выбрать файлы selecting them, or pasting from the clipboard.


I think so yes since I need the mnemonic of the related hazard type. I can check this.


@athanatos I guess we don't need aio_write_queue_ops any more..


Direct commit since this file is frozen


## Add Render Flexibility
The purpose of these changes are to make it easier to take advantage of wkhtmltopdf's ability to render multiple pages as a single PDF document.
- I have changed `convert_to_pdf` to accept a list instead of a string.
- Factored out some repeating logic in `render_pdf_from_template`.


run


这样 IE8 下会挂的吧。


agree, I did code review here https://github.com/shabanovd/exist/commit/2a496a68641b9d176c5c2ff0e1555ea90f6e31c9


Oh, ok, didn't know that. Maybe open a PR?


C'est quoi ces indentations?!? ;P


Nice tests :+1:

I speculate the explanation for your TODO is the "file detection" code. fish does some processing on history commands to determine which arguments were files, for autosuggestion hinting. It does this in the background so that fish doesn't become unresponsive. History is only saved once this background processing is complete. See `history_t::add_pending_with_file_detection` for where this stuff occurs.

One consequence is that if you set HISTFILE, you may pick up "old" commands that were run before the HISTFILE change - there's a race.


db.sqlite3 added new column (profession varchar) to provide more informative example of filterable table (multicolumn filter)


Technically, this should be `wibox.widget.textbox()`, not `wibox.widget.textbox`... (Feel free to ignore this)


which means if and old item failed validation for any reason and did not get added to the new_items array it gets deleted if user confirmed deletion. This is why unset was used for both arrays before 'continue'


I'd prefer removing the `-> Void` in closure to reduce redundancy of code. And the brackets are also useless as there is only one argument.


yeah actually this was a bad idea.  mobile sucks now.


this then is not needed here because there is no results table at this point


Done. I now check for the version number (>1899, that is V19 development version) with AC_TRY_RUN instead.


I don't see problems if debug builds have every reasonable check, but regular builds skip the less important checks.


there i wasn't sure yet. for first iteration i thought it would be okay if a span is only created if there is an existing trace.
Maybe this should be controllable with flag in the annotation?
e.g. @CreateSleuthSpan(createTraceIfNotExists = true)


Done


@markusrubey @aaron what do you prefer? Currently I return {deleted: true} in the api.


After some poking around I found other places where surface->view is used. Would you mind adding additional field parent_view for each surface which will contain the main view for each surface?


@alanfx I don't think so, it's copy&paste from a class created by @tristantarrant Using ctrl-c, ctrl-v IMHO doesn't deserve any credit :smile:


@nobu Exclusive version guards are easier to understand so I prefer them as well (to decide in which one "2.3.1" would fall).


could it be that this breaks the prod build?

`TypeError: Path must be a string. Received [ { src: 'src/assets/main.css', inject: true } ]
    at assertPath (path.js:8:11)
    at Object.win32.join (path.js:221:5)
    at Gulp.<anonymous> (D:\idea_workspaces\...\tools\tasks\build.assets.prod.ts:22:15)
`


You mean if the item cannot be created on L1230? I don't know why that would ever occur. or what do you mean?


- First point fixed.
- According to http://portaudio.com/docs/v19-doxydocs/portaudio_8h.html#a4949e4a8ef9f9dbe8cbee414ce69841d all PortAudio errors are negative. They don't say anything about positive numbers though. If you still think it's better to check for paNoError, please say it and I will happily change it. No problem.


I'd strongly object against using `map[string]interface{}` as you then need to to type conversions all the time and need to handle the types yourself. You're loosing the advantage of a strongly typed language like Go.

Having input-structs for functions requiring a lot of parameters which are then passed to the API is a standard I see all over Go libraries (for example libraries by Amazon, Google and other widely used libraries) so I tend to follow their example when implementing this. You know what to pass in and what type to pass in when you're building your application around the library instead of passing a generic map not telling you which parameters are there, which are required and what type they are…


Ist es nicht unnötig erst mit `if((ui_bufferPF & 0x10) >0)` zu prüfen, ob das Bit gesetzt ist. Es wird ja mit
`ui_buttons |= ((ui_bufferPF & 0x10) << 3);` sowieso der Wert von `ui_bufferPF` an der Stelle reingeschrieben. Ob der jetzt Eins oder Null ist, ist dann doch egal. Oder sehe ich das falsch?


i debugged a little for a related issue and this looks good :+1:


If you think it's ok we'll go for the current solution, it's rather unlikely they would start with positive errors.


pourquoi se donner un alias ? c'est pas assez explicite CMAKE_CURRENT_SOURCE_DIR ?


Cool, thanks, do it in next PR


good catch


:) yeah I like it too, more than I had expected beforehand...


perhaps you could also add following in the config.vm.define section -  to make names in VirtualBox easier to read

`c6401.vm.provider :virtualbox do |vb| vb.name="c6401" end`

This you could do for each node - OR you could use a node.json file to make configuration easier


Actually `bindImageClickListeners()` and `unbindImageClickListeners()` have local `imageElement` taken from function arguments. But in `getImageSrc()` I've got a bug: `image` should be used instead of `imageElement`. Could fix this and make both variables local-scoped? Amend new changes.


hm, never mind, seems this is a problem on my side, I copied the seed project a few weeks ago and am copying the latest commits manually since then. Must have missed something.