> You're using exceptions for flow control and integers for error-handling, that's not exactly the best practice!

Wait, I misunderstood. Sorry Jirka, you're actually agreeing with me.

I take everything above back :P


Awesome!

Have you considered using the wide variants of RegisterClassEx and CreateWindow (as well as ancillary functions) to support non-English window titles?


Zdobycie punktu, zdobycie checkpointu, utrata życia, zabicie przeciwnika, ukończenie planszy

Takie mi przychodzą na szybko do głowy, ale nie wiem czy znajdziesz na wszystkie odpowiednie soundy


xD


I can perform `ball.type` on this, which is good.

I'd like to be able to instantiate multiple objects.

Play aroun d with something like this, and update this PR, please
```js
var Ball = function() {};
ball1 = new Ball();
ball2 = new Ball();
```


I think I remember what this is for, and we might need to bring it back as `_PseudoRandomAccessCollection`, because LLDB’s interface to this stuff has standardized on indexing with `Int`.


but if we *do* bring it back, it should be *way* smarter than this!  This makes random access linear time even when the underlying collection has random access!


コメントをするとしても、状況によって隠さなければいけないこともあるのですね。
実際のお仕事を交えて教えていただきありがとうございます！


Отключил


Oh haha, i was just sending Travis this empty commit in order to trigger a new build and kick the tires on linting, after I got some weird 'could not reach bus' error there which made the initial build error out


Given how closely `Yc` and `Yu` seem to be related, I think it would be fine to rather use

    if 'Yc' in options or 'Yu' in options:

Not only because it's a smaller patch, but mostly because any chance we do to handling `Yu` is *probably* also going to be relevant for `Yc`.


cool!


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


For .html extension it is a pain not to have them for now
Weren't you gonna do color scheme
Organization needs work
Scroll down ;)
Oops, lol

On Sat, Jan 23, 2016 at 4:47 PM, Aloup Shmederheisen <
notifications@github.com> wrote:

> No actually.
> 1.) Get rid of the .html extension of the links.
> 2.) Color scheme is just no.
> 3.) Organisation of menu is ech.
> 4.) There was nothing wrong with the menu button.
> 5.) I was going to commit...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/FCCPSTSA/MEH-Web-Design-2015/commit/104ad43f3a8f3c1a0dd6619e3607e1694d63d279#commitcomment-15632693>
> .
>

--
From, James Trombo

The files and/or conversations contained within this email are solely
intended for the user it was addressed to. If you have received this email
in mistake please contact the sender immediately, and delete this email
from your account and system. If you are not the named addressee you should
not distribute or copy this e-mail. If it was sent to you in error and you
read it please use a "Men in Black" style mind wipe to
insure confidentiality of this email.


Отключил показ ошибок


I think this definition doesn't pull its own weight: `PYTHON_BINARY` is just one character less than `sys.executable` and means exactly the same (at least to Python programmers, I guess :-)).


ok


I re loaded program and now working OK????

On Sun, Jan 24, 2016 at 10:58 AM, carbuncle grim <mclarekin@gmail.com>
wrote:

> stilll got problem with enable hybrid
>
> donut looks good, then I enable hybrids and it looks bad??
>
> On Sun, Jan 24, 2016 at 10:06 AM, Krzysztof Marczak <
> notifications@github.com> wrote:
>
>> UI just added to repository :)
>>
>> 2016-01-23 22:03 GMT+01:00 mclarekin <notifications@github.com>:
>>
>> > UI ???, I want to try it :)
>> >
>> > —
>> > Reply to this email directly or view it on GitHub
>> > <
>> https://github.com/buddhi1980/mandelbulber2/commit/adace058544ac5028949713d3713d60b41dfb48b#commitcomment-15632481
>> >
>> > .
>> >
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/buddhi1980/mandelbulber2/commit/adace058544ac5028949713d3713d60b41dfb48b#commitcomment-15632490>
>> .
>>
>
>


This feels like something that could be extracted out into "getting started" or something along those lines.


You actually don't need these calls here, since you have them below on Line 44.


Please change this to Julius


Julius was alive


Return returns a value


:+1: +1  dla spisu dźwięku Bartka - wystarczy w zupełności  :)


localhost not localhsot?


Looks fairly good to me.


Thanks for the PR!

I was playing with rise earlier -- I think this matches more with the effect. What do you think?

```
/*
 *
 * Rise
 *
 */
@import 'shared';

%rise,
.rise {
  @extend %filter-base;
  filter: brightness(1.05) sepia(.2) contrast(.9) saturate(.9);

  &::after{
    background: radial-gradient(circle, rgba(232, 197, 152, .8), transparent 90%);
    mix-blend-mode: overlay;
    opacity: .6;
  }

  &::before{
    radial-gradient: (circle, rgba(236, 205, 169, .15) 55%, rgba(50, 30, 7, .4));
    mix-blend-mode: multiply;
  }
}
```


Perhaps this should discuss how a user might go about verifying that the presented certificate is the the one they expected when it's not trusted by the browser? Examples could include contacting the server administrator, attempting to connect from multiple locations, or even something like http://convergence.io/


Awesome fix! Working for me with Django 1.9.1. Edited my views.py as long as this is getting merged back into the master branch


This can probably be moved to the getting started page


EOF NL!


nit pick: I prefer "skip" to "do not", so `skipUpdateTransform`.


antipattern:
hey foo

justification:
heyfoo

references:
heyfoo

created on antipattern.io


This is awesome! Maybe there's a chance this service could be used for automatically evaluating SSL clients and regression testing?


antipattern:
foo

justification:
foo

references:
foo

created on antipattern.io


you can remove the `&jsonp={{callback}}` -- We'll call this with `$.getJSON()` which means we don't need the JSON response to be wrapped in a function call


antipattern:
foorific

justification:
foo

references:
foo

created on antipattern.io


the $remainder is a string in `$_` by default so you can just do `my $remainder = $_`;


bin


OCD much


Yes!


OK it looks good


antipattern:
some foo

justification:
some foo

references:
some foo

created on antipattern.io


This is the same definition as in the other tests, which makes it possible to pull it out later.

I also think that `sys.executable` should be an implementation detail. The test does not need to know how the path to the python binary is determined. There are a lot of alternatives, e.g. getting it from the PATH. We should be possible to adjust the implementation later.


antipattern:
soem other foo

justification:
some other foo

references:
some other foo

created on antipattern.io


antipattern:
whatthefoo

justification:
whatthefoo

references:
as;ldfkdfjks

created on antipattern.io


can you squash this into the other commit please. I'd rather not have whitespace commits if they can be avoided.


Finally! :-)


Add force move app storage option.
Allows apps that don't declare an install location
in their manifest to be moved.

Change-Id: I31bdee79afb1d7bb9319b92fda6a8ea331f0a662

Patchset: 1
http://review.cyanogenmod.org/#/c/126215/


antipattern:
comment

justification:
comment

references:
comment

created on antipattern.io


@gr2m so basically I think you mean we need to add : 'admin: false' inside of accounts.update(), now I need to figure out how to add something to your pull request? ?


Applied.


That doesn't work but `selected: o.value == value ? "true" : null` does.


thanks :)


http://www.cplusplus.com/reference/vector/vector/begin/

"If the container is empty, the returned iterator value shall not be dereferenced."


You can use /* and */ to comment out large things.


Shasha, instead of uncommenting it, enable it with a preprocessor macro and create a different CCTLIb_XXXX.a file that only your client uses. See how I do it when "USE_TREE_BASED_FOR_DATA_CENTRIC" flag is enabled.
Blindly enabling these two fields bloats the size and makes CCTLib not work on small memory machines.


As as the previous comment. Use these only under a MACRO that a seperate version of CCTLib uses.


Remeber to do pull request man! :smile:

checkout from master and create a branch.


nit: I would normally use square brackets here (`%w[...]`) because it is producing an array.


This is the bit changed. It is a completely new pathfinding system that is crazy simple compared to the old one, from 50 lines down to about 15.


Ok, I'll buy the 'Doing the same thing the same way simplifies identifying common code to factor out.' argument.

I guess if one wanted to be dogmatic, one could have insisted on applying that refactoring before even adding any new tests. However, I'm not dogmatic. So all good! :+1:


Instead of here, pass it in as a compilation flag. And in the source code just check to make sure it is enables by having a MACRO like this

#ifndef  USE_TREE_BASED_FOR_DATA_CENTRIC
     #error "RedSpy XXX Client can work only with  USE_TREE_BASED_FOR_DATA_CENTRIC"
#endif


antipattern:
foo

justification:
foo

references:
foo

created on antipattern.io


Does this allow `import_dev_dependencies` to be used with local configurations?


Check the date of the commit Joe. 😛


i actually realized this when i was putting the PR up, oh well


It might be worth adding a sentence noting where something like this would be useful (e.g. in tests).


Should be "fix crash on..."


Proponuję jeszcze dodać muzykę do menu jeśli uda ci się na szybko jakąś znaleźć


That's just a warning that you get garbage.
If you reserve size in a vector, get the .begin() iterator, push some stuff in and then dereference the iterator, the specs dictate it should still be valid, since the vector didn't relocate.
Hence begin() must (!) give you the garbage in the right address which is exactly what we need.


Instead of replicating all this code simply include "shadow_memory.cpp" look at how I use it in https://github.com/chabbimilind/cctlib/blob/master/tests/omp_datarace_client.cpp.


Well this would assume that one knows after two examples how mid-term result will look like. I am a fan of having 3-5 examples before I see how things can be cleaned up. But sure, that is a matter of taste and experience.

Happy for constructive discussions here :)


I'm loving this.


Related to #1


btw, I prefer the &cont[0] style which also must work without checking for cont.empty().


Last time, I swear! Maybe we should lock master :smirk:


I think the `uuid` dependency can be moved to `devDependencies` since it is now only used in `spec.git-api.patch.js` no?


ha!

rock!


foo foo foo


Say the user goes `0w`. Then `--repcount==UINT_MAX`. Rip.


still OCD much


change xxxxx to 9999999 to allow travis to compile, also, move set go_guid to top of file.


haha. You have write access because you are in the collab list so that's why.


Rip me


very good


Meh


En este documento no se trabaja en la estética y diseño de la página. Se procura mostrar diferentes resultados en función de lo aplicado en css.


Why would we test it against this outdated version?


TeamCity pull requests :: netty [Build 1097](https://garage.netty.io/teamcity/viewLog.html?buildId=5012&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 6394, ignored: 23; resource leak detected Build time: 01:21:01


This just modifies .git/hooks/pre-commit for automatic invocation. It's not required for running the check explicitly.


The pull request is in!  I'm not 100% confident that I've hit all of the
LINQ query edge cases.  But those should show up in further testing as you
do change tracking.

Brant

On Fri, Jan 22, 2016 at 10:41 PM, Jeffry Morris <notifications@github.com>
wrote:

> @brantburnett <https://github.com/brantburnett> -
>
> Do you think this is ready for merging? I can use it for changing
> tracking, since there are quite a few holes and this is a big one. If so,
> send a PR and I'll review and merge it this weekend and then rebase my WIP
> change tracking on top of it.
>
> ATM, i am kind of blocked because change tracking requires a resolvable
> primary key, so some UT's fail.
>
> Thanks!
>
> -Jeff
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/brantburnett/Linq2Couchbase/commit/53ffd03d55f9f6ed9a86b013d01bcd08861e4fd5#commitcomment-15627798>
> .
>

--
Thanks,
Brant Burnett
CenterEdge Software
(336) 598-5934
http://centeredgesoftware.com


Wow, did I make a mess of this diff. :flushed:


@valib Why did you remove ```@SuppressWarnings("restriction")```? It's bad that non API code is used so that it relies on an Oracle JRE, but fixing that is a big rewrite (and HTTP V1 use that as well). As long as we have it here, I prefer not to show the warnings because they mask other issues so you don't see them either.

The other change is good imo, I was thinking wrong when I placed it :+1:


Is there a smarter way to define environment variables? Can we separate this?


I like that this will only allow the specified colors. Defaulting to "red" was an interesting choice.

Our [Ruby example](https://github.com/ogryzek/oop_review/blob/master/ghost.rb#L4) shows an approach that _does not allow_ parameters to be passed into the constructor, and automatically assigns a random color among the four available.

Please update your PR to include randomization and auto-color assignment.


did you mean `Xor` instead of `xor`?


Böyle palindrome mu taranır yahu


use dev.txt from other branch here (`blueberrypy[dev]` is a dependency for running tests)


i.e. `if r.func in (Eq, Ne)`?


LGTM. Can you just fix the grammar? Should say "We need *to* raise..."


Not even sure how this happened, but since were using arrow functions we don't need `.bind(this)`
Can you update it to

    this._intervalId = window.setInterval(() => {
      if (!this.state.hovering) {
        this.slideToIndex(this.state.currentIndex + 1);
      }
    }, this.props.slideInterval);

And the last thing, can you rebase on master, I recently merged something. Thanks!


La Hoja de estilo en cascada o CSS se puede codificar con el codigo html o linkear una hoja externa con el código css, que de hecho, es la forma más adecuada.


can you please put android/screenshots/CriticalMaps-Maps-Berlin.jpg back in?
we need it in https://github.com/criticalmaps/criticalmaps-android/blob/master/README.md


Drop this


`dev.txt`


@Nadahar the Eclipse is marking that as nedundant.


how so


Adding a comment to see if this reflect in the console.


Der commit hat außerdem neuen doppelten code eingefügt  :unamused:


.idea* ne ide pod version control. to je user specific


Since this routine is self-contained, and only called from one place where the bank doesn't matter, I'll eventually be moving this into its own bank to free up room for more trainers, because space in that bank is getting rather low.

Also on the to-do list involving trainers is to make up a better system for custom moves than the one I'm using now, which is just MacGyver'd to read Yellow Version's format.


just for reference the `twitter.*` `share.*` `facebook.*` images in the top level are for meta stuff. maybe I'll put them in a folder haha


sviđa mi se što pišeš komentare na class-ama i metodama


što throws Exception? o ovome ćemo da pričamo kad se vidimo


new String(path, "UTF-8")


I wonder if we'd like to add the `Framework :: Django :: X.X` also?


dbContacts.isEmpty()


Look interesting. Are there any obvious downsides to having such parametric types?


preduge metode ne čitam. ove bi se moglo vjerovatno da razvrsta u više metoda.


@GBH is this done/ready for PR?


Due to multiple complaints about Brock being disproportionately difficult at that stage in the game, and after discussing potential solutions, it was decided to remove the non-fossil Pokémon from his team. While I was at it, I re-organized them so the highest level is last in the list.


ne mora biti u constructoru. obično ide odmah gdje se deklariše varijabla


This change uncovers a bug in the email validation library, which is fixed with this commit:
https://github.com/egulias/EmailValidator/commit/2ec7babda64bcac584de3d2d7a505ae4d0f34e9c

The net effect is that valid email addresses are considered invalid. Upgading composer.json to include "egulias/email-validator": "1.2.4" fixes it.


@ssiloti does this make sense? do you think this would be sufficient? (it's documented in the header file)


Thanks @marcustisater


I'm not sure of the logic behind this. As a counter example, consider Lt(f, g) where

```python
>>> x = symbols('x', positive=True)
>>> f = x
>>> g = -x - 1
>>> not Xor(*[i.is_positive for i in (f-g).as_numer_denom()])  # Lt(f, g) is False
True
```

If you try this in your branch it will pass but it does so because this case is already handled at instantiation, but if it weren't the logic you have here would give the wrong answer. So what you have done must not always hold (or I have gotten something wrong).


ako ti je jasno ovo sve što koristiš ovdje to je jako dobro. pričati ćemo o svemu oveme ;)


što contact mora da bude Serializable?


I was a little confused when I was messing with the directory and may have deleted them since they were named identically.


Die views reichen mir eigentlich schon. Ich muss jetzt nur noch herausfinden, wie ich passende Daten erzeuge, da das Format nicht so ganz trivial ist.


makeCacheMatrix <- function(x = matrix()) {
#This functionis to create a special "matrix" and return list containing
#1.set the value of the matrix
#2.get the value of the matrix
#3.set the inverse matrix of the matrix
#4.get the inverse matrix of the matrix

  m <- NULL

  set <- function(y) {
    x <<- y
    m <<- NULL
  }

  get <- function() x

  setInverse <- function(inverse) m <<- inverse

  getInverse <- function() m
  list(set = set, get = get, setInverse = setInverse, getInverse = getInverse)
}

cacheSolve <- function(x, ...) {
# Return a matrix that is the inverse of 'x'
#This function is to calculates the inverse matrix of the special "matrix"
#created with the above makeCacheMatrix function.It will first check to see if the inverse
#has already been calculated. If so, it gets the inverse from the cache and skips the computation.
#if not,  it calculates the inverse of the data and sets the value of the inverse.

  inv <- x$getInverse()
  if (!is.null(inv)) {
    message("getting cached data")
    return(inv)
  }
  mat <- x$get()
  inv <- solve(mat, ...)
  x$setInverse(inv)
  inv
}


Neither Fedora 23 nor Ubuntu 14.04 had a "yacc" command when I installed bison, nor was "yacc" available as a direct install on Fedora, If we want to adjust that to make yacc default to yacc, that's fine, but I do think it should be a parameterized command.


art/dex2oat/dex2oat.cc: In member function 'void art::Dex2Oat::ParseArgs(int, char**)':
art/dex2oat/dex2oat.cc:590:10: error: variable 'requested_specific_compiler' set but not used [-Werror=unused-but-set-variable]
     bool requested_specific_compiler = false;
          ^
In file included from art/runtime/elf_file.h:25:0,
                 from art/dex2oat/dex2oat.cc:58:
art/runtime/./elf.h: At global scope:
art/runtime/./elf.h:63:12: error: 'ELFMAG0' defined but not used [-Werror=unused-variable]
 const char ELFMAG0 = ElfMagic[EI_MAG0];
            ^
art/runtime/./elf.h:64:12: error: 'ELFMAG1' defined but not used [-Werror=unused-variable]
 const char ELFMAG1 = ElfMagic[EI_MAG1];
            ^
art/runtime/./elf.h:65:12: error: 'ELFMAG2' defined but not used [-Werror=unused-variable]
 const char ELFMAG2 = ElfMagic[EI_MAG2];
            ^
art/runtime/./elf.h:66:12: error: 'ELFMAG3' defined but not used [-Werror=unused-variable]
 const char ELFMAG3 = ElfMagic[EI_MAG3];
            ^
cc1plus: all warnings being treated as errors


yeah no worries! i just put them into a folder now so that should fix the confusion!


Sve u svemu super! Nisam ovako očekivao! Ima nekih sitnica koje nisu baš najbolje, ali nije ništa strašno. Jedva čekam da porazgovaramo o svemu ovome.


I don't feel strongly about this; but I think line 29 should read "than yacc itself is.", otherwise I don't understand the comment. :)


They would've worked, but IMO it's best to stay away from areas where you are only rescued by the spec, not necessarily the implementation.


Hmm, using your code (when it is both blocking and non-blocking), if you send an invalid command to ssh (i.e. "cata ~"), it still hangs, so I am not sure if there would be a difference. Even using stream_set_timeout on the stream does not help.

At this point, using your code, I am just checking for 100,000 empty fread's to consider it hung. I guess there might not really be a good solution for ssh.exec since, as you said, ssh keeps itself open.


Oops, that could be possible. I was in a rush.
Will try to fix tomorrow


That's a concern if this set would be updated all the time, but this set is only generated once per nodedef, and the client gets it through deserialization from the server, which is also only once per nodedef. After that it's only ever looked up in.

As far as I know the lookups in std::set are just as fast as std::vector. And it makes the insertion code a bit cleaner.


I've already changed the code so it doesn;t give a build error.. np


Great, thanks


Lero lero.


ack, I'll make this a bit more verbose too


Yes, I did that on purpose as I wanted to display the game id inside <>, e.g. Welcome to game <id>, like on the wireframes. It's not something really necessary so I'll change it. Thank you


*welcome to game <id>


`parentModule` is a function.


"< 1 >"


I can't upload the task nether, I don't understand the process of.


solr auf travis zum Laufen zu bringen ist übrigens kein Problem, da es bei mir lokal problemlos ohne root-Rechte läuft.


I will take a look at it tomorrow (so, basically later today :-)).


Whoops... missed the period at the end.


Also changed the secret keys...


I went to go fix this back up, and there isn't any tab/space consistency in the code. Jamming everything through a formatter is probably a good move after we get prototypes and header files working the way we want, but since it'll effectively rewrite everything in the repository I'm kind of leery doing it early.


ok!


nevermind. changed the structure, added the play store badge and re-referenced everything from the readme correctly. https://github.com/criticalmaps/criticalmaps-android/pull/154


looks good!


Fixed!


Okay.


Add an exclamation point after the word "here".


ok


reack


Seems GREAT!


This website is too confusing to me. I'll have to do my research elsewhere. All I wanted was your list of 20,000 English words, but I have no idea of how to obtain that list.


Funnily enough, I'm guessing I probably shouldn't have pushed this, but if there was a warning in the dev guide, I missed it @paulproteus


should've normalized the whitespace convention in this file & committed before actually inserting new logic.


:laughing:


no needs for extra line :)


TeamCity pull requests :: netty [Build 1098](https://garage.netty.io/teamcity/viewLog.html?buildId=5013&buildTypeId=pulls_netty&guest=1) is now running


@Nikurashi do you know how to use Git or GitHub?
You can easily download the raw 20k list here: https://github.com/first20hours/google-10000-english/raw/master/20k.txt


This looks like a potential SQL Injection point. It would be best to avoid String interpolation and use Arel structures or SQL escaping methods to put all of this together.

I'm especially worried about the hard-coded single quotes surrounding ts_headline_options in the ts_headline method.

Let me know if you'd like any assistance converting this code to use Arel structures. It can be a bit esoteric, but you can find some examples elsewhere in this codebase.


as Controller simulates Constructor pattern - use this, or var vm = this, and attach all properties to this not $scope.


OK good, except you deleted the "maybe" one!  :-)


ew


I see, thanks!


You will need egrep, if your using windows then it won't obviously work. Using ubuntu does work you just need to install the right applications. ;)


nig


method getAccounts in Controller do nothing as you're using getAccounts from service.
dataService.getAccounts()
   .then(function(response) {
       $scope.accounts = response.data;
    });


Nice - didn't know you could do that


@stevekinney is there a reason these were removed? They were the building blocks for the praxis part of each markdown file in `tutorials`. Without these files, the tests break, and the student does not have a starting point to implement the `Your Turn` sections.


:movie_camera: :video_camera: :grin:


looks good


This commit caused our VM tests randomly fail.

```
...
tester# [   14.238923] dhcpcd[561]: eth0: waiting for carrier
tester# [   14.244360] dhcpcd[561]: eth0: carrier acquired
tester# [   14.255393] systemd[1]: Started Networking Setup.
tester# [   14.256986] systemd[1]: Started DHCP Client.
tester# [   14.261491] dhcpcd[561]: DUID 00:01:00:01:1e:36:89:de:52:54:00:12:34:56
tester# [   14.267155] dhcpcd[561]: eth0: IAID 00:12:34:56
tester: exit status 0
tester: running command: systemctl list-jobs --full 2>&1
tester# [   14.286101] systemd-logind[500]: New seat seat0.
tester# [   14.297997] systemd-logind[500]: Watching system buttons on /dev/input/event2 (Power Button)
tester# [   14.309653] systemd[1]: Started Login Service.
tester# [   14.312934] systemd[1]: Starting Extra networking commands....
tester: exit status 0
error: unit ‘network.target’ is inactive and there are no pending jobs
unit ‘network.target’ is inactive and there are no pending jobs
cleaning up
```

It only happens about half the time. @edolstra , do you have any idea why?


also updated Vagrantfile to require version 0.2 of box (with Scipy installed)


this is my first commit


a vector that shrinks on pop will make this entire function crash and burn.


Yeah. It's only used locally. I'll fix it momentarily I suppose!


Test!


They shouldn't be in the tutorials section since they are not tutorials.


What happened?


Fair enough. Can I add them back under, say, a new `praxis` folder that mimics `tutorials`?


FWIW the use of this key is:

- If someone has access to your laptop, and

- They have this file (which uh now they do), then

- They can SSH into the vagrant box for this project.

So it's not a hugely important thing to maintain secrecy of, but we should fix https://github.com/sandstorm-io/vagrant-spk/issues/30 so that no one is confused in the future.

For now, I recommend:

`git rm -r --cached .sandstorm/.vagrant`


All of the `test` folders should also be removed from `tutorials` as well if that is the case.


I cant find a pull request for this or see that it was committed.


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


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


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


You're really coming along! Great job!


You sure are!


Parse's default notification title is "Notification Received". [I inherited that default in Android(https://github.com/givenow/givenow-android/blob/782f4f9ed6cec6fac528b91c7b02b153d22d4b81/app/src/main/java/io/givenow/app/GiveNowPushReceiver.java#L100-L100), you could follow the same convention


`m_` instead of `m_`? ;)


Nice move ladyada... is there any chance of renaming the github repo to match? Especially as there is a library properties file there now to make the library a bit more Arduino 1.5 friendly.


Lexical Analyzer will now ignore single line and multi-line comments


Uhm... Nope :P


Travis build is failing because it doesn't know about the navx jar.  Both IDEA and gradle recognize the jar...how do I tell Travis about it?


Don't "need" to.  I debated and, trying to help the organization, decided separate to start since already have two with two and thought may be more.  Can easily move to one if not, if simply one in each one.


Could we do the polyfill thing here and utilize `Object.assign` if it exists? Since newer node supports that.


:+1: Thanks!


Careful using this function - remember that the donor's name won't be readable to other users if and only if they are connected via the confirmed volunteering handshake. Come to think of it, you probably shouldn't even need this function almost anywhere - because for notifications, for example, the cloud code supplies you with the name if it's there. [Take a look at the cloud code here](https://github.com/GiveNow/givenow-cloud/blob/83d0e48341fd129d38f4bf3336d63d7c6b8be600/cloud/main.js#L71-L86) - the donor name, if it exists, is passed back to you in addition to the address.


I don't think that'll have the intended effect when error display is turned off, surely a `is_readable()` or `file_exists()` check would be better, something like this:

````
$path = Jetpack::get_module_path( $module );
if ( ! file_exists( $path ) ) {
// remove module
continue;
}
require $path;
````


Fixes #451, #470.
Closes #468.


No idea. Don't even know what rebase does.


@Americas what's this project's versioning strategy? Should this change not require a (major) version bump? Thanks in advance!


I think we're using this function for displaying something -similar- to the
notification in the app (for example, the pop up menu). But it's a good
point, will see if we can avoid using this!
On Fri, Jan 22, 2016 at 7:07 PM Afik <notifications@github.com> wrote:

> Careful using this function - remember that the donor's name won't be
> readable to other users if and only if they are connected via the confirmed
> volunteering handshake. Come to think of it, you probably shouldn't even
> need this function almost anywhere - because for notifications, for
> example, the cloud code supplies you with the name if it's there. Take a
> look at the cloud code here
> <https://github.com/GiveNow/givenow-cloud/blob/83d0e48341fd129d38f4bf3336d63d7c6b8be600/cloud/main.js#L71-L86>
> - the donor name, if it exists, is passed back to you in addition to the
> address.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GiveNow/givenow-ios/commit/b85274c3450a5b5f6bca95e5fdae250dfe4d1558#commitcomment-15627684>
> .
>


Certainly. This will give you the result you're looking for. I've also changed the glyph to match what Calypso uses for the sharing menu.

[icon-menu-sharing.pdf](https://github.com/wordpress-mobile/WordPress-iOS/files/101696/icon-menu-sharing.pdf)

Before:
![screen shot 2016-01-22 at 10 10 55 pm](https://cloud.githubusercontent.com/assets/1396552/12527975/d42ab476-c155-11e5-9a7c-3dce7e8a453e.png)

After:
![screen shot 2016-01-22 at 10 15 54 pm](https://cloud.githubusercontent.com/assets/1396552/12527976/d715c946-c155-11e5-88b4-b18b776fbd98.png)


Good point! I do get sloppy with the old optionals. Will go through and use
that pattern!
On Fri, Jan 22, 2016 at 6:57 PM Afik <notifications@github.com> wrote:

> I noticed you nil check Optionals a lot - what do you think of the if let
> locKey = locKey { } paradigm instead?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GiveNow/givenow-ios/commit/1622a21617ef3b55527dcbdcc6a0b52cd8c59d56#commitcomment-15627640>
> .
>


Access modifier.


what is serverResponee...


Excuse me.
I have a question about the rpmb function.
According to emmc spec, the rpmb command sequence should be CMD23 -> CMD25 for each rpmb request.
But I don't see CMD23 in the code.
How does this utils send CMD23 to mmc card?


don't list FileLoader.hs here, not sure about others


Why was that commented out?


There are some comically bad bugs in this commit, so at the risk of outing myself as careless I will document them here (for the record, I was in deadline crunch mode; I probably shouldn't have been adding features in the first place):  First, `n_pruned_dist += 0`.  What?  So `n_pruned_dist` never increases?  That's useful. Second, `best_so_far` is never updated, so it's always `INT_MAX`, so `n_below < min(query.shape[0], sequences[n].shape[0])` will never happen.  So in some sense the `n_pruned_dist += 0` line will never happen, so maybe that bug isn't so bad.  What makes it all funny is that I recall concluding that this method of pruning basically just wasn't helpful because `n_pruned_dist` was always being reported as 0, when in fact my implementation was broken in two ways.  Really, an argument for unit testing and code review always, even in academia.


can we go with `mainOctTree + ...`, as in master?


autobuild ？


В коде не используется написание ПРОПИСНЫМИ БУКВАМИ, это очень плохая практика. Если в макете присутствуют элементы, например заголовки или надписи на кнопках, то используется CSS свойство text-transform (http://htmlbook.ru/css/text-transform).
Ниже по всей разметке тоже исправить.


just a note, some of these `CInt`s should become `CChar`s when merging


Ah yes, I see what you mean. I considered checking file existence, but dismissed it because it would be slower. I'll first try turning on error reporting using `Jetpack::catch_errors` and if it doesn't help, will implement `is_readable`.


No reason for this to extend Exception


@ccx-testerbot runtests master


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


@ober01 How to use the func: func animationTabBarHidden(isHidden:Bool)?


If you are not modifying value, it shd be final.


那access-token是要存到数据库里吗？如果开启授权这个功能，应该改哪里，比较小白，谢谢帮助


@Nekokabu thanks for reporting this issue. I will try to fix it as soon as possible.


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


В данном случае нужно использовать тег <nav>


If you're really worried a `file_exists()` call is slower than enabling error reporting and an output buffer, you could just check the return value of the include statement.
```
if ( ! @include $path ) {
   // deactivate module
}
```


Combine the nested ifs


Why is this commented out?


@brantburnett -

Do you think this is ready for merging? I can use it for changing tracking, since there are quite a few holes and this is a big one. If so, send a PR and I'll review and merge it this weekend and then rebase my WIP change tracking on top of it.

ATM, i am kind of blocked because change tracking requires a resolvable primary key, so some UT's fail.

Thanks!

-Jeff


This commit does not exist


I don't understand this. Does it have a side effect? If so, it's not obvious.


rough but simple


We're keeping it Web Bootcamp Club


what...?


@mikepb Here's a more mundane question. We have `verifyDelta` and `generate_key`. We can:

1. Use `verifyDelta` and `generateKey`, which I feel like is better JS, but we'd have to add in a backward compatibility bit to allow previous users to use `generate_key`.
2. Use `verify_delta` and `generate_key`, which negates the need to have a backward compatibility thing but is worse JS style.

Leaning toward #1 myself but LMK what you think.


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

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


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


This commit looks good


要存在数据中的，要实现这个方法 https://github.com/iiYii/yii2-rest-demo/blob/master/models%2FUser.php#L31


@ccx-testerbot runtests master


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


Isn't it a little dangerous to change public functions' fields? This will affect all the example and existing codes. I would suggest preserving the previous getXYZ and getAcceleration, and add new supplement functions.


총판코드가 잘 넘어오지 않고


day la bai hoc git dau tien cua toi


Yes, I got ahead of myself.
With Known Devices, any static assignments made in ethers are transferred into uci - and as such will appear in dhcp.leases


flgで入る値も１か０なのでboolean型がいいのではと思います。


section - не простой тег для понимания, давай разбираться с его применением.
Вот спецификация к которой стоит обращаться первое время, что бы проверить правильное применение тегов: https://html.spec.whatwg.org/multipage/semantics.html#the-section-element
теги article и section используют для смысловой разбивки контента страницы. Логотип, навигация, блок входа и корзины - это не контент.
В данном случае просто div


См. выше.


adding this do we still need to build power.8974?


Вот тут начинается контент страницы и логично использовать тег main


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

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


Следи за отступами. Когда код написан аккуратно и по однообразному стайлгайду - он легко читаем, в этом месте потерялся отступ у элемента списка и уже не понятно, где начало, а где конец списка. Поправь отступы по всему коду.


Looks like you've figured out how to integrate our decks and cards into your function, I can help you with the actual process of moving the cards to the board.


It depends on how you say it. If you say it like it's a word then it's 'a fib', whereas if you say the letters it's 'an f-i-b'. I was guessing that you don't say fib so then it would be 'an'.


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


Are you asking why I added 'A' before useful? It's needed to be a complete sentence.


Remove variables unused.


如果不好翻译，我觉得可以保留range原文。
没必要每个词都翻译为中文（那可能是出版规范的要求，其实很不利于理解原文）。


Sorry for inconvenience.
I updated because I found 1.7.4 on page [npmjs](https://www.npmjs.com/package/bower) at last night.

My current local environment.
```console
$ bower --version
1.7.4
```

And my testing deployment was succeeded at last night.
```
-----> Deleting 4 files matching .slugignore patterns.
-----> Fetching set buildpack git://github.com/qnyp/heroku-buildpack-ruby-bower.git#run-bower... done
(snip)
-----> Using Node.js version: 0.10.21
-----> Installing JavaScript dependencies using Bower 1.7.4
```

However, now v1.7.4 is not available at npmjs and also v1.7.3 was unpublished.
https://twitter.com/bower/status/690182713043783680

So, I just reverted last commit.
Thank you for notifying!


没有建议。只是因为你这个术语精校，所有挑刺也多一点


感觉很饶（读者不会进入你设定的场景的）。


:clap: :heart: :kiss:


I have a feeling you'll warm to it with use and exposure.


@mikepb Any thoughts on whether we should deprecate the QR codes functionality or should we keep it? Seems to me that something like this doesn't necessarily need to return URLs to QR codes, and it might be a bad idea to encourage loading QR codes for secrets from a third party instead of making the user generate their own.


What's it happening if an unexpected value is entered? You should prepare users' mistakes.


Comment


Please take a look at EventInfo, EventData, EventManager, EventCollection and EventProperties classes in game's decompiled code. It looks like CO already made a basic implementation of events a while ago but for some reason haven't released that feature yet. I think you better use existing events framework. First, because if CO eventually brings events into the game your mod will be already ready for the feature. Second, it's always nice to reuse code instead of duplicating ;)


Two switch statements are similar, so KEEP IT SIMPLE. You'd better to user FUNCTION to remove redundancy code.


PlantCompBaseClassAgain (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (2056 of 2056 tests passed, 329 test warnings)

 * 334 tests had: AUD diffs.
 * 329 tests had: BND diffs.

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


use `Control.Monad.unless`


@ccx-testerbot runtests master


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


Главным образом input предназначен для создания текстовых полей, различных кнопок, переключателей и флажков. По спецификации input не требуется в обязательном порядке помещать внутрь контейнера form, но если введенные пользователем данные должны быть отправлены на сервер, где их обрабатывает серверная программа, то указывать form обязательно.


what did i do here im so cnfused...


Please disregard this. Looks like I overlooked the exclusion of character `c` in `IdentityEscape`.


I like that you're returning here instead of nesting further


chỗ này định nghĩa rồi mà?
Thêm vào làm gì nữa?


Т.к. эта кнопка нам нужна для отправки формы, то нужно использовать input c type=submit


yeah... xd


If no use, delete it.


is it `file_loader_stub.h` or `FileLoader_stub.h`?


good catch, although i'm not sure gradle much cares. This is a key:value association, meaning each element in the list has the associated variable its affecting. in this case, the extra comma is read as a null element in the list, which the parser ignores, since no ke is specified.


what is it used for?


oh I blame you ;)


If you're rebasing, you might as well delete this file when it was first added.


Nice! I didn't know we were still maintaining this. I am long due for an update.


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

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


Cool.


object s last assigned on line 3 could be null and is dereferenced at line 4

Hello.java[2] : start of procedure test()

Hello.java[3] :

Hello.java[4] :


Or you could use `try` and `except`...


Upon further review, I think this code is actually working properly.


Chakra behaves differently regarding this, too. `/[\c%]/` (any character except `-` and `]` after `\c`) is evaluated the same way `/[\ca]/` is, for example:

    /[\c%]/.test('\x05') === true; // 0x05 === '%'.charCodeAt(0) % 32
    /[\c%]/.test('%') === false;
    /[\c%]/.test('\\') === false;


thanks.


:+1: for fixing that security hole


object s last assigned on line 3 could be null and is dereferenced at line 4


@bgerstle does this look ok? Do u need others to play nice with vim?


Removed this due to copyright reasons (I dont want to get caught redistributing these.


commit 后面是版本号 e86ecb82e2614f2e9ab489e969ea6dba01a7d164
https://github.com/wal139491/yanshi/commit/e86ecb82e2614f2e9ab489e969ea6dba01a7d164


Hello, @Bartekus. Why use the "compose"? i don't see  him calling


ok


Intellij telling me: Cannot resolve symbol defaultVal.
I think you should use method getDefaultValue


Hi,Any reason for the revert?


Why?


Chỗ nào m nhể.

t search thấy paser data của post method cần dùng, nên định nghĩa thêm vào đó.


Eh, looks fine to me.


as long as this is your insertion point, this will work just fine on the fly.


```TeX
$ l_{Bai's nose} = \infty $
```


Thanks @slavapestov ! The commits I remember most fondly are the ones that make code smaller.

I have https://github.com/apple/swift/pull/1058 that reinstates the changes, and fixes the problem which caused the build to break. (cc @dabrahams).


melody


@luiges90 貌似有问题，如果是result += i.id + " " + i.dialog+ " ";的话跟dialog没有区别啊，相当于复制了一个dialog  ？不是应该写在人物列传上吗？


那句是儲存，格式與dialog一樣。儲存的實際位置在SaveGameData那裡決定


Что такое тег element?


@kpxp 你那邊是否有問題，同一個信息你發了五次有多了 :sweat:


Uh, did your crate's *dependencies* rely on this commit? Then I'll issue a new release soon.


我用的是手机热点，我按了很多下，延迟惹的祸


В этой строке два элемента.


Ah good point. I just work in emacs so I don't notice the IntelliJ stuff


@luiges90  不是，我是想问这个具体作用是什么？


把年表內容轉成字串型式儲存


No need for "with interception" in this xml-doc comment.


Looks good!


пока основная ветка


Thanks Fernando!


为什么我按照你所说的去改，在人物列传看不到任何有关事件的记录@luiges90


Очень общее название класса, что значит "элемент контента"?
Имена классов нужно придумывать так, что бы не открывая макет было понятно, что размечено.
Вариантов много, это можно назвать каталогом, товарами и т.д.


I don't have a short repro for it, but a large case is repro'ing that this is checking on an END marker, and that's not legal.  Do you see a possibility that such a situation could happen?  *(I see it checking against `len` below, could ++val be beyond the valid data and hit an END marker?)*


你劇本如何改的？


git ls-files -i --exclude-standard


Опять непонятное название класса, как понять что такое "карточка элемента" ?


Here's the test from back then, which failed handily (the relevant branch no longer exists so github gets a little confused showing a diff, but browsing the code reveals it really is switching the move selection to use only score):
http://tests.stockfishchess.org/tests/view/5637f6a20ebc59227df16e25


can some one please explain how to use this in your website


Опять же не говорящее имя класса.
Дальше не буду комментировать каждое имя - пересмотри все названия классов, так что бы не глядя в макет было понятно что размечено.


Changed.


@anschang Why would you do this instead of, for example, using `array.pop()`?


Это стандартный элемент, называется pagination как делается очень подробно разобрано вот этом курсе https://htmlacademy.ru/courses/55


t vừa sửa rồi nhé, m check xem sao.


Interestingly enough :-) I did just change VAL_TYPE to go from 0..255 instead of 0..64!  The reason is for performance.  Because the low 2 bits are used for other purposes (marking END, writability) the type is in 6 bits above.  But then that means if you want it to be extracted you extract it by having to shift.  If the type constants themselves are shifted you can just mask the bits without a shift and compare...

So an assert is of course a good idea!  But my point about the mask of 0xFF is that it may be "safe" to stay in the array but "unsafe" in what it means or what the code does.  Maybe always those bits are 0 for some technical reason (etc.)

Anyway there is now VAL_TYPE_0 (get the type as a number starting at 0) and REB_MAX_0, in addition to the faster VAL_TYPE and REB_MAX.  We should pick a static assert and put it here, but also lots of places can probably use more static asserts.  So if you see one you like best let me know.


按照dialog的格式


Hey thanks for the tip Fernando!


My basic premise for this class with the map was so if custom types needed to be specified they could use the same class.


Hey thanks for the comments and the tip Fernando!


Rather than return `RelationshipType?`, let's just return `RelationshipType` and throw exceptions instead of returning null.  Perhaps our best bet would be an `ArgumentException` (https://msdn.microsoft.com/en-us/library/system.argumentexception(v=vs.110).aspx)?


"Has no" (not "has no **a**"), or "doesn't have a".


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


Reviewed. For some reason it won't let me cherry pick that into the version 3.0 branch. No idea why. Can you push it there too?


We're not depending on the commit, but `chrono` is the source of `rustc-serialize` being added to our transitive deps, which is slowing down our builds by a decent chunk.


This is good because MVC does not prefer this.


欄位名稱是 ScenBiography 有沒寫錯？


Likewise, we can throw an exception here.  It can be of type `System.Exception`, unless you have another suggestion. For the message, we might want something like: `string.Format("Cannot determine relationship type for {0} property on {1}", propertyName, entityType.FullName)`


在 https://github.com/kpxp/ZhongHuaSanGuoZhi-New-Code/blob/master/GameObjects/GameObjects/GameScenario.cs#L4618 這裡，把try-catch取消掉，看一下是否讀取出錯...


@luiges90 栏位名称没错


还是不明白，这一栏到底是什么作用==


這欄就是寫武將列傳的內容....


(囧，我一直指的是武將詳細資料裏面，列傳部分，劇本下面的內容，不是右按空格那個年表......)


下JJ


The Win is just placeholder art, and will get replaced with the exit tubes later... jst havent added that yet


when u commit all files u changed instead of just the 1 u meant
o h well


:'] It's beautiful


you don't have to `return false;` here.

![bey]
(http://static.guim.co.uk/sys-images/Guardian/Pix/pictures/2014/8/19/1408440593122/Beyonce-single-ladies-Gla-011.jpg)


Старый gcc будет ругаться тогда, что метод без return.


Damn, I thought I fixed this :smile_cat:


아드레노아이더가 애초에 gpu를조종하다보니 평상시에는 썩차이가나진않지만 동영상감상등 gpu가활용되는부분에서 효과를발휘할겁니다. 그래도없는것보단낫겟죠


Метод класса-враппера вызывается. env это не JNIEnv * здесь


Sono cara, tive que quebrar a cabeça para fazer esses botões diferentes com um significado parecido, porque a tradução real é grande e como o layout é fixo, não dá para usar palavras grandes nas imagens, tive que procurar alternativas que fossem semelhantes.

Já os erros no .ini foi falta de atenção mesmo, fiz de madrugada e.e


Never do a squash on the master usually  it will require force. It is just too dangerous. squashes should be left to pull requests.
Did we mention pull request as a better approach ? :)


extracts selected words when you press enter key


this line was too long, i can't read it.


@luiges90  那这个算不算是copy  dialog的内容？


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


Here's my suggested language.:

> Abbreviations are stored using universal variables. You can create abbreviations directly on the command line, and they will be saved automatically. Do not place them in config.fish, because this will lead to worse startup performance.

I'm fine with krader's idea of giving an example too.


Here's my suggested language:

> Do not append to universal variables in config.fish, because these variables will get longer with each new shell instance. Instead, simply set them once at the command line.


Thank you.


I forgot that the version option is -V, and verbose mode is not enabled now... Sorry for that.


There are many times where mint unable to connect to server. Logs spamming timeout connection from mint.


Isn't this going to take the binary UID and decode it as UTF-8, replacement characters and all? Or is this storing some ASCII serialization of the UID already? (I've forgotten how the mess of encodings works here.)


I think the last two args are out of order?


Same comment as on line 62.


ここなんですが、パスワードなしでprofile編集するために、deviseのregistrationsコントローラのカスタマイズが必要なので追加しました。他は今は使う予定がありませんが、いざという時に上書きできたら便利なので追加してあります。


Why would the interceptors be mandatory?  If there are no interceptors, then `ApplyChanges` just sets entity state normally.  Let's go ahead and switch the order so that interceptors is last and optional (`= null`).


@jcome Yes. ^__^


propertyOwner should be a Player object.


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


I also forgot to mention that my test box only had Chrome and IE at the time, no FF. So I had to modify this to be able to load that file.


Don't set a variable here, just return the random int.


I mostly use nosetests as my test runner and nose eats stdout by default, showing it only if the test fails which is the best of both worlds.


Don't have a class variable here.  In the while loop make a local variable to store the result of rollDice.


PlantCompBaseClassAgain (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2056 of 2056 tests passed, 329 test warnings)

 * 334 tests had: AUD diffs.
 * 329 tests had: BND diffs.

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


Putting these in an ArrayList will make this easier, especially when you want to add more properties later.


Instead of doing this, consider overwriting the toString method in Property.  Then you can just call System.out.println("They landed on " + property);


This is of the format  if (value == true) return true else return false.  Instead, just return the value.


`IsTrue()` is actually the [Truthy](https://developer.mozilla.org/en-US/docs/Glossary/Truthy) check?


I have disable all the plugins ,but still has this issue,can not find pom ,if I open a new file in new directory.


php7 bump work is still continue..!


サーセン


Use a local variable instead so other functions don't mess with this value and cause bugs.


Pass this into the function instead of using a class variable.


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


What if they have exactly the amount of money it costs to buy it?


I just realized I didn't test this before I merged it. We should test this tomorrow.


You could also implement toString in the player object to just be their token and then you can just do sopl(player + " has bought" etc.


Property owner should be a Player so this should just be .equals(player)


This is a boring story line, make up something that happens.  Possibly randomly chosen from a list!


複製dialog內容用來測試的...


@sgrif Okay, done!

Note: I was a bit unsure how to test this commit. I originally thought that this commit will remove `rand` dependencies from the crate itself, but it took a while to realize that this is not the case. Rather a new crate depending on `chrono` should not receive the `rand` dep, which was what I should have been testing (and I confirmed that for this commit).


You should probably return a value here instead of System.exiting in case you want to have multiple games happening in one run of the program.  Maybe change the main loop from while(true) to while(checkBalance) or something.


is this correct


Isn't this just switching the lookup to generate `depends` from the methods in `catkin_pkg.find_packages()` to `rospkg.RosPack().get_depends()`?

What extra is the `rospkg` call is getting you here?


Great work @danielsega, can you give me a brief on what all the systems are doing and the general overview of how the Quest System will work?


k is an array that contains the player's cells.


Should be changed back to non-nullable.


@priyatam You're the :bomb:! Thanks man!


Lol


Change `RelationshipType?` to `RelationshipType` in this class.


Ah, default call with the rospack approach uses `implicit=True`.  I'm not sure we should support this - since the package is using `message_generation` directly it really ought to be an explicit dependency.


本来は不要な記述は消すべきだね。
残ってるのを他の人見たら混乱するし、残してるメリットもそんなないし。(必要な時に追記すれば良い)

一人で開発やってる時はどっちでもいいけど、
今回は俺という共同開発者(?)がいるので、消しておいて。


Here's where the issue was.


And I think this would mean all users of a message package would in turn be marked as a package to be `generated`. We don't want that.


Fix for issue #27


「余り（よくわからない子たち）」って言われても俺もわかんないから、分かるようなコミット名にして


このコミット名もだね。
何やってるのか分かんない。


Fix for issue #10 and duplicate #31


Hi,
I committed your additional removal of '&' changes to the main codebase:
https://github.com/openemr/openemr/commit/c13771728cc9b83bf9152a829b08984e7dbd93c7
-brady


Make this a space


git clone git@github.com:brave/LinkBubble.git


Ack.


We should change the name of `ChangeStateByInterceptors` to `SetEntityState` and refactor as follows:

```csharp
private static void SetEntityState(DbContext context,
    ITrackable item, ITrackable parent, string propertyName,
    EntityState state, IList<IStateInterceptor> interceptors)
{
    // Set state normally if we cannot perform interception
    if (interceptors == null && parent == null && propertyName == null)
    {
        context.Entry(item).State = state;
        return;
    }

    // If there are any state interceptors, call them to try to get state
    RelationshipType relationType = GetRelationshipType(context, parent.GetType(), propertyName);
    foreach (IStateInterceptor interceptor in interceptors)
    {
        // If current interceptor returns the state, use it
        var entityState = interceptor.GetEntityState(item, relationType);
        if (entityState != null)
            context.Entry(item).State = (EntityState)entityState;
    }
}
```
This will allow us to call `SetEntityState` whenever we needed to call `context.Entry(item).State = state`.


This formatting is off, it should be 4 space indents


Hi,

I separated your adodb into two commits on this branch:
https://github.com/bradymiller/openemr/commits/php7-work_1

Note the first commit upgrade adodb(note I did not include the testing scripts):
https://github.com/bradymiller/openemr/commit/c1c95108ae2b848a154c4ac637e1caa9336acdc0

Note the second commit just shows the library/sql.inc edits:
https://github.com/bradymiller/openemr/commit/3d0dfe491f84146e0677cdd16a5cda8a61831fe7

-brady


`extends` should be an 8 character continuation indent


Hey Eric,
Extremly sorry for not committing everyday as I was working on reduction of
implementation as it was around 1300 lines so I reduced it my best possible
way .
During that I was completly messed up with the code so couldn't able to
commit everyday.

Style thing is not working just some thing need to be done will update then
and will commit tonight.

I have committed code please check that out

Warm regards,
Piyush
Hey  Eric,
I have committed some changes sorry for the delay
Logo dynamically created
icon theme added.

On Tue, Jan 19, 2016 at 4:20 PM, PIYUSH PATEL <6394piyush@gmail.com> wrote:

> Ohk thanks ! :)
>
> Regards PIYUSH PATEL
> On Jan 19, 2016 4:12 PM, "Eric Gopak" <notifications@github.com> wrote:
>
>> I hope you can find the answer yourself. It is possible.
>>
>> Am Dienstag, 19. Januar 2016 schrieb 6394piyush :
>>
>> > I will add font options also.
>> > I want that when grey icon is selected then all the colored icons
>> should be
>> > grey. is it possible or I should manually convert each of them grey and
>> > again use the DataUrl ?
>> >
>> >
>> > On Tue, Jan 19, 2016 at 4:00 PM, Eric Gopak <notifications@github.com
>> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
>> > wrote:
>> >
>> > > Instead of icons you might want to use FontAwesome ;)
>> > >
>> > > Am Dienstag, 19. Januar 2016 schrieb 6394piyush :
>> > >
>> > > > Sorry I mean to say Grey icons From existing colored icons is that
>> > > > possible?
>> > > >
>> > > > Regards PIYUSH PATEL
>> > > > On Jan 19, 2016 3:41 PM, "Eric Gopak" <notifications@github.com
>> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>
>> > > > <javascript:_e(%7B%7D,'cvml','notifications@github.com
>> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>');>> wrote:
>> > > >
>> > > > > What do you mean by grey shade? There is a CSS rule for box-shadow
>> > > > > <https://developer.mozilla.org/en-US/docs/Web/CSS/box-shadow>
>> > > > >
>> > > > > —
>> > > > > Reply to this email directly or view it on GitHub
>> > > > > <
>> > > >
>> > >
>> >
>> https://github.com/6394piyush/Mail-Signature/commit/cd0f5f94a1a969b0c66817e026d0e80c6d8209f6#commitcomment-15525730
>> > > > >
>> > > > > .
>> > > > >
>> > > >
>> > > > —
>> > > > Reply to this email directly or view it on GitHub
>> > > > <
>> > >
>> >
>> https://github.com/6394piyush/Mail-Signature/commit/cd0f5f94a1a969b0c66817e026d0e80c6d8209f6#commitcomment-15525768
>> > > >
>> > > > .
>> > >
>> > > >
>> > >
>> > > —
>> > > Reply to this email directly or view it on GitHub
>> > > <
>> >
>> https://github.com/6394piyush/Mail-Signature/commit/cd0f5f94a1a969b0c66817e026d0e80c6d8209f6#commitcomment-15526078
>> > >
>> > > .
>> > >
>> >
>> > —
>> > Reply to this email directly or view it on GitHub
>> > <
>> https://github.com/6394piyush/Mail-Signature/commit/cd0f5f94a1a969b0c66817e026d0e80c6d8209f6#commitcomment-15526208
>> >
>> > .
>> >
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/6394piyush/Mail-Signature/commit/cd0f5f94a1a969b0c66817e026d0e80c6d8209f6#commitcomment-15526334>
>> .
>>
>


What is this magic?


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


Hi,
I am a bit confused by the library/sql.inc edits since there are still functions there that are not setting $binds to false by default. Shouldn't they all be setting $binds to false by default?
-brady


i delete config and start from empty
now start test after fix rcu problem

m.
On Jan 23, 2016 03:37, "Gao Feng" <notifications@github.com> wrote:

> which kernel config options could resolve the issue?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gfreewind/linuximq-HighPerf/commit/439f320fa195800fcf989e990434b67b4ffdfbb8#commitcomment-15627223>
> .
>


Format this like
```java
@Test
public void testPrepare()
{
    assertStatement("PREPARE myquery FROM select * from foo",
            new Prepare("myquery", simpleQuery(
                    selectList(new AllColumns()),
                    table(QualifiedName.of("foo")))));
}


Check all these assignments for null


Nice :+1:


this at least needs allow-scripts or it will break ad serving


Also,

Have you gone through the installation of OpenEMR on PHP7. It's just odd that phpgacl install is working since it has it's own adodb outdated library here:
https://github.com/openemr/openemr/tree/master/gacl/adodb

-brady


I am a bit confused by the library/sql.inc edits since there are still functions there that are not setting $binds to false by default. Shouldn't they all be setting $binds to false by default?


you should change this back to a >> 8 and compare against 248 & 8.


It is best to avoid having logic in multiple constructors. Instead, have all but one constructor call `this(...)` so the assignments happen in a single place.


Will the phpgacl adodb library also need to be updated(or removed and direct to the library/adodb library)?
https://github.com/openemr/openemr/tree/master/gacl/adodb


Check these for null before putting in map


Use `ImmutableMap.copyOf(...)`


This should be `ImmutableMap.of()`


Move this method above `explain`, to match the others


oh right. thanks


存在问题：

1. 抓取结果没有抓取结果的日期

2. 抓取结果没有区分红蓝球和排序


Since this is relying on the formatter for correctness, we should verify it round trips. Make the `getFormattedSql` method in `CreateViewTask` take a `Query` and reuse it.


I'm looking at your @clusterfack %)


???

A removed footprint, and ice not added?

Also I'm kinda against including shell scripts without a makefile. One of the reasons mpvc hasn't been put into this repo yet.


@mikepb Any idea what this is doing here? It seems like it should be `options.token` but the test fails for obvious reasons.


Make these null checks standalone. (I already find the `addSetSessionProperties` method hard to read and this one is more complex)


I think i have an issue with memory on my 4GB mac book air. In particular when trying to get amenities withgetAmenitiesList().


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


Let's call this `name` instead of `identifier`


So there are a few issues with the tests. First off, you have built the system with resolution 10 and 100 beads. That means that the Residue objects are actually destroyed since they won't be simulated. So the TempResidues that are now assigned to the same bead actually store references to the highest resolution particle available - the residue 10 beads. So you don't expect 581 particles from get_atomic_residues() - only 59 or so. I don't know of a better way to deal with this.

The tests for rigid members are definitely good though, I will try to fix those.


![simulator screen shot jan 23 2016 12 14 09 am](https://cloud.githubusercontent.com/assets/16545991/12528957/01ab9c88-c176-11e5-99a5-83f821df094e.png)
![simulator screen shot jan 23 2016 12 14 14 am](https://cloud.githubusercontent.com/assets/16545991/12528962/11817f42-c176-11e5-8fd4-ed0bf1b7cd23.png)


Вопросы:
* закоммичен генерированный код
* потокобезопасен ли доступ к _participants_? Исправить, если нет
* именуйте примитивы синхронизации в соответствии с тем, что они защищают
* каким образом обрабатывается ситуация прихода части сообщений по TCP-протоколу?


The prepared statement name is an identifier which could contain anything (if quoted) such as equals signs, newlines, etc. It will need to be encoded.


The same goes for the value.


Any data of benchmark for performance improvement?


Nit: space after `//`


Personally I would like to remove the warning completely - but your text is at least more clear!


Wrong name for threads


Fix for issue #15


Yes I know... Thanks for reminding me :)


Thank you :)

For lines 213-216 I have a change also, I'll handle them.


@AdFad666

Double definition of

<!-- enable doze powersaving mode -->
<bool name="config_enableAutoPowerModes">true</bool>

see string 227


I pulled changes from sh and merged with mine.

There was a mistake, I pushed the wrong copy of index that I use in my
working copy. Sorry my bad.

On 1/22/16, Shubham Prasad <notifications@github.com> wrote:
> Take important part from body of this commit and put them in
> views/homepage.html and then revert back the changes to this page to its
> original state.
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/shubham2192/zPrj/commit/197a901b0e0cbfe1d140a8f4c9b90486150aed45#commitcomment-15610244


@DaveChild Thanks for pointing that out. But this wasn't working in the local environment, any idea why?


@Crotalus sorry for not making an issue about it. I suggest we open a new PR and discuss the implementation there?


Same with transport-lock this feature wasn't consistently working. I suggest we discuss it in a new PR.

My reason for not making an issue is that issues should revolve around the current codebase -- not code/features that aren't merged.


I had to change it back to ManaDex to get it to work, but it did work fine after that.  I'm really interested in knowing how it would work the other way.


我发现我是个疯子.(I found out that I was a madman.)


Вопросы:
* закоммичен генерированный код
* убита асинхронность записи:
```
                while (!written) {
                    buffer.position(0);
                    try {
                        currReciever.write(buffer, null, handler);
                        written = true;
                    } catch (WritePendingException e) {}
                }
```


Also fix the bug after finishing running, i.e.,  "bin/singa-run.sh: line 109:  5943 Segmentation fault      (core dumped) $singa_run"


inv <- NULL
set <- function(y) {
mat <<- y
inv <<- NULL
}
get <- function() mat
setInverse <- function(solve) inv <<- solve
getInverse <- function() inv
list(set = set, get = get,
setInverse = setInverse,
getInverse = getInverse)
}


Вопросы:
* Пояснить необходимость _synchronized_ в методе _broadcast_. В случае отсутствия необходимости - исправить
* При высокой нагрузке кидается _WritePendingException_


help is missing, issue:#726


This changes a behavior unexpectedly, and cause failing test.  I'd like to revert.


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


will this now be a permanent change or will you revert this here later again?


> Simple Module Config file (MyModuleName/config/thumbnails.php)

```php
<?php
[
return [
    'coverImage' => [
        'fit' => [
            'width' => '200',
            'height' => null,
            'callback' => function ($constraint) {
                $constraint->upsize();
            }
        ],
        'watermark' => [
            'source' => 'public/assets/watermark.png', // if changing required
            'position' => 'top-right' // if changing required
        ]
    ]
];
```


delete


Local tuning with 50% higher ck values moved much more. This is either very close to optimum, or ELO-insensitive. It's really high time for last attempt here.


First commit


great. makes sense. Thank you!


下JJ


You did a great job, neolib is exciting :-)
Thx for sharing!

/Andreas


是乱码，看不了


Test


You should have set your name and email, something along the lines of

```
git --global user.name "[your name]"
git --global user.email "[your email]"
```

But I already graded you, you did great work.


Weekly passed: https://scala-webapps.epfl.ch/jenkins/view/Scala.js/job/scalajs-weekly/79/


You'll have to work that out for yourself. The only suggestion I have is to check you're loading the file over http://, not file://.


Sorry, it wasn't clear: the porrectus (and porrectus flexus) longqueue should appear only when the first ambitus is 1 (for now)


1. Do not push the code which is added for local development purpose. For ex: jpa.showSql=true in application-config.properties

2. BudgetHeadFormatter.java, SchemeFormatter.java, SubSchemeFormatter.java, SubSchemeAdaptor.java
 should be pushed to respective module/projects only as we already discussed couple of times during previous code reviews. Kindly move these files to respective modules.

3. No need to write findById methods in Repositories.

4. Go through spring data jpa reference doc once before adding @Query. Spring data jpa doesn't required a @query to be created most of the time if its a simple query. For ex: You can write directly JPA style method without @Query for the method written in LineEstimateRepository-

@Query("select le from LineEstimate le where le.lineEstimateNumber =:lineEstimateNumber")
LineEstimate getLineEstimateByLineEstimateNumber(@Param("lineEstimateNumber") String lineEstimateNumber);

can be written without using @Query as

 LineEstimate findByLineEstimateNumber(String lineEstimateNumber);

5. Do not create spring beanid's for schemeService, subSchemeService from different module in applicationContext-works.xml. Instead use autowiring of these services for dependency injection.

6. Make use of common-labels.properties for commons labels across the module. Do not duplicate the same lables in labels.properties in works module.


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


Actually `alGenSources` initializes the sources with exactly that values above. I took those from an example for checking how things work. Now they are redundant.


also add m n o


add m n o


Same here. These values are initilalized in `alGenSources`.


sure


yup


 public EgwStatus getStatusByCodeAndModuleType(final String code, final String moduleName) {
       return persistenceService.find("from EgwStatus where moduleType=? and code=?", moduleName, code);
   }

can be replaced with egwStatusHibernateDAO.getStatusByModuleAndCode(code, moduleName);


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


@jbrooksuk Can you take a look at this?


I think a recent change to Samba or another package causes this code to report Samba (smbd) as not running.


It is not important to check the active_symbols is exists or not. Also it's not necessary to check the changes. Always the server response is valid. So I don't think we need this change.


It is not important to check the asset_index is exists or not. Also it's not necessary to check the changes. Always the server response is valid. So I don't think we need this change.


@DaveChild I meant that the example files were not working in the local environment with that specification, that is why I suggested these changes. Anyhow, I didn't know of the `http`, `https` relation to it, thanks for that.


hello


Is the usage of `force_str` instead of `force_text` deliberate?


The OpenAL error stack needs to be initialized by calling `alGetError()` respectively `alcGetError` once (in `Audio` c'tor before thread starts) to avoid falsely reported errors.


@ccx-testerbot runtests master


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


ok


Awesome, @arubdesu!
Thanks a lot for making this happen.

I will review it thoroughly first week of February.


Thanks for your pull request. I didn't merge it but have upgraded my repo to CodeIgniter v3.0.4 which includes your changes as well.


there could be an option to put charter name there, but I'm not sure if it would be useful, since not everyone uses that author field :(


@Nadahar yes I only removed that method. The ```Base64()``` doesn't throw any exception so to catch general Exception can hide errors made somewhere else.


Why mDate = null?


Yes, I already know I am losing my time doing that... If I use it, though, someone else probably does...


1.7.x 的代码与 1.8 的代码同步更新，但今后只会更新代码部分，注释、TODO 等非关键改动将不再同步。


lololololol


@palswim

that's true. but two days ago fix was pushed too.
https://github.com/xbianonpi/xbian-update/commit/440969808ef3b9fccd0f44a3fc904301510aae30

if it is still not correct report back.


I think it should raise an exception if match is None.


By removing this line, no one can authorized anymore.


the opencv addons for example uses static libs but it auto detects them, doesn't use addons config. perhaps check that the env var name is defined and the name is correct. in a terminal you can type set and it'll show all the defined vars if i remember well


how much of an improvement is it ? cant see the diff here


that was merge conflict resolution,,, followed by proper file, wasn't it ?


I guess you don't want to store all the results of ``secondary_source_selection`` in one big matrix, right? If this is the case, then I could just suggest to replace ``~all`` by ``any`` and ``eq`` by ``ne``/``~=``.


looks fine


It has to be ```i```, not ```i+1```


:joy::joy:


I'd put this comment above the statement it affects ;)


Like already said, pimpl'ifying the `Audio` code has several advantages:

* It allows us to provide a clean API (internal to qTox objects).
* It avoids spreading OpenAL stuff all over qTox like it was the situation before and keeps things in place.
* It encapsulates OpenAL (which even allows us to exchange it in the future with probably another lib).

Thus I disagree with this very commit, but I make it your decision.


didn't you want to remove the square brackets?


ah you're introducing an ordering problem here, let's sort paths alphabetically; just replace `range_paths` with `sorted(range_paths)` here hm?


this doesn't belong to this commit but to the previous one ;)


otherwise good


On my system, `xbian-config services status samba` returns `samba 2`, so even [the fix](https://github.com/xbianonpi/xbian-update/commit/440969808ef3b9fccd0f44a3fc904301510aae30) doesn't cover my case.

Perhaps `samba 2` means that something has caused an error with my system or configuration.


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

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


`On your main` I suppose


The classes are not showing up for some reason?


I need to update the Wiki. there is a 3rd feature mentioned below: background_colour_toggle


Summary of above:

- 1 file garbled up (as expected): XDS -> 9.mpg
- 3 timing changes in the ms ranges: General -> 5.mov, 6.mov, 7.mov
- 2 first lines gone: General -> 14.mpg and Broken -> 4.mpg
- 1 falsely reported error: Options -> Closedcaption_atsc_epg.ts with -out=null parameter


The `Notes` heading is missing


this is related, i.e. still doesn't fix it?


maybe we could have every file in one commit? So they're separated in logical changes.


1) Необходимость в synchronized отсутствует.
2) Добавил объект Future в ожидании записи.


Do you know why? I have asked @myfluxi but no answer..


@Abhishek9634 if logic is already written, then please remove this line :)


this is required by android for recreation from rotate, etc.


ok


_"Now if you REALLY wanted to get user-friendly with that you could have the object oriented XML-y thing also generate variable configs for less sophisticated users to use."_

I'd be willing to do that if that's what needed to happen. I've also been thinking about adding the config system to the Minecraft config GUI system, kinda like Plant Mega Pack does it. Not sure yet though - might be more trouble than it's worth - plus, I'm not sure the GUI would actually make it easier than just tweaking a config file.

As for the composite approach, yeah, something like that might be worth exploring too.

Although, I've just been looking at all the differences in all of the terrain files that we have, and now I'm kinda leaning towards leaving it as it is. I mean, I know all the logic is dumped in the generateNoise() routine, and there's a lot of code duplication across similarly-terrained biomes, but... it's not that big of a deal really, if I'm honest. Sure, there are 300+ biomes to consider, but those can all be broken down into about a dozen different 'types' of terrain, so it sounds worse than it is. As much as it appeals to me to clean up the way the terrain files are handled, I'm not sure any of the approaches we discussed are worth losing the flexibility of being able to just go into whatever biome you want and tweak it exactly how you want, without having to worry about affecting any other biomes.


Dizigold?


over-engineered version: https://github.com/DanielFGray/dotfiles/blob/master/local/bin/diffsed


this is ok to merge now, but the "This also applies to running 'ipfs repo gc' concurrently
 +with '--flush=false' operations." part is really a bug.


CLI tests pass on Linux and Windows.

Untagged source map tests fail because source maps refer to GitHub even for Reversi.scala. So tagging to do tagged source map tests.


make that fresh olive oil


Just tried to install plotIt with the latest change, I get a yaml-related crash:

curl: Saved to filename 'yaml-cpp-release-0.5.3.tar.gz'
-- The C compiler identification is GNU
-- The CXX compiler identification is GNU
-- Check for working C compiler: /cvmfs/cms.cern.ch/slc6_amd64_gcc491/external/gcc/4.9.1-cms/bin/gcc
-- Check for working C compiler: /cvmfs/cms.cern.ch/slc6_amd64_gcc491/external/gcc/4.9.1-cms/bin/gcc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /cvmfs/cms.cern.ch/slc6_amd64_gcc491/external/gcc/4.9.1-cms/bin/c++
-- Check for working CXX compiler: /cvmfs/cms.cern.ch/slc6_amd64_gcc491/external/gcc/4.9.1-cms/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Boost version: 1.57.0
-- Found the following Boost libraries:
-- Performing Test FLAG_WEXTRA
-- Performing Test FLAG_WEXTRA - Success
CMake Error at CMakeLists.txt:311 (export):
  export Unknown arguments.

-- Configuring incomplete, errors occurred!
make: *** No targets specified and no makefile found.  Stop.

Does someone reproduce this?

Thanks,

S.


alert('hehe')


I like this, all it needs is a detection if maybe already the -N was entered by the user :)


:+1:
I will try it soon.


count primes: 2 ways.
valid number: think in detail
ugly number: ugly number2 -> ugly number 3


I always use the void way because it stops any URL changing :)


No need for that ... a double -N is acceptable for par2 ... ;-)


aa


하트!


Small typo here in title (`mehtod`).


i like this :+1:


Решение некорректно, оно убивает всю асинхронность работы сервера


What do you think of removing this `elsif statement` and using just one if statement. Line 29 will become `if params[:searcher] || session[:searcher]` and line 30 will become `session[:searcher] = params[:searcher] if params[:searcher]`


@swatijadhav  could you take a look and see if the above works from your pov? I've tried to refactor things so that the model is handling the state transition. This way, the controller is simplified and we can make adjustments to the model via the console and not get out of sync.


for me I got different memory_hole values, so if I apply this patch, my phone will start up and than just stuck. Does nothing, my values are here: https://github.com/Tkkg1994/Nova-Kernel/blob/kccat6-TW502/arch/arm/boot/dts/qcom/apq8084.dtsi#L5059 d400000 to c100000 but that didn't help

I already changed


You don't need to return an empty array here since you will still convert the empty array to nil in line 35.


Should be `method_call(:private_class_method)` for consistency.


spirng


Can you explain why a `@decorator` tag is being added? What is this being used for?


Status NotModifed is used by browsers from caching pages during GET requests and shouldnt be used for business logic.
In our case we should probably return an 403 error.
Info about statuses i getting from this useful page :)
http://www.restapitutorial.com/httpstatuscodes.html


From the line above, it appears this script was broken since the multisite feature was added(since sqlconf.php was migrated to sites directory.)


most likely to stay reverted.
I had received many reports with kmalloc errors.
If you don't have issues, do not revert.


I don't think this test is complete. You didn't test for `Taking them to the specific taskee's profile on login`


Well ... tagged source map tests fail too. Reversi and Scala.js libs are mapped correctly, but Scala libs refer to the scala-js repo too :-s

I'll release anyway, and mark this as a known issue. We've waited too long for this release already.


It indeed can happen if wide = 1 (set operation) :-(  I must correct code to skip zombie checks in that case. Thank for look at!


Publishing


Wouldn't it be impossible to bring the site and bkpdir via cli mode? Don't they need to be brought in as parameters?


In what case would a parameter not be an `AstNode`?


I better stay on the save side and revert this too, thank you :)


On 22/01/2016 22:57, Paul Emsley wrote:
> On 22 January 2016 at 21:04, Paul Emsley <pemsley@gmail.com> wrote:
>
>  >
>  >
>  > On 22 January 2016 at 16:55, Bernhard Lohkamp <notifications@github.com>
>  > wrote:
>  >
>  >> will try to fix it again....
>  >>
>  >
>  >
> OK, but do it with timeouts, not gtk pending events.

I guess you mean without using gtk.main_iteration(False)!?

Any particular reason? I cannot remember why I did it the way it was
done. Maybe because it gave the best performance. Or chance. Well,
according to:

http://wiki.laptop.org/go/PyGTK/Smooth_Animation_with_PyGTK

it may be the best way. Although I will see if I can come up with a
working idle_add alternative...

B


Disregard above comment. I realized there is a script at sqlconf.php that redirects user to the correct sites directory.


TeamCity TestCI :: TestConfiguration [Build 12](http://localhost:8888/viewLog.html?buildId=22&buildTypeId=TestCI_TestConfiguration) is now running


TeamCity TestCI :: TestConfiguration [Build 12](http://localhost:8888/viewLog.html?buildId=22&buildTypeId=TestCI_TestConfiguration) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:01


Best to place the require_once outside the function at the top to avoid nasty out of scope bugs. Also, I don't think the sqlconf.php script is needed here since already brought in via globals.php.


You cannot use "true" default value since "false" is never serialized. So it always returns true regardless of the stored value.
(See #5593)


updateも必要ではないでしょうか？( ˘ω˘)


こちらもupdateも必要ではないでしょうか？( ˘ω˘)


Should throw exception


А, точно.
Странно, что названия не совпадают у функций MakeString и ToStdString, это с какой целью ты порефакторил?


I'm not sure I like the idea of defining methods on objects at runtime, as there are quite a few perf issues here. It seems like there is a better place to store this data other than shoved in a method. One place might be the [`#extra_state` openstruct](http://www.rubydoc.info/gems/yard/0.8.7.4/YARD%2FHandlers%2FProcessor%3Aextra_state).


Why don't you use static?


@VolodymyrPortianko Agree, redone.


@felakuti4life, the import works in development, but fails in production. I guess this is because the test doesn't perform the import in production?


What's the advantage here? It's not like we're going to lose track of memory in a 50-line program, and I prefer to be explicit about calling the destructor.


test


Yeah this function is causing troubles, but for otehr reasons. It seems the string is appended or prependend? depending on hasPlotyyReference()


What is the purpose of returning the methods here?


@borsch , це сарказм?
можна і так зробити. далі додам


I'm pretty sure that we need this only for the 1st phase of the project. In the next phases, the coupon id will be set without this method, because all coupons will be built from the db and sent to the client.


Can't that be done using `getFieldOptional`?


When would `transfer_docstring` or `transfer_source` not be `true`? I'm not seeing this exercised in the PR. I would argue that if you don't need it, there's no need to add it to the API (YAGNI).


Чтобы не путаться ) Есть идея, как это сделать еще красивее, но потом. Пока не хочу отвлекаться.


TeamCity TestCI :: TestConfiguration [Build 13](http://localhost:8888/viewLog.html?buildId=24&buildTypeId=TestCI_TestConfiguration) is now running


TeamCity TestCI :: TestConfiguration [Build 13](http://localhost:8888/viewLog.html?buildId=24&buildTypeId=TestCI_TestConfiguration) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:02


TeamCity TestCI :: TestConfiguration [Build 14](http://localhost:8888/viewLog.html?buildId=25&buildTypeId=TestCI_TestConfiguration) is now running


@mattahorton, this url does not work in production because it is absolute. The URL path should be relative so that it can work in any environment (development, staging, or production'


TeamCity TestCI :: TestConfiguration [Build 14](http://localhost:8888/viewLog.html?buildId=25&buildTypeId=TestCI_TestConfiguration) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:02


Actually, disregard the above comment since you don't know what the site setting is yet in cli mode.


Since this is a working script, may want to think about how to not have it break for folks that are currently using it (ie. when they set the 1st and 2nd parameter). So, would make sense to have following in cli mode:
1st parameter still takes $webserver_root, which is ignored
2nd parameter becomes $bkpdir
Then a 3rd parameter becomes $_GET['site']


TeamCity TestCI :: TestConfiguration [Build 15](http://localhost:8888/viewLog.html?buildId=27&buildTypeId=TestCI_TestConfiguration) is now running


TeamCity TestCI :: TestConfiguration [Build 15](http://localhost:8888/viewLog.html?buildId=27&buildTypeId=TestCI_TestConfiguration) outcome was **SUCCESS**
Summary: Running Build time: 00:00:02


TeamCity TestCI :: TestConfiguration [Build 16](http://localhost:8888/viewLog.html?buildId=28&buildTypeId=TestCI_TestConfiguration) is now running


TeamCity TestCI :: TestConfiguration [Build 16](http://localhost:8888/viewLog.html?buildId=28&buildTypeId=TestCI_TestConfiguration) outcome was **SUCCESS**
Summary: Running Build time: 00:00:02


TeamCity TestCI :: TestConfiguration [Build 17](http://localhost:8888/viewLog.html?buildId=29&buildTypeId=TestCI_TestConfiguration) is now running


TeamCity TestCI :: TestConfiguration [Build 17](http://localhost:8888/viewLog.html?buildId=29&buildTypeId=TestCI_TestConfiguration) outcome was **SUCCESS**
Summary: Running Build time: 00:00:02


Hi, I can't make it work. I get this error n the console when I try:

Uncaught SyntaxError: Unexpected token ;                                          WM277:1


Didn't we remove curried parameter lists?


I hope we can fix that some day :-)


BTW, CityEventManager.cs is missing from repo


@d-j-a-y  tested 861b4dd - it works fine now! ^__^


Je viens de faire le test avec mon PC pour X=20 :
Diviseur
232792560
Divider: 14889ms
232792560
Divider2: 1ms

On est pas tout à fait un 5 min ;-)


summary for Strobogrammatic Number 1 2 3.
Strobogrammatic Number 1 is easy, use hashMap to check
Strobogrammatic Number 2, use backtracking algorithm. The tricky part is to put "index" and "n - index - 1" at the same time
Strobogrammatic Number 3 is extension for Strobogrammatic Number 2. use a counter. There might be some optimization. Investigation later.


This is not the correct way to copy over a docstring, though I would really like to understand the logic here better. (more on this in general comments)


Definitely. Most things can be done in more than one way. I've tried to only use the ifPresent-approach for small statements though.


zd


You're testing the legacy parser? Does the legacy parser actually support this too? I don't see that in the PR.


TeamCity TestCI :: TestConfiguration [Build 18](http://localhost:8888/viewLog.html?buildId=30&buildTypeId=TestCI_TestConfiguration) is now running


TeamCity TestCI :: TestConfiguration [Build 18](http://localhost:8888/viewLog.html?buildId=30&buildTypeId=TestCI_TestConfiguration) outcome was **SUCCESS**
Summary: Running Build time: 00:00:02


>  > Buhu. You broke it. It was working before. Maybe there was a temporary
>  > problem due to some syntax change of the json file changes (bad pdbe). I
>  > will try to fix it again....
>  >
>
> That's strange - it was broken before completely, now it downloads the
> files but doesn't show the progress bars.

You recall what was broken? It was working for me previously. Maybe not
later any more due to a syntax change in the json file (actually only
getting SF file) but coordinates were still there.

B


Why not ask jolokia to have an option to not return stacktrace.

@rhuss is there any option in jolokia we can turn on to not include stacktraces in responses?


Autoloads should be sorted alphabetically.


@kemiKondratenko, ні не сарказм. так як ти зробив можна, але це більше звернень до БД і це не ок


I'm afraid id is not so easy... See method `ApplyChanges` on line 129:

```C#
// Exit if item is not deleted, parent is deleted, and not a M-1 relation
if (parent != null
    && item.TrackingState != TrackingState.Deleted
    && parent.TrackingState == TrackingState.Deleted
    && !context.IsRelatedProperty(parent.GetType(),
    propertyName, RelationshipType.ManyToOne))
{
    ChangeStateByInterceptors(context, item, parent, propertyName, interceptors);
    return;
}
```

In this case there is no `context.Entry(item).State = state` and still I want to call the interceptors. So I cannot use `SetEntityState` from your [comment] (https://github.com/luboshl/trackable-entities/commit/99ac84896ca29c029a2c9a6ec8734847e34f125f#commitcomment-15628385). But maybe it can be refactored. Everywhere where we call `context.Entry(item).State = state` we will use this new method and on ohers (like on line 128) we will use `ChangeStateByInterceptors`. I refactor them that `SetEntityState` will call `ChangeStateByInterceptors`.


Mmmhh ... Stai diventando troppo bravo...
ACK


```vim
let g:neosnippet#completed_pairs= {}
let g:neosnippet#completed_pairs.java = {'(' : ')'}
```


Hi @xpatient,

Please post your question in the issues section.
https://github.com/medyo/Fancybuttons/issues

Thanks


yes, of course ;-). It's a query parameter `includeStackTrace=false`. See also the [documentation](https://jolokia.org/reference/html/protocol.html#processing-parameters).


The file is project info


wenn man aus versehen ein get raw ohne einen Wert abschickt, kommt ein no answer, könntest Du das bitte auch noch abfangen?


btw, this can be added to the agent options when starting so its global for every request.


Ah thanks yeah we need to set this globally so anyone cannot tamper with the url, and set it to true, eg if its set global, then any end user should *NOT* be able to override the value, as server side stacktraces is an attack vector.

@rhuss so if we turn this off on the server side in the agent, then we cannot turn it back on in the url by setting a query parameter - is that right?


Maybe we should use some custom Exception type. `ArgumentException` should be used, if one argument is wrong. Here we don't know, which argument exactly is wrong.

Also we need to throw an exception instead of returning null in the beginning of the method:
```C#
// Get navigation property
var edmEntityType = dbContext.GetEdmSpaceType(entityType);
if (edmEntityType == null)
  throw SomeException(...)
var navProp = edmEntityType.NavigationProperties
    .SingleOrDefault(p => p.Name == propertyName);
if (navProp == null)
  throw SomeException(...)
```


have to check that, but I'm afraid that the global config only serves as a default.


That was quick :smiley:
Will take a look asap. Thx !


Is it actually possible for a method to not respond to `visibility=`? It doesn't seem like that should ever happen.


Yeah just found that detail in the docs at table 3.1
https://jolokia.org/reference/html/agents.html#war-agent-installation

Wonder if we could add a new option (with a good name) maybe
`neverIncludeStacktTrace` or what a good name can be. That can overrule them all


It makes a whole lot of sense, and it has been implemented.


I swear I really can spell...


This needs to happen in the controller, because record selection happens a level earlier.


No sir!
Set your system date to the December of a year, next try it: `chance.exp_month({future: true})`.


nm, it was committed 15 days ago, was looking at wrong thing i guess