Can we extract the necessary properties from `MouseEvent` and store those instead of the `MouseEvent` itself so it can get recycled/garbage-collected/whatever the system wants to do with it under the hood?


rax retry tests


@vladikoff: might be worth making a new patch release from the changes so far.


@Enigmatologist You could also consider storing the index with the task itself in Firebase. Although, even as is, I don't believe this new order will persist or be reflected upon a page refresh.

By storing the index in the database you could do something like this: http://stackoverflow.com/questions/19844359/how-do-i-get-angularfire-objects-to-orderby-with-ngrepeat

Of course, you'll have to write code to maintain the order field too, but that's basically what you're doing here more or less anyway.


`@conn.exec( "SELECT * FROM contacts" )` already returns a list of rows, so creating another array `all_contacts` and pushing the results into it feels redundant. You could probably get away just with something like:

```
def read_contacts
  @conn.exec( "SELECT * FROM contacts" )
end
```


init


@vladikoff: it seems you missed the tag here.


`const char *` instead of `std::string` ? Questionable as its not homogeneous with the rest of the API.


Change to:
```
// TODO(atotic): ...
```


I think that this is where things can get expensive. I think that if there is a previous `Range` underlined by the last move, we should not call `_getSuggestion()` again until the mouse leaves that `Range`.

We may also want to use `debounce()`.


I like how clean your Contact class is and how you've separated the SQL calls out into your ContactDB class! Good work!


I've not added this resource to the repository. 8Mo font, just for tests sake ? One just need to get some package or a `locate` to find it.


Yeah, that was a bug ...


Testing non 2^n textures.


If you're looking for a challenge, can you think of a way of writing this method so that you don't have to update all the fields at once? For example, maybe you only want to update the `name` attribute.

Hint: one way to approach this is to write a few snippets of SQL and then to combine them based on which attributes you're updating.


Oops, never mind this commit...


Initial Commit


This whole initialisation step might seem heavy. In fact, it's the less painful choice to make.


Regarding this discussion about idempotency you may want to consider saying "...should have the same effect.".  The risk of saying "result" is that people assume it means that if the request is issued twice then the same response must be returned.  In the case of DELETE the first response may be 200/204 and the second might be 404.  This is OK, because the effect on the system is identical whether the request was issued once or twice.


> Looks good to me. I'll keep it as a reference on how to add GUI elements :)

@akien-mga Definitely something you could have done. :)


s/wich/which


Indeed, the API result format is wrong. I just pushed a slightly different fix, see commit #958b18e.

I'm more reserved about the introduction of a context-dependant format change in the method return value, tough (a list or an arrayref), as this kind of smartness easily turn into a subtile trap for the unexpecting user. Is there any valid use case for this ?


Sounds good


ksu does not do this anymore, as of krb5 1.13.  (Note that the default_ccname can be set in krb5.conf, including several variable substitutions.)

See https://github.com/krb5/krb5/commit/3a456898af626dcab4e1ab0749ca2ccb9ad6162b


Well, the second hunk served to test the first :-)


$index does not exists.


I realized that starting the core could trigger a lot of class loading, and that could slow down the UI initial show. The difference between this and what we have in master, is that we only do lazy load of the core/engine stuff and the UI is loaded in the main UI thread. In master (with swing), the UI is loaded together with core during the process of creating all components.

Let's see if we can keep 100% this idea.


Renamed the [PortalID ] column.
Tested export of Data-tier Application on SQL 2012 prior the change (failed to export) and after the change (export successful) to confirm that issue is fixed.


why that?


why is a 1d array not enough?


No, this will have to wait '`-'`


Something I tried to manage as working but did not finish


No to work in fxml, but with the binding mechanism of jfx. Still figuring out how to handle the transfers UI update.


Good catches

I'll have to think a bit more about how to disambiguate `Range` and `Array<Range>`


I wonder then if there's a way to know for certain that the main window and its main components have been loaded, in order to then invoke the core/engine class loading afterwards, but to truly do it, only after the window has been rendered.


No worries, Taylor fixed it with [this commit](https://github.com/laravel/framework/commit/f25c5eced340d842294cce69301fd89e2d642ace)


SQL is very efficient at finding a row in your database with a particular ID. Can you think of a way to refactor this code so looking for the contact that matches the supplied ID happens in SQL rather than in ruby?

Hint: you may have to write some more generalized method for DB lookups, or alternatively separate methods, since you'll probably want to do a query like `SELECT * FROM contacts WHERE id = X`


Now it should only give any id to the products. I upload my generator you can choose.


> Multiple properties animated together are known as a multiple property animation.

I'm bad at grammar sometimes


Well, we only need the concept of "services.Loader" for lazy async load, but you can change the name if you want. It's the services loader, but in the UI (not a UI loader).


Agreed but I was under the impression that this was just a proof-of-concept and we should confirm with Atom that this is a good way to go before implementing these types of optimizations


This is normally used to detect if the endpoint is a valid v3 endpoint. Have we decided that the check for .json is enough? This should probably just return null and suppress the errors since reading the index is basically IsV3().


If you find general use for these, we could iron it out and make something a bit more robust.


Good point


I'd most def. separate them, open up to the possibility of an independent core loader


because it's more flexible at low cost?


I don't know but I think I'm more comfortable in splitting this in `d, err := ...` and the condition below (although we don't care about d if `err != nil`), more readable maybe, fell free to skip this comment btw, just my brain


I believe `metaKey` does refer to the cmd key.


Sounds good


Nice


@chatelak Why is only the first constant species used to recalculate the volume? Also, should the recalculation of the volume happen before the residual was calculated?


separate what? there is no "ui.Loader"


Yep, I'll change to a `Set`


How is it used? in sklearn it will be re-converted in to an array anyways at some point. I think we should not do this unless there is some clear reason to do this. Actually it is more exapansive as you will create a new object, the array slice / index returns a view.


Ok


... and here you reconvert i to an array?


`src/main/java/com/frostwire/ui/Loader.java` ?


:+1: For fixing indentation


That's for load the core services (inside the UI). At some point UI and core will touch each other :) no magic here.


Ah, thank you for the note. This was developed on rhel6 against krb5 1.10. Perhaps changing default_ccname would be a better approach here.


but why


I tested it and I think @marcelinkaaa should take a look at it.
![screen shot 2015-05-12 at 6 09 12 pm](https://cloud.githubusercontent.com/assets/143813/7599663/03cceac0-f8d2-11e4-814a-bbc1f26d9ebb.png)


I couldn't find anything about this, or perhaps[I did find something related](https://www.typesafe.com/)?


`can_train_footman?` returns a boolean, so you could also write this as:

```
return nil unless can_train_footman?
```

Or

```
return nil if !can_train_footman?
```


What is this? Is it meant to return a reference to volume 1 or to the current volume? If first then I suggest a new suffix: "CURRENTVOLUME". There is no way to get the information about currently selected volume. If second then I suggest renaming it to "CURRENTVOLUME" to make it less ambiguous.


:+1: Nice refactor!


Fix indentation.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.132](http://teamcity.monogame.net/viewLog.html?buildId=7281&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:23:27


E:\ProgramacaoAvancada
git init
git add .
git commit -m "first commit"
git remote add origin https://github.com/flaviorsm/ProgramacaoAvancadaGit.git
git push -u origin master


I think she's been playing with it after we did the `processResources` invocation.


Ease of development, nobody actually ever used FML itself. Lots of reasons.


Are the ```tests/unit/MY_Doctrine/``` files actually used?  All references seem to have been removed later.


+.. Warning:: cualquier cambio en los datos introducidos recrea los certificados SSL y reconfigura todo el sistema. El proceso podría tomar un tiempo.


When creating a background session configuration in a test target, it throws an assertion saying you just have a bundle id. The test target does not have a bundle id.


excellent!!


wrong copyright date, it's 2015 now :-)


Did someone call me?

Something in need of Polishing, huh.
POLISHing. I'll see myself out.


Good catch. I meant to make that respondsToSelector. Will update.


If you want to check if the enemy is an instance of the SeigeEngine class you would need to fix the first part of this expression.


Ok, I can remove it. There's a bug if you pass a list because it tries X.shape[0] == y.shape[0], but I can just use len(y) if you prefer.


Ya I can add that


Do all the classes in *.impl package have to be public?


Yes I did... looks great.


Super Commit, danke <3 :dancers:


I'll admit to being quite a novice when it comes to this NSinvocation stuff. I'll take another pass.


Just write the least surprising code :)


@shlevy propagatedBuildInputs should be used in case libcurl exposes those
libs in headers. If not, then they are not propagatedBuildInputs.

On Tue, May 12, 2015 at 7:50 PM, Shea Levy <notifications@github.com> wrote:

> Anyway, for now I've merged #7826
> <https://github.com/NixOS/nixpkgs/pull/7826> since that was the only
> setup where nothing was broken. I really think you shouldn't reintroduce
> this stuff at all, but if you do please do test it on darwin first.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/26c37ded73ce084275d748be26887cf4023c91d6#commitcomment-11160250>
> .
>

--
NixOS Linux <http://nixos.org>


@dimitrinicolas Bon bah en fait Louis m'a fait l'icône :laughing:


This is core to the typesafe (and big scala ecosystem ) stack https://github.com/typesafehub/config

It is a pure-java framework. small, light, simple and json friendly. Take a look at it. I'm still not using it (need to decide with you).


Cool.  My immediate use case here was to avoid `install_github` / `installGithub.r` in Dockerfiles, which seems to be throwing `403` errors at me often enough to disrupt my builds.  I was thinking (perhaps mistakenly) that calling `install_packages` on drat/gh-pages would be a bit more reliable.

Also it's just a better option than `install_github` for users, since it can handle other non-cran dependencies, (provided they're on the drat repo) and it plays nicely with `update.packages()`


Merci ;) j'ai vu sur le site qui sont en maintenance aussi


Wrong assert failure comment, updated from 5 to 6 but comment still says 5.


@lethalman If libraries that link to -lcurl also have to link to -lidn, then presumably some sort of interface is exposed.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.133](http://teamcity.monogame.net/viewLog.html?buildId=7295&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Anyway, since when is that the standard?


Whoops, I ended up converting some tabs in unrelated code in TranslationHelper


Oh, you might also have noticed I toggled the `devtools::build` `quiet` mode off, since it's really annoying when the build fails and I don't get an error message.


<?php echo date('Y'); ?>


But shall we, or shall we not accept y as a list? If so we need to test for it, don't we?


@abenkovskii the goal was to have `:VOLUME` represent the drive that was attached to the executing processor. If we want `:CURRENTVOLUME` somewhere i dont think it belongs in `CORE:` as `CORE:` is the executing processor


<?=date('Y');?> would also be ok. And yes: "<?=" is allowed, "<?" is not! ;-)


sorry -- my version of TFTD didn't have it.  I'll add it to the list.


@chriscool, yes. I have recommitted this.


is fine with me to accept it as list sure.


@beriberikix Sorry, we are working on a few issues which should be fixed in 1.5.0 milestone https://github.com/platformio/platformio/milestones/1.5.0

If you don't wait for the final release, you can try development snapshot http://docs.platformio.org/en/latest/installation.html#development-version

P.S: I hope, I'll release it this week (maybe Saturday/Sunday).


I was more considered about the conversion to list which seems a bit unnecessary.


:+1: Nice refactor! These should definitely be attr_readers


system/extras/ext4_utils/make_ext4fs.c:229: error: undefined reference to 'LZ4_compressBound'
system/extras/ext4_utils/make_ext4fs.c:252: error: undefined reference to 'LZ4_compressHC'
?


it's just to in the tests to make sure it's working fine.


:+1: These are much better names for the variables.


@zolza jeśli podajesz adres linku, to używaj też protokołu w adresie np. `https://www.linkedin.com/in/kingakoziol`


Yep, you totally right.
Thank you a lot, @eriksencosta.


:+1: Good use of super


I've been thinking about this (in general but it applies here) that a log-if-error is the Right Thing to do in that case; in general you don't want all the output, but if the build does fail then output should be produced.  This could be a reasonable motivating case.


OK, used the date function to get parts of the date.
I hope that's what you meant...


You need to add `Method::Generate::Accessor` too, to fix errors from setters as well as the constructor.


the reason it is currently not implemented is that the volumemanager doesnt actually know which volume is the processor's


Actually I think the code needs to be thread-safe so would require an @synchronized block if it reuses the invocation object. Your current implementation is thread safe. I don't know which version would be better/faster without testing.


If it's not injected, it is very hard (or impossible) to ever test it, hence all instances need to be created through DI or factories.


Fixed this in 5e189c6f9347dab9aa8fada32117b6c74780dc06.


I systematically convert y to an array.

there are two y: at the fit, and at the score.

the y.tolist() is in the test to check that gat works with lists.


Popraw
```javascript
var list = new Array();
```
na

```javascript
var list = [];
```
A do tablicy `list` pushnij po prostu cały obiekt `place`


In which case you can just return a `JsonResponse` manually. The response is only converted if it is an array.


There is a way to have active and inactive periods on the same user. This is misleading because there was a period that Ben was not covered under an agreement. I'm not sure the exact syntax but there is examples of this arleady in people.yaml. Please figure out how to do this (either by looking through people.yaml or talking with Ned)


ahhhh sorry, now I got it. Fine with me, don't change it.


I think we shouldn't have state in controller.


Plus I strongly suspect our interns are staying longer than the end of July - last year most stayed to mid-late August.


Replace this line with:

```
---
layout: post
title: Prism break, cuttlefish style
---
```


veri nice, getting rid of the C that is


Interns should have a comment saying they're interns, and an expiration date.


## This is a markdown file


Shouldn't beginYear be required? What happens if it is missing?


Spaces


Relative links break in individual posts. :(

Replace image link with: `({{ site.url }}/images/2015-05-16-prism-break.jpg)`

This SHOULD work!


there is no need to run browser.sleep here nor in other tests.

generally whenever there is a sleep there is always a differnt approach that permits you to watch a promise. avoiding a sleep is important in order to not have bugs related on the progessing-time.

for all the old tests i've already fixed it in devel; look at them or ask me for specific cases.

tomorrow i will look a this.


Ah, I finally remember why I did this: DiskBlockObjectWriter won't update its `bytesWritten` metrics _during_ writes unless we also increment its records written counter.  It appears that the current semantics of recordWritten() are that it's set once the record hits disk, not when the record enters the sort code.


I just made a comment here
https://github.com/official-stockfish/Stockfish/pull/348


Agreed.  If we want access to read which volume is the current volume, it should go in a different struct or bound variable.  I'm not sure what would be the best choice would be.


>  also noted that you could just _.toArray() a NodeList before using it with an array

Right but there's really no required need to convert the array-like object. It doesn't access any array methods in the processing so could be handled without the need to convert and without special casing nodelists explicitly.


> So when would you ever emit the dot with the member name here, or the brackets with the member name below?

I would emit it every time the accessor is defined. So all this cases:

```typescript
new object.accessor(..args)// accessor is an identifier
new object[1](..args)// accessor is a numeric literal
new object[foo.bar](..args)// accessor is a property access expression
```

And I will emit the original node expression if the accessor is undefined:
```typescript
new object(...arg);
```

>I wonder if it makes more sense to just emit the whole expression when you emit the target the first time, and not do the special casing for emitCallTarget. Seems like you do not care about the target for a new expression.

I'm not sure I understand. Can you explain in code?


1836664-ReturnWaterTempReset (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1653 of 1653 tests passed)

![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)


Not sure about how this gonna end. Will it be like the Sprengine ?


Yup, log-on-error sounds good.  otherwise one really just wants a progress
indicator, which the current log system you have does rather nicely.

On Tue, May 12, 2015 at 3:21 PM Rich FitzJohn <notifications@github.com>
wrote:

> I've been thinking about this (in general but it applies here) that a
> log-if-error is the Right Thing to do in that case; in general you don't
> want all the output, but if the build does fail then output should be
> produced. This could be a reasonable motivating case.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cboettig/drat/commit/423cb6adb486f1b5d0e00b4c29262666d279fdc1#commitcomment-11165223>
> .
>


This is now a **global**!!!!
You should have received a warning message for this.  Make sure to check the log when developing.


I think specifying the actual hex value would be better here.
Same for the white values. Using `#fff;` would be more "Bootstrappy".


Added website address


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Since this is static you can just do:
```Lua
local loss_prob = {
	["default:cobble"] = 3,
	-- ...
}
```


Indeed!

https://github.com/dart-lang/linter/pull/95


Also a **global**!


@retep998, interested in accepting the `DECLARE_INTERFACE_` macro upstream? If so I'll work up a cleaned-up pull request based on master. The macro makes defining COM interfaces easier and makes using COM instances less tedious. I'll likely add a `Deref` impl for `$interface` so that Rust code can do `instance.Method(...)` instead of `((*(*instance).lpVtbl).Method)(&mut *instance, ...)`


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.133](http://teamcity.monogame.net/viewLog.html?buildId=7295&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:20:28


Maybe school-year-info or something similar would be more descriptive class name. It would also make it harder to confuse the school-timeline-info and school-timeinfo elements.


To be tested, on next week so.


@nathansobo's idea.


The thing is that I really really don't want to have (even indirect - through interfaces or so) interdependencies.
Each modifications of a text area should be done through the Dumb Font Engine (I already ear talking about heresy). Why ? To avoid messaging mechanism and so. With strong typing, we can easily achieve this.
Of course, this means one big API on engine side.


message.data.soundPath here


this is copied from the option parsing code we are using for ceph.conf……

do you mean we dont need to support using interger as bool?


@TheEnigmaBlade this is the best I could come up with, without rewritting usernotes entirely.  Thoughts?


Shape of the yearsActive object could be specified here so it's clear for anyone using the SchoolTimelineInfo component.


I love this consolidated effort 💩


Point taken — I do agree this may be useful sometimes.
Did you use UdpContext::connect directly or did you rely on the old behaviour of WiFiUdp::beginPacket?


I haven't tested it, but that's probably a good place to put a version warning.


Release this fix please


LGTM. As discussed we should later share the validation code we wrote for ARM.


@bartvde sharing since I moved the StoryPin class to the core library and use it in an angular module.

I have some minor wrapping up to do here - thankfully this doesn't conflict with your efforts :smile:


FacilityTabGridView (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


LGTM


Tell me about it !
There's also one REALLY BIG stuff here : We're only talking 'bout texts. But what about that time we'll want to integrate this to a GUI ? How will we manage layering with heterogeneous graphic objects ? How will we share "virtual viewports" (if this kind of sharing is needed/wanted) ?


Change this to "rehtori". The previous boxes had text in lowercase so I guess we should stay consistent with these boxes as well. Later we can make text in all boxes capitalized if we like it better.


HI @Gray, the cancelHoldPulse was used to stop pulse event on scroller. When itreaches to the end it is change paging control status as disabled and move focus to the other side. We canceled hold pulse event in this case, because we dont want to continue scrolling to opposite direction.


No, not necessarily. It's only needed for curl, not other libs linking to
curl. -lfoo is needed because of the .la file, in fact I even believe
because of rpath that those libs are not necessary in .la at all, will have
to study this case.


Change this to "sijainti".


Change this to "arkisto".


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.134](http://teamcity.monogame.net/viewLog.html?buildId=7319&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


FacilityTabGridView (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2178 of 2200 tests passed)

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


rax retry tests


Why is this `false`?  Shouldn't the default be the best option?  Just fall back to the included mini-GMP.  Also, use `find_package_handle_standard_args()`; see the other modules.


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Where does this "180" come from?


It may be wise to also remove `label` and avoid that potential conflict as well.


:clap:


Hmm, why do we need the .la files at all? Shouldn't everything be using dynamic linking?


Summary of Traders and diplomats by E. Preeg.Rmd

I should have said over-production and each other's domestic production


How rude of it!  Will fix this evening.


Thanks for report


1836664-ReturnWaterTempReset (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1659 of 1659 tests passed)

![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)


Not his is wrong. This is meant to be an array of each version and their dependencies, and is linked to the folder of the docs. This shouldn't be changed, but a new version should be appended at the end of the array.


Actually, that check would miss objects which are created as `Object.create(null)`. So the (final, I hope :smile:) correct check is:

```
do {
  Object.getOwnPropertyNames(object).forEach(collectProp);
  object = Object.getPrototypeOf(object);
} while (object && object !== Object.prototype);
```

Mind updating the PR, that we can merge it?


You can't yield the exception


Same as above


It didn't tell anywhere it should have been an array, and the property is also `version` instead of `versions` :)


最高です


@shlevy afaik debian removes .la files in fact from all packages... I'm
just not knowledged enough, but for sure those libs shouldn't be in
propagatedBuildInputs. They are in propagatedBuildInputs just because it's
easier to make libtool happy. I'd like to spend some time in experimenting
the removal of .la files.

However, maybe .la files are helpful for .dll for windows? Don't know.


This is not how `find_package_handle_standard_args` is supposed to work.  Use something like this:
```CMake
option(ENABLE_SYSTEM_GMP "Use GMP from system" TRUE)
mark_as_advanced(GMP_LIBRARY GMP_INCLUDE_DIR)
set(USE_SYSTEM_GMP FALSE)

include(FindPackageHandleStandardArgs)

if(ENABLE_SYSTEM_GMP)
	find_library(GMP_LIBRARY NAMES libgmp.so)
	find_path(GMP_INCLUDE_DIR NAMES gmp.h)

	find_package_handle_standard_args(System_GMP DEFAULT_MSG GMP_LIBRARY GMP_INCLUDE_DIR)
else()
	message (STATUS "System GMP disabled! (ENABLE_SYSTEM_GMP=0)")
endif()

if(NOT SYSTEM_GMP_FOUND)
	set(GMP_INCLUDE_DIR ${PROJECT_SOURCE_DIR}/gmp)
	set(GMP_LIBRARY gmp)
	find_package_handle_standard_args(MiniGMP DEFAULT_MSG GMP_LIBRARY GMP_INCLUDE_DIR)
	add_subdirectory(gmp)
endif()
```


Comment impair. Le système de message semble avoir substitué Cyberden et "Cyberben", y compris moi dans ce fil de conversation :)

Utilisation de Google Translate pour écrire ce message en français

Cool!

Cheers!


Which is why I said I would do it. And yeah because it's an array of objects and one of the properties of that object is version.


So no support for `@first` and others. :-(


4906-Documentation-Issues (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1651 of 1651 tests passed)

![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)


Hamachi huh? Nice touch. Will we be using Hamachi in our scripting course?


My vote is definitely for removing the .la files and seeing what, if anything, breaks. I don't know much about the windows case, @edolstra ?


The problem is that libcurl.la has -lfoo without -L, now every dep of
libcurl fails to build. That's because libcurl and other similar libs that
have a customized build system don't care about -L, thus producing broken
.la files and .pc files for us.

Regular libs that use pkgconfig without customizing anything in the build
system work because -L is in .pc files and gets propagated correctly for
private libs.


Why do you think it is unneeded? A transaction can become invalid (e.g. if it is double-spent), in that case it won't be present neither in block nor in mempool.

Chromanode will give you {"status":"fail","data":{"type":"TxNotFound"}} and you should return None, False
Which is interpreted as status 'invalid' by TxDb.


Even many PC files seem to be broken, just not as badly as .la files. I
still think we could do a good job patching those so that they are correct.

On Tue, May 12, 2015, 15:53 lethalman <notifications@github.com> wrote:

> The problem is that libcurl.la has -lfoo without -L, now every dep of
> libcurl fails to build. That's because libcurl and other similar libs that
> have a customized build system don't care about -L, thus producing broken
> .la files and .pc files for us.
>
> Regular libs that use pkgconfig without customizing anything in the build
> system work because -L is in .pc files and gets propagated correctly for
> private libs.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/26c37ded73ce084275d748be26887cf4023c91d6#commitcomment-11165651>
> .
>


[![Book Cover with Virupa and Guru with Amitabhas](http://api.artsmia.org/images/31188/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=31188)


4906-Documentation-Issues (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1140 of 1140 tests passed)

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


You'll probably want to send your requests to https://api.npmjs.org/ directly, since it is the source of npm-stat.com. npm-stat.com proxies all requests for http://npm-stat.com/downloads/ and its children to https://api.npmjs.org/ (for avoiding foreign domain problems). The path at api.npmjs.org is the same, e.g. https://api.npmjs.org/downloads/range/2013-05-13:2015-05-13/clone


Oops, for some reason the file names did not stay lower-case. I just think we should make them lower-case before we turn them in, since Prof. Zander seems to prefer that convention, and we have to play by her rules.


I stink finally compile x64 release mode, against, not LLVM release, I had 6 strange error, I think the LLVM Submodule is not yet ready to be compiled with VS 2015.

And the problem also, I do not understand why, Visual stidio does not detect the folder in sdk 10.0.10075.0 version is the last version to date, how to tell him that this version exists?
![capture d ecran 2015-05-13 00 58 28](https://cloud.githubusercontent.com/assets/5261759/7600458/7c0471fa-f90b-11e4-8bca-b5b585a29f9a.png)
![capture d ecran 2015-05-13 00 59 04](https://cloud.githubusercontent.com/assets/5261759/7600459/8061489a-f90b-11e4-9e25-4ac55ead9a6c.png)


I've started on [bug 1154349](https://bugzilla.mozilla.org/show_bug.cgi?id=1154349), where errors change from text generated at the time of error detection to error slugs and detail dictionaries. I think you'll like it better than these tests.


why is this a coroutine?


This doesn't need to be a coroutine either


we can return the null cache impl from my understanding - what so ever this class has to die


All documentation was pulled from the published MSDN API docs for the Windows APIs.  I fixed quite a few typos, but due to sheer volume, I'm sure I missed some.


I think the variable name could already now be more specific even if it's at this point the only variable in the UIStore. Something like _schoolTimelineYear.


@mfojtik This may not be needed as by default, the value of ETCD_INITIAL_CLUSTER_STATE is new.


I'm sorry, but maybe you could leave this behavior in? Functionally everything was working properly, it was only visual glitch and only with tangleberry, which is not a big deal. And it's a very nice feature to be able to have more than one function with one flower. :)


Forgot to remove it when remove future thing. Updated


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.134](http://teamcity.monogame.net/viewLog.html?buildId=7319&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:20:39


Same to this method name.


1836664-ReturnWaterTempReset (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1659 of 1659 tests passed)

![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)


Yes, this is the current MSDN documentation. :)


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.135](http://teamcity.monogame.net/viewLog.html?buildId=7323&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Thanks Nell!

On Tue, May 12, 2015 at 3:47 PM, Nell Shamrell-Harrington <
notifications@github.com> wrote:

> How rude of it! Will fix this evening.
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/OperationCode/operationcode/commit/87f072cef1717558d00e6ede71bd6fa85469d6b4#commitcomment-11165580>
> .
>

--
David
http://davidmolina.github.io/ | t. @davidcmolina
<https://twitter.com/davidcmolina>


Why not? Doing this stops us just ignoring warnings.


I was primarily getting rid of path modification to use $TOP_DIR while running collector and query engine. Hard coding to production in the path was accidental and my mistake. I will revert to $TARGET. Thanks for catching it.


No, this isn't good. This doesn't make a graceful fallback to bundled gmp, but makes an error message instead.


Does `getCommentIds()` work properly when `maxComments === 'all'`?


Apparently it wants a host app on iOS 7 but not on iOS 8.


jobIds will print the object reference here. Can you make it to use Arrays.toString or something similar.


You would be right, but in this case the classes involved are not static. They're private ones that don't need any special constructor (just a simple initialization of their member vars to some meaningful state).

But now that you made me look into this... I can't find a way to instantiate, f.e., a `Tess2.DictNode` from outside the `Tess2` class, even using `@:privateAccess`/`access()` tricks. I'm not sure, but maybe this was possible before (or my attempts are clearly wrong)?!

Can't be reproduced on try.haxe, as a test for this would require two different files/packages. I'm gonna look further into it and file an issue on the haxe repo if I find something.


All hail the Javascript! Condition from `getCommentIds()` `length > that.maxComments && length > 3` will result into `false`, i.e. any number is less than alpha-string.


Also not sure what can we do with just the jobId, we need the jobId and what the job is about.


nie wiem dlaczego, ale kiedy pushuję cały obiekt place, na finalnej liście widzę kilka takich samych obiektów, a nie różnych. Działa prawidło dla tego przypisywania co jest w kodzie.


:shipit:  after addressing feedback


Thank you for cleaning up after me :)


Well, maybe this is a more proper method then, @AgentRev .
486ee20f014ae8973b60c3950501ef037181b6e9

I wasn't sure if I should use publicVariableServer, considering the function is running on the server.


Hmmm, you could do this then, but it basically just removes some annoying messages then.
```CMake
# ...
if(ENABLE_SYSTEM_GMP)
    find_library(GMP_LIBRARY NAMES libgmp.so)
    find_path(GMP_INCLUDE_DIR NAMES gmp.h)
else()
    message (STATUS "System GMP disabled! (ENABLE_SYSTEM_GMP=0)")
endif()

if(NOT (GMP_LIBRARY AND GMP_INCLUDE_DIR))
    set(GMP_INCLUDE_DIR ${PROJECT_SOURCE_DIR}/gmp)
    set(GMP_LIBRARY gmp)
    add_subdirectory(gmp)
endif()
find_package_handle_standard_args(GMP DEFAULT_MSG GMP_LIBRARY GMP_INCLUDE_DIR)
```


Ok. Your class works. But we can create an IDGenerator-object in our overall-start class for the whole program, so you don't need to create it in the same class. And you don't need the method to be static then. But it's just an idea on my part, when you want to make it this way, i'll agree to that.


lololol "spans spans everywhere"


Yeah, there's clearly a c/p error there.  I'll fix it as soon as I get the chance.


Langauge


@nairdo Want to talk to you about this issue we found. Disabling the button OnClientClick sometimes keeps IE from running the postback. Tried a bunch of fixes and could not get it working. Since it's causing issues in check-in we commented it out for now.


JS! *wincing*
I'd suggest making it more explicit anyway, for the sake of the future contributors.


lol

On Tue, May 12, 2015 at 3:59 PM, celeryellery <notifications@github.com>
wrote:

> Oops, for some reason the file names did not stay lower-case. I just think
> we should make them lower-case before we turn them in, since Prof. Zander
> seems to prefer that convention, and we have to play by her rules.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/celeryellery/CSS-343-Lab-4/commit/890d4d95a38703116b1847b1dd48918a1d028e72#commitcomment-11165736>
> .
>


Deal.


Maybe change `join` name to `compose` because it is the mathematical name for that concept (http://en.wikipedia.org/wiki/Function_composition)


1836664-ReturnWaterTempReset (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Some in-code comment, maybe, as a last resort?


closes #1


Welp, that was fast.


Prawdopodobnie po pushnieciu powinieneś czyścić obiekt place. Jutro
przeanalizuje kod

Sent with AquaMail for Android
http://www.aqua-mail.com

On May 13, 2015 1:09:18 AM Piotr Grobelny <notifications@github.com> wrote:

> nie wiem dlaczego, ale kiedy pushuję cały obiekt place, na finalnej liście
> widzę kilka takich samych obiektów, a nie różnych. Działa prawidło dla tego
> przypisywania co jest w kodzie.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/debian-sh/Placetag/commit/3e45064c836eff2d61c01ecd794fd00f5c741134#commitcomment-11165843


It is way behind. I think I might work on bringing that up to parity after
I do some more test stuff.

On Tue, May 12, 2015 at 5:56 PM, Hans Kristian Rosbach <
notifications@github.com> wrote:

> I agree, Cmake is probably a more suited platform and the amount of
> duplicated effort we'd need like it is today is rediculous, not to mention
> a pain to test thoroughly.
>
> However, my first meeting with Cmake (when updating a MariaDB rpm spec
> file) was not at all an easy task when you have not used it before. So we
> either need really good documentation (INSTALL file for example), or
> possibly a ./configure wrapper that accepts configuration parameters and
> prints out the cmake command needed (thus educating the user at the same
> time).
>
> When we have cmake up to par with configure, we'll drop configure (or
> ideally replace it with a wrapper).
> Until that time, we'll have to maintain both since cmake really is waay
> behind.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Dead2/zlib-ng/commit/7776efc81ba38e4877344ce75174a03b0fdf8057#commitcomment-11149406>
> .
>


The moment the Touhou Wiki had it, I added it.


Should be removed.


I have updated, but I still see the device support error.


@PullMyFinger good to merge!


Yes, the SAM3A, SAM3X, SAM3U and SAM3S all appear to have the same reset controller (all registers and bits are identical). Though, I don't have a SAM3X to test any code on.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.135](http://teamcity.monogame.net/viewLog.html?buildId=7323&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:22:37


```10 < 'all'
false
10 > 'all'
false
10 == 'all'
false```

http://stackoverflow.com/questions/10863092/why-is-string-11-less-than-string-3


Should leave comment as to why we're wrapping it in a block. Also, the comment above this should probably be inside the block by the code outputting it.


Miałeś rację. Teraz działa dobrze.


4906-Documentation-Issues (rraustad) - i386-Windows-7-VisualStudio-12: OK (1651 of 1651 tests passed)

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


Is this project still under active development ? If not why was it abandoned ?


@maxogden :tired_face:


Abdul, I am getting the error "Notice: Undefined property: stdClass::$title in _meedan_inline_thumbnail_bg() (line 477 of /var/www/checkdesk/drupal/sites/all/modules/meedan/meedan_oembed/meedan_oembed.module)." Please make sure to turn on error logging to the page to catch and fix those kind of warnings.


I'll take a PR :]


Add a test validating that the `ConditionalAttributeCollapser` doesn't rewrite attributes with `@@`. If it changes in the future and starts rewriting them that'd be a big nono.


I d'ont mind where it should be. Essential it works anywhere =).
Öhm i hope we can use an existing. But this we can ask on friday or so.


Well... yeah :lollipop: Ultimately, it returns exactly what we need -- false if `maxComments` is set to anything but number. But I agree that it's better to make it human readable.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.136](http://teamcity.monogame.net/viewLog.html?buildId=7337&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


The type signatures changed here to return c_double, making these functions differently-typed based on platform. Does this change leak into any public APIs?


Ah, I see now. I like them, as they help debugging configurations.


4906-Documentation-Issues (rraustad) - Win64-Windows-7-VisualStudio-12: OK (1651 of 1651 tests passed)

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


Oh, actually that's not what's happening here. These double functions are pure additions. Why are these added?


rax retry tests


Nope, doing that breaks other tests.


These are private to this file so can't be used. Seems like this shouldn't have been added.


:exclamation: This should actually always be `10.0`. However, we can remove this line as soon as you no longer intend to support Visual Studio 2010 (which it sounds like you may be doing by moving to .NET 4.5).


yeah, I just was confused by "_any number is less than alpha-string_" and decided to check if it is always like that in JS


lectura análoga por raspberry


Good catch! I will try to append the parenthesis at the end.


This is fine with me.


I'm not very happy with the actual codebase nor with this attempt. I see a few issues discussed here:
- IP argument as a resource, or as a resource list
- Single reporting of all operation progresses, or multiple (see vm.delete)
- Iface/IP mix and legacy code still in

Stick to what you're working on. Don't push hundreds of features at once: we have a tons of operations reported by vm.delete() during vm stop and actual deletion - I think we can be happy with a first version of multiple-argument ip commands that does the same.

Also try to produce small gandi.cli.ip code, please.

We should discuss how to refactor the whole ip/iface mess, including the required work on the API.
The discussed UX has slipped a bit and produces awkward snippets, for example gandi/cli/commands/vm.py:278 (that can be simplified now we moved script in the API)

```python
    if ip and not vlan:
        gandi.echo("--ip can't be used without --vlan.")
        return

    if vlan and not ip_version and script:
        gandi.echo("--script can't be used with a private ip only vm.")
        return

    if not vlan and not ip_version:
        ip_version = 6

    if not ip_version:
        gandi.echo("* Private only ip vm (can't enable emergency web console "
                   'access).')
```


@dylanor Sorry, I was so busy last few days so can not answer earlier. I'm still investigating the problem, will get back in 1 or 2 days.


@jonmeredith Good question about the lack of monitors. This piece of code I'm commenting on is where dead watcher processes get (eventually) removed. When I started writing this PR, I was using monitors, but it ended up being slightly hairy to implement due to some other existing code we have in riak_ensemble_peer that catches DOWN signals and passes them to the storage backend. This approach where we check for dead pids before adding a new one seemed a bit more elegant and efficient anyway, so I ended up going with this instead. Dead watchers will hang around for some period of time until a new watcher is added, but we're guaranteed that we won't have any long-term leaks where we just keep getting more and more dead watchers forever without cleaning up after them. (In the typical case I'm envisioning, we'd have some process running under a supervisor that would restart immediately and re-register as a watcher, and so the stale pid would get noticed and removed right away.)

On the flip side, maybe this is enough of an un-idiomatic pattern that I just shouldn't be doing it. If you've read all that and think I should rewrite it to use monitors instead I could probably be persuaded :)

For your other question: if the watcher wants to actively unregister itself, we'll have to add another function to allow it to deliberately do that. But (for me at least) this code is still useful without that capability, so I wasn't planning on implementing that functionality in the short term. If it's something you'd like to have anyway though, I'd be happy to implement it, since it should be pretty simple and straightforward.


Is this project abandoned ? If so why ? Or is salesforce just so static that it does not require changes ? Any plans for a newer version of JPA support or hibernate ?


Well that answers that.


This is the :hankey:


Either way, I'm still learning and know what to do now


I think this mixes Z_SOLO and your new define in ways that aren't quite correct. I think your new define covers some cases that would normally be compiled in when Z_SOLO isn't defined, which is breaking my test suite.

Is there any reason you couldn't break this into two patches? Patch 1 create the new define, and patch 2 could remove solo...


"be" numbers?


correct. as planed.


Atlas, on the other hand, does care. I am desparate to close #41, which can only be done if both the unit tests (on travis) and the provisioning (on atlas) runs to completion without errors.


GO_CALL_TRAM It seems not working？


But if you want, you can remove them too.


indentation is wrong here.


proccess of code undigerible, almost. Idea stablished, commitment Ready and on.
files: none
thank you so much for the code experience here I am.


Using it here:
https://github.com/jbondc/TypeScript/commit/d9d2f793120fcdf032b3855739286e8a6cb58ec2#diff-9ce3660cf40b1a480fc1f5107e0ef3d1R34


The research I'm citing here can be found:
[](http://lsr.nellco.org/cgi/viewcontent.cgi?article=1476&context=nyu_plltwp)


1836664-ReturnWaterTempReset (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (627 of 627 tests passed)

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


This is kind of silly... you already have the element of topcolor pointing at header two lines above.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.136](http://teamcity.monogame.net/viewLog.html?buildId=7337&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:20:56


Can you test against https://github.com/terrellt/active_fedora/tree/feature/UpdateAT ?


See https://github.com/projecthydra/active_fedora/pull/798


I would suggest using clearer names, such as `dupDataPropertyRegexp` instead of `regx1`. Same applies for subsequent regexp instances named with a generic name.


How do we want to organize this part? My thinking is something like:

* Choosing the right graph type.
* Choosing the right colors.
* Choosing more than one graph (both small multiples, and maybe a 3-visual sequence to tell a more complex story?)

I'm already a bit worried this would make for too much day one content, but probably best to over stuff it and edit it down and move things once all ideas are down.


Why was this line removed?


Good catch, though this is a typo which was copy-pasted from other parameter descriptions...


hmm……should not convert einval to bool……will fix


Fixed


keze@Asus-x99-Series:~/Descargas/XML-JSON-XML-master$ valgrind ./main -json2xml archivo.json faf
==3866== Memcheck, a memory error detector
==3866== Copyright (C) 2002-2013, and GNU GPL'd, by Julian Seward et al.
==3866== Using Valgrind-3.10.1 and LibVEX; rerun with -h for copyright info
==3866== Command: ./main -json2xml archivo.json faf
==3866==
==3866==
==3866== HEAP SUMMARY:
==3866==     in use at exit: 0 bytes in 0 blocks
==3866==   total heap usage: 80 allocs, 80 frees, 11,851 bytes allocated
==3866==
==3866== All heap blocks were freed -- no leaks are possible
==3866==
==3866== For counts of detected and suppressed errors, rerun with: -v
==3866== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)
keze@Asus-x99-Series:~/Descargas/XML-JSON-XML-master$ valgrind ./main -xml2json archivo.xml faf
==3867== Memcheck, a memory error detector
==3867== Copyright (C) 2002-2013, and GNU GPL'd, by Julian Seward et al.
==3867== Using Valgrind-3.10.1 and LibVEX; rerun with -h for copyright info
==3867== Command: ./main -xml2json archivo.xml faf
==3867==
==3867==
==3867== HEAP SUMMARY:
==3867==     in use at exit: 552 bytes in 1 blocks
==3867==   total heap usage: 72 allocs, 71 frees, 10,747 bytes allocated
==3867==
==3867== LEAK SUMMARY:
==3867==    definitely lost: 0 bytes in 0 blocks
==3867==    indirectly lost: 0 bytes in 0 blocks
==3867==      possibly lost: 0 bytes in 0 blocks
==3867==    still reachable: 552 bytes in 1 blocks
==3867==         suppressed: 0 bytes in 0 blocks
==3867== Rerun with --leak-check=full to see details of leaked memory
==3867==
==3867== For counts of detected and suppressed errors, rerun with: -v
==3867== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)
keze@Asus-x99-Series:~/Descargas/XML-JSON-XML-master$


first:
I change the frameworks/av code souce in .repo/manifests,
xy@debian:/works/cyanogenmod/cm11_armv6/.repo/manifests$ git diff
diff --git a/default.xml b/default.xml
index c9a9c14..7038917 100644
--- a/default.xml
+++ b/default.xml
@@ -301,7 +301,7 @@
   <project path="external/zip" name="CyanogenMod/android_external_zip" />
   <project path="external/zlib" name="CyanogenMod/android_external_zlib" groups="pdk" />
   <project path="external/zxing" name="CyanogenMod/android_external_zxing" />
-  <project path="frameworks/av" name="androidarmv6/android_frameworks_av" groups="pdk" />
+  <project path="frameworks/av" name="CyanogenMod/android_frameworks_av" groups="pdk" />
   <project path="frameworks/base" name="androidarmv6/android_frameworks_base" />
   <project path="frameworks/compile/libbcc" name="androidarmv6/android_frameworks_compile_libbcc" groups="pdk" />
   <project path="frameworks/compile/mclinker" name="CyanogenMod/android_frameworks_compile_mclinker" groups="pdk" />

then I block some code in frameworks/av:xy@debian:/works/cyanogenmod/cm11_armv6/frameworks/av$ git diff
diff --git a/media/libstagefright/Utils.cpp b/media/libstagefright/Utils.cpp
index d43f5af..349edc2 100644
--- a/media/libstagefright/Utils.cpp
+++ b/media/libstagefright/Utils.cpp
@@ -531,6 +531,7 @@ status_t sendMetaDataToHal(sp<MediaPlayerBase::AudioSink>& sink,
     if (meta->findInt32(kKeyIsADTS, &isADTS)) {
         param.addInt(String8(AUDIO_OFFLOAD_CODEC_FORMAT), 0x02 /*SND_AUDIOSTREAMFORMAT_MP4ADTS*/);
     }
+/*
 #ifdef ENABLE_AV_ENHANCEMENTS
     if (meta->findInt32(kKeyMinBlkSize, &minBlkSize)) {
         param.addInt(String8(AUDIO_OFFLOAD_CODEC_FLAC_MIN_BLK_SIZE), minBlkSize);
@@ -545,6 +546,7 @@ status_t sendMetaDataToHal(sp<MediaPlayerBase::AudioSink>& sink,
         param.addInt(String8(AUDIO_OFFLOAD_CODEC_FLAC_MAX_FRAME_SIZE), maxFrmSize);
     }
 #endif
+*/

     ALOGV("sendMetaDataToHal: bitRate %d, sampleRate %d, chanMask %d,"
           "delaySample %d, paddingSample %d", bitRate, sampleRate,


You do emit those constructs, yes. But you only emit them inside emitCallTargetAndAccessor. By the time you get out here to emitNewExpression, you have an identifier instead (namely the temp that you created for the target and accessor). This is because you only evaluate such expressions once. So the dot and bracket code would be necessary in emitCallTargetAndAccessor, but not in emitNewExpression.

What I am trying to say is that you don't have to call emitCallTargetAndAccessor. Instead, you can just call emitCallTarget on node.expression (the same place you call emitCallTargetAndAccessor now), and then get the temp that results from that. Then when you are ready to emit the thisArg, you can just emit the temp that you got from emitCallTarget.

This also means that emitCallTargetAndAccessor is only used in emitCallWithSpread, and it doesn't have to return the accessor anymore.


db.define_table('alumno',
                Field('tipo_doc'),
                Field('nro_doc'),
                Field('apellido'),
                Field('apellido_materno'),
                Field('nombre'),
                Field('segundo_nombre')
               )


db.define_table('legajo',
                Field("alumno_id", db.alumno,
                      requires = IS_IN_DB(db, db.alumno.id, "(%(id)s) %(apellido)s %(apellido_materno)s, %(nombre)s %(segundo_nombre)s" )),
                Field('calle'),
                Field('nro_calle'),
                Field('localidad'),
                Field('cuil')
                )


`$(window).on('resize', function () {})` を使いましょう。


Why skip admins?


woah, what is MAS?


83785560WtoAirHP_ExampleFiles (wfbuhl) - i386-Windows-7-VisualStudio-12: OK (1658 of 1685 tests passed)

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


Are you sure about this? :worried:


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14786/testReport/) for this commit**


marginLeft, margin-left のように lowerCamelCase と hyphens を混在させるのは可能な限りやめましょう。クラス名やID名などDOMの取得に必要な名前についてはこの限りではないです。


Hi,handsome man.I am a new learner of Node.js. Now,I want to read edf files using nodejs,so what I should do in Ubuntu?Please help me,thanks very much.


I'm trying it both ways. I mostly use my phone and the choice is between going over my carbs on the phone or not.

Example 1: 2000 calories, 5% carbs for 25g. I aim for 25g fiber so I just try to go over my carbs by no more than the fiber number (so if I hit my fiber goal exactly, I exceed carbs by 25g)

Example 2: 2000 calories, 10% carbs for 50g. Same fiber target, same 25g net carb target, so as I eat carbs and fiber I approach my limit rather than exceeding it. Now instead of seeing if my overage is less than my fiber, I just make sure I hit my fiber goal and stay under carb goal. This means no overages. However, as I've played with this way (which requires the modification to the script for having a useful remaining net carbs) I've found that my fat number is lower than recommended by the keto calculator because I have to take that 5% from fat to make carbs 10%. The confusing thing is that by calorie or by percent the graph on MFP is wrong because fiber is included in the carb measure. It's just a dumb thing to work around on MFP.

I made this modification to see my remaining net carbs correctly when MFP can't do it right, but I think going over your carbs by your fiber amount (example 1) is actually easier to understand. Then you fat number is correct, you carbs are actually your net, and you use fiber grams as you "allowed to be over carb goal by this amount."

I'm not sure my version is right, so unless you see this a lot, maybe this isn't important. The thing I would love is the ability to override MFP's goals with my own and remove exercise calories. All the stuff they released as premium features that no way in hell I'm paying $10/month for.


Test


`bind` は廃止されていないが古いメソッドなので `on` を使いましょう。


rax retry tests


I don't think this should be here - LABELs are already set in the base image. Any reason to reset them?


@mohamedmorsy Changes look good


`margin-left` ではなく `position: absolute` にした上で `left` を動かすようにしましょう。`margin-left`


Having this show as zero instead of just missing seems like a better option here.  If it's throwing an error let's just make sure that devicelist is a valid array with 0 records.


全体的にシンプルでいいと思います。
１つだけ指摘すると、nullの場合は、早期リターンで、メソッドの一番先頭で返すように修正しましょう。
もう少しシンプルになると思います。
今後も早期リターンを心がけるようにしてください。


@brycemd here I'm generating a new multidimensional array


nullの場合は、早期リターンで返すようにお願いします。


DOMにまつわることは極力jQueryのメソッドを使いましょう。

```js
figureElm = $('<figure>').append($('img').attr('src', photoSrc));
```


There is no graph


`$(thumb).on('click', function (e) {})` イベントハンドラの設定には `on` を使いましょう。


Ok, let me polish this. By the way, I think I'll add a QQ provider in these days.


`'px'` が省略可能なら省略しましょう。可読性が上がります。


Don't merge the vidc firmware.


May 12, 2015
Twitter和Facebook够挂了


Wrap this update_metadata call in the 1 < count check.


Because they are skipped in <code>/alts</code> and this shouldn't be a way to bypass that restriction.


Move this back to where you have it below the other delete_metadata, no `if`


boolは`is`を選択しがちがですが、`hasTouch` とかもありですよ。


Haven't had a chance to try this, but for the KnockoutComponent, what if you aren't using a frame? In that case, the bindings won't be applied at all. All of the examples on GitHub (eg, brjstodo) does the following:

// todo input Blade
  var inputModel = new InputViewModel();
  var inputComponent = new KnockoutComponent( 'brjstodo.input.view-template', inputModel );
  var inputEl = inputComponent.getElement();
  todoAppEl.appendChild( inputEl );

In this case, easily solved with adding a SimpleFrame and calling 'attach' later as in the Dashboard, but definitely sample apps will break that use KnockoutComponent. If going as far as calling an "atta

Maybe an "if" statement to keep existing behavior unless you use a frame (m_oFrame != null). Or, a hack with a setTimeout to call the bindings a few seconds after the first time getElement() gets called [very ugly hack but existing examples likely would still work]


oh man


Merge


@brycemcd here is I'm generating a new array


@bryanbraun: you should be able to install the cli and use it without the full path, yes.

I had opened a PR to add this info in the npm docs, but didn't get merged. The fact is, there is absolutely no need to use the full path since npm looks in those paths by default and secondly it will break stuff on Windows.


Nice.


Very well organized!


Great progress.


missing quotes


OKです。
いいですね。好みの問題ですが、
早期リターンは、メソッドの先頭でするのが好きです。
つまり、7行目と8行目の間！


Impressive.


The Python code of section `Generating some data` should have `plt.show()` in the last line to display the figure normally.


問題ないです。
レビューOKです。
> test3でもそうですが、引数でnullを渡された時の処理の書き方がいまいちわかりませんでした。
問題ありません。これからも早期リターンを心懸けてください。


@brycemcd I know I can't do that, and I know it will be pushing the grade until the end, it means each element of array will have value 73. Ideally I wanted to create an add(grade, week) function, where I can pass a grade and a week.


This is causing dolphin to abort with a `std::bad_alloc` after failing to reserve an *enormous* chunk of memory (`1383741509 * sizeof(SFileInfo)`; over 40 GiB) for `CFileSystemGCWii::m_FileInfoVector`.

In order to get dolphin to start, I have to revert this commit.


Is `c(nbind)'` correct? Does it solve #2?


contains()メソッドではなく、もう少し適切なメソッドがあります。
APIを確認してみてください。
https://docs.oracle.com/javase/jp/6/api/java/util/AbstractCollection.html#containsAll(java.util.Collection)


でっかい if 文ができたら、それ以外の条件を判定して早めに return するようにリファクタリングしましょう。可読性が上がります。

```js
if (this.absX <= imgHalfWidth) {
  return;
}

if (this.diffX > 0) {
...
```

> リーダブルコード 7.5 関数から早く返す


@jonallured thanks!


Indention levels look off here on the GitHub UI, is this correct?  Personally, I actually prefer the original first commit without the extra variables.  I'll think about this, but thanks for the feedback.  Please feel free to ping me/us if you don't hear from anybody in a few days.


@laurencer are you ok with this, as per @stephanh's comments?

If so I'll get this merged


@JacquesCarette : As I understand it, Lazy does everything that Partial does. There is a pending task for decommissioning both Partial and Disintegrate, which I interpret as removing their usages in the code and moving their respective modules to Historical/.

@zaxtax : In case you're asking why Disintegrate.hs isn't being deprecated too, I will get to that task momentarily. In case you're asking why Disintegrate.hs is being deprecated too, it's for the same reason as Partial - Lazy does everything that Disintegrate does.


Final commit to revert back from #14


That's because the git client I used at school didn't allow me to actually login to my Github account, it just allowed me to set a generic username.


please stop


tetta hefur ekkert ad gera med login
burt med allt tetta session dot


コードレビューしてほしいと聞いて。


notadu default SessionStorag, ekki senda neitt in i constructor


Sebastian Wiesner <notifications@github.com> writes:

> @thierryvolpiatto I don't think that you still need
> advice-add/advice-remove if you already have defadvice.

I don't need it, but advice-add/remove is much better and I want to use
it for all 24.4+ users.
Also the code is ready with advice-add, as soon as we have one more
emacs release in our back I will remove the defadvice.

> As far as I know, advice-add is a substitute for defadvice, and
> defadvice still works on Emacs 24.4, so I presume that this actually
> defines the advice twice.

It is not a problem because the defadvice is not activated.

--
Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997


Before: ![before](http://i.imgur.com/VVV1xLd.png)
After: ![after](http://i.imgur.com/TkPPezZ.png)


@drrobotnik Just downloaded this. Should there be buttons in the Theme modal to rollback at this point already? I'm not seeing them.


Because..?


Perfd is unnecessary.  The performance server module is attached to (hosted by) mpdecision via the libqc-mpctl static library on 8974 devices.  On newer SoCs, it is its own binary: perfd.


burt med allt session dot


First commit


版本库地址：http://leelina.github.io/New-Work/index.html


Any ETA on the release?


I'm assuming this is the correct link for the models...


test

- [ ] a


This is wrong?


But it doesn't build on jenkins?


@pravnar I'm just commenting that it will be decommissioned as well. Thanks again!


Remove ','


Para crear inicialmente el archivo package.json debemos ejecutar el
comando `$ npm init`, este comando nos hará algunas preguntas iniciales
sobre nuestro proyecto y es aquí donde definiremos las dependencias de
nuestra aplicación.

En el archivo .gitignore vamos a ignorar inicialmente la carpeta
node_modules pues no es una buena práctica subir las dependencias al
sistema de control de versiones.


:heart:


Not necessarily. As it stands, OS X only has one supported CPU architecture, but it used to have more. If Apple decided to have a 32-bit API that didn't have the Carbon UI APIs, then my change here would be a problem.


- Nice Job So far, Keep it up


:+1: just giving you some cheers for the effort.


I feel like this API change makes for a more confusing API;

```perl
$meta->add( 0 , $name => $code );
```

Just looks weird.


@maxious Created a new property under your GovHack GA account and repointed.

Data thus far:

![2015_05_12_12_08_35_pages_google_analytics](https://cloud.githubusercontent.com/assets/1672960/7580004/a94842ca-f89f-11e4-8302-4444100272b0.png)


Looks legit, it's your app after all :)


I'm using Ubuntu and Debian, and all works without that fix. Anyway I am already merged it.


Here can also perform `self._clean_quote()` instead of assuming the pass-in token has after-clean-quote value (i.e. instead of running `self._clean_quote()` on `attribute_name_token` before passing it in). This way puts all the `self._clean_quote()` process together.


`storeCreateError` really just means `saveCreateError`.  Don't be confused by the `store` inside this `store` lol.


our API is now internal no need for localhost
i believe the code should be:
$http.post("/api/v1/students.json", student)


I'm really not sure what this is trying to do. You're still essentially modifying `@_` for no purpose.

This looks like a footgun just waiting to happen.

It seems like you *might* be trying to support a function being called as a function and a method, but that is generally a very bad idea, and you should only support one of the two, not both.


same here


I have deleted it from my own repo, even if it was already tagged as "stale". Move it to the archives, maybe?
Sorry if I sound ignorant, but what is the use of this branch, except for being a historical monument?


@syl20bnr You made a typo here :)


@syl20bnr same here


怎么样厉害吧


woopz


哈哈哈南


呼呼呼


nice


yeah.. this is a merge issue related to jscs.  We can just cherry pick https://github.com/marionettejs/backbone.marionette/commit/b6a7df9e9a757b89b1a511a0030c89538c644a86 right?


好地，我来给你fork下发个PR


@ibuildthecloud Looking at this merge, I want to confirm that this also closes https://github.com/rancherio/os/issues/225 (whenever Godeps is updated in rancherio/os). Is that correct?


I just googled your username.

![e5tqxah](https://cloud.githubusercontent.com/assets/4640241/7580136/3010adf0-f86f-11e4-9228-8602a3d02a12.gif)


确实很diao


Hmm.. the goal was to make the bottom equal to the larger of the textview or the imageView. So I specified that they should both have their bottom less than or equal to the content's bottom. I think this is right.


this is now an eternal api no need for localhost
i believe the code should be:
$http.post("/api/v1/students.json", student)..... . . . ..


Awesome!  Thanks!


I see no objection to put it back.

-syl20bnr-

On Mon, May 11, 2015 at 11:25 AM, Tristan Hume <notifications@github.com>
wrote:

> I used this a lot, what should I be using instead now that this is gone?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/syl20bnr/spacemacs/commit/7b6678efd6cece5bbb3419579590b843943f9e13#commitcomment-11136286>
> .
>


finally this is gone!


Hi Guys,
Think we might want to relax this constraint and apply it to everything. the public/templates/en/workflow/edit_metadata.tpl.hml template assumes $isSuperAdminstrator is set in order to determine if it wants to show the purge link.


@a-detiste Great, thank you!

> Is this -iwad parameter mandatory ? For all engines ?

All source ports that I know/use/maintain fall back to loading `doom2.wad` if the -iwad parameter is omitted. All but doomsday, that is, which always gets in the way by trying to be "different" or "special" or extra clever or whatever...

But @smcv has a point: it is not possible to buy nerve.wad without also getting doom2.wad, so let's just keep it this way.


明明网页上就可以改的，已给你发PR了


Each item in the `self._split_tokens_by_comma` does not contain token `comma`? If all the tokens are identified as non-column definition, the end result of `non_columns_definition_tokens` will not contain commas and won't be the same as original token lists. The user won't be able to do further processing if they'd like to.


Wordsmithing: “End Users” → “End users”, a comma after both occurrences of “app”, and a `+` before the `[`.


Did you mean to set the ivar here?


@person808 oops, thank you :-) you can PR a fix if you want, I'm going to bed.


KEY should remain LINE. The comment should read:

    The Opheim algorithm can produce unexpected results if the path
    returns back on itself while remaining within TOL from the LINE.


Waverunner look in the odb/chat rooms index, the source file is on it, but the access is prohibited with the gitignore file  when you clone the directory, is empty ,and you don't want, admit that.


No, not yet. @AbrarSyed needs to finish up the necessary changes to FG2.0 before a full build can be done.


There are other job boards such as PHP Vietnam Group. Besides, it's never been our goal to cover every languages on the market, we only aim to get Ruby and other languages that we have resources to filter out poor quality job posts.


The preference is called “Mapbox Metrics” and defaults to enabled, and the key is called `MGLMapboxMetricsEnabled`. Let’s use positive terminology and clarify what is being opted into/out of: `-setShowsMapboxMetricsEnabledSettingInApp:`. It’s verbose, but that’s par for the course in iOS development.


Duly noted. Now that we have a full set of test helpers, time to document testing in the guides. Thanks @josevalim!


``self.assertGreater`` would give us better information in case of failure ("*X is not bigger than 0*" instead of "*False is not True*").


Fixed a typo


Is this function used anywhere?


This is not a simplification to me: the number of terms are always the same with the difference that before there was a clear logic. IMO this should be proved stronger than master to be accepted.


こういうのどうでしょう
[git 管理から外す指定方法 3 パターンの使い分け - Qiita](http://qiita.com/qurage/items/0333a210c151324064e8)


to be continued


<!-- PMABOT:SOB -->
This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check [contributing documentation](https://github.com/phpmyadmin/phpmyadmin/blob/master/CONTRIBUTING.md) for more information.


Do NOT exceed 80 columns in each line.


you can delete the old code.


Glad you liked =)


thanks but not for this patch - let's clean that up separately - thanks!


@kaiwren for performance reasons, could this be just #{yield \`self.slice()\`}? I.e. use the JS way to clone the array instead of the slower Ruby way (since dup does more work)?


I was wondering why that wasn't working. Thanks!


You can try
ans[i][0] = 0 if obstacleGrid[i][0] == 1 else ans[i - 1][0]
same as below

http://stackoverflow.com/questions/2191890/conditional-operator-in-python


asdasd


@aarononeal This version nor 0.6.0 are showing up on npm


actualizacion


Class name "Camera" does not sound like an operation, better to use "ManipulateCamera" or some other proper name as in the case of other operation classes.


also for some reason loading a savestate after restarting the browser causes gambatte to lockup, at least on firefox in windows.
Earlier today I was able to get a save to work across restarts on firefox in linux


Had to do a nasty filecopy instead of uploading to the correct folder because it seems BrowserFS and FS.createDataFile don't get along well.


Two comments :-).

1. I love that you're working on edb. If you merge these in with the clone repo, and do a pull request some of these changes can be merged into the official tree!

2. Just wanted to let you know that there is already an API for "temporary breakpoints", or more specifically "one shot" breakpoints. If you look in Debugger.cpp:977 There is an example of setting a "one time" breakpoint which is both internal (meaning that it won't show up to the user) and one time (meaning it will self delete when hit once).

I look forward to seeing what you end up doing with edb!


I _think_ this is what broke the RSS feed again:

> https://18f.gsa.gov/feed/

I'm working on a fix.


you can use +=


here is the 3 slides for the company part of the presentation. @2014280198 + @2015mostafa  please add your part to the presentation.


userSessionRule is static but not annotated with @ClassRule. Is that possible ?


error on line 12 ?????


error?????? what to write here


thanks for 团子 teaching me javascript:)


Simplification or not is debatable. But what I really don't like is skipping the STC test. If you do direct LTC testing, probability of type I error is significantly higher.


Shall we change class name "AlreadyRegisteredActivity" to "RegisteredActivity"?, because "Already" doesn't make much sense here.


HI Gokul,

Thanks for your response and Guidance . I am trying to run the same code but I got the error mentioned below
Error is :  Traceback (most recent call last):
  File "log2kafka.py", line 6, in <module>
    import kafka
  File "/usr/local/lib/python2.7/dist-packages/kafka/__init__.py", line 9, in <module>
    from kafka.client import KafkaClient
  File "/usr/local/lib/python2.7/dist-packages/kafka/client.py", line 16, in <module>
    from kafka.protocol import KafkaProtocol
  File "/usr/local/lib/python2.7/dist-packages/kafka/protocol.py", line 8, in <module>
    from kafka.codec import (
  File "/usr/local/lib/python2.7/dist-packages/kafka/codec.py", line 2, in <module>
    from io import BytesIO
ImportError: cannot import name BytesIO
I have tried to add the kafka dir which was found in tail2kafka and I got the below  mentioned Error

Traceback (most recent call last):
  File "log2kafka.py", line 43, in <module>
    main()
  File "log2kafka.py", line 33, in main
    if options.topic is None or options.logfile is None:
NameError: global name 'options' is not defined

 Thanks in Advance......


Those will be d'Henri the next release will be out


github에 commit에 코멘트를 달아보자


Thanks @darekx4 , appreciate that !


this const int is declared in more than one file. should it be in the driver file only?


i saw your comment on the other files. does it work now?


so this happened:

```
$ kitchen destroy
-----> Starting Kitchen (v1.4.0)
-----> Destroying <default-ubuntu-1404>...
       ==> default: Forcing shutdown of VM...
       ==> default: Destroying VM and associated drives...
       Vagrant instance <default-ubuntu-1404> destroyed.
       Finished destroying <default-ubuntu-1404> (6m5.69s).
-----> Destroying <default-centos-66>...
       ==> default: Forcing shutdown of VM...
       ==> default: Destroying VM and associated drives...
       Vagrant instance <default-centos-66> destroyed.
       Finished destroying <default-centos-66> (0m23.35s).
-----> Destroying <delivery-10-ubuntu-1404>...
       ==> default: Forcing shutdown of VM...
       ==> default: Destroying VM and associated drives...
       Vagrant instance <delivery-10-ubuntu-1404> destroyed.
       Finished destroying <delivery-10-ubuntu-1404> (0m8.37s).
-----> Destroying <delivery-10-centos-66>...
       ==> default: Forcing shutdown of VM...
       ==> default: Destroying VM and associated drives...
       Vagrant instance <delivery-10-centos-66> destroyed.
       Finished destroying <delivery-10-centos-66> (0m5.84s).
-----> Destroying <delivery-11-ubuntu-1404>...
       ==> default: Forcing shutdown of VM...
       ==> default: Destroying VM and associated drives...
       Vagrant instance <delivery-11-ubuntu-1404> destroyed.
       Finished destroying <delivery-11-ubuntu-1404> (0m5.72s).
-----> Destroying <delivery-11-centos-66>...
       ==> default: Forcing shutdown of VM...
       ==> default: Destroying VM and associated drives...
       Vagrant instance <delivery-11-centos-66> destroyed.
       Finished destroying <delivery-11-centos-66> (0m5.19s).
-----> Destroying <delivery-12-ubuntu-1404>...
       ==> default: Forcing shutdown of VM...
       ==> default: Destroying VM and associated drives...
       Vagrant instance <delivery-12-ubuntu-1404> destroyed.
       Finished destroying <delivery-12-ubuntu-1404> (0m5.34s).
-----> Destroying <delivery-12-centos-66>...
       ==> default: Forcing shutdown of VM...
       ==> default: Destroying VM and associated drives...
       Vagrant instance <delivery-12-centos-66> destroyed.
       Finished destroying <delivery-12-centos-66> (0m8.20s).
-----> Destroying <delivery-13-ubuntu-1404>...
       ==> default: Forcing shutdown of VM...
       ==> default: Destroying VM and associated drives...
       Vagrant instance <delivery-13-ubuntu-1404> destroyed.
       Finished destroying <delivery-13-ubuntu-1404> (0m4.94s).
-----> Destroying <delivery-13-centos-66>...
       Finished destroying <delivery-13-centos-66> (0m0.00s).
-----> Destroying <delivery-14-ubuntu-1404>...
       Finished destroying <delivery-14-ubuntu-1404> (0m0.00s).
-----> Destroying <delivery-14-centos-66>...
       Finished destroying <delivery-14-centos-66> (0m0.00s).
-----> Destroying <delivery-15-ubuntu-1404>...
       Finished destroying <delivery-15-ubuntu-1404> (0m0.00s).
-----> Destroying <delivery-15-centos-66>...
       Finished destroying <delivery-15-centos-66> (0m0.00s).
-----> Destroying <delivery-16-ubuntu-1404>...
       Finished destroying <delivery-16-ubuntu-1404> (0m0.00s).
-----> Destroying <delivery-16-centos-66>...
       Finished destroying <delivery-16-centos-66> (0m0.00s).
-----> Destroying <delivery-17-ubuntu-1404>...
       Finished destroying <delivery-17-ubuntu-1404> (0m0.00s).
-----> Destroying <delivery-17-centos-66>...
       Finished destroying <delivery-17-centos-66> (0m0.00s).
-----> Kitchen is finished. (16m25.95s)
```


It's probably better to indicate if the dialect is not specified, the parser will parse given sql statements based on generic sql syntax.


we don't use this file, do we? can be deleted?


lol Might as well add the HxC team :P http://github.com/HxCKDMS KeldonSlayer, KarelMikie3, Mfernflower


Thanks! BitHub has sent payment of  $10.77USD for this commit.


or, since it is used in member functions, should it only be in the header file, and not in the driver file? the #include for the header file would pass it to main? i'm not sure.


@PragatiDoshi-BTC - client.address_city returns string value. So no need to use "to_s" and use string interpolation for e.g. "#{first_name} #{last_name}" in both methods. Please change this.

String Interpolation is faster than << and +
http://stackoverflow.com/questions/13558255/rails-best-way-to-concatenate-a-string-with-a-variable-in-a-view


Ok. Fixed.


Fixed.


See commit 72710b0.


@PragatiDoshi-BTC  - Change method name from "name" to "full_name" because it returns client's full name


is this really safe change?


코드리뷰는?


same as above


one more


dtto


tested and working


Thank you!


Not sure, if it should return a http.InternalConfigError... On the end, the user tryies to configure a dir, and misspelled a path, with an Internal server error, and a clear error message perhaps is better.


Please review this, @casey-collab


i didn't get the reason for adding '_' ... a convention?


Ok. I really don't care. Your lines are more informative, so we'll take them.
Fixed.


Fixed too.


import ex25


谢谢


TeamCity Nimbus :: 0. Feature Branches [Build 0.0.0.0](http://twcbuild.cloudapp.net/viewLog.html?buildId=1152&buildTypeId=bt8) outcome was **UNKNOWN**
Summary: Canceled Build time: 05:22:22


import ex25 so the functions could be called


packages to user


this is misspelled.


We need @thlorenz to publish to npm since I'm not an owner on the entry.


What happened to get()?


+Work around on favicon display


Super+! no more optional homework this week. :)


It should fix https://github.com/twpayne/igc2kmz/issues/8


`form_set_error('mail', t('Email yang digunakan harus gmail'));`

tolong perbaiki


>This seems like a hack. Creating another instance of the same class just to destroyed it a few lines below.

In this case, C++ is a completely hack! Here we are constantly creating objects to immediately remove them. Just a little less obviously. Here is an example:
```c++
SomeClass var;
// ...
var = SomeClass(arg1, arg2);
// Here we create another instance of SomeClass,
// then call SomeClass::operator=(const SomeClass &other) of first instance,
// then destroy second instance
```

and so on...

And now to the point. Here it is in order to reuse DownloadManager::downloadUrl() logic.


Yes deck.h can be deleted
On May 11, 2015 10:19 PM, "goldfish" <notifications@github.com> wrote:

> we don't use this file, do we? can be deleted?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fishtez/blackjack/commit/1c9e61af1e7135a8b4749cb8701b73e9a7743c30#commitcomment-11147737>
> .
>


@Fmressel I started work on the backbone and some essentials


The const int should be in driver only
On May 11, 2015 10:17 PM, "goldfish" <notifications@github.com> wrote:

> this const int is declared in more than one file. should it be in the
> driver file only?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fishtez/blackjack/commit/4d59d27cba2e922c68066c189794da841d5953c6#commitcomment-11147716>
> .
>


I like it.

I was sort of attached to using lifetimes to represent the sqlite API constraints; but stepping back, what was really important was to avoid allocation, and this use of Rc<> doesn't add any dynamic allocation; it just costs one word per DatabaseConnection and per PreparedStatement and a tiny bit of refcounting overhead. Pretty good bang-for-the-buck.

I suggest you add yourself to the list of authors, by the way.


I think it would be best to put it in the header fil as a fucntion of the
class and then define with the other card functions in card.cpp
On May 11, 2015 10:22 PM, "goldfish" <notifications@github.com> wrote:

> or, since it is used in member functions, should it only be in the header
> file, and not in the driver file? the #include for the header file would
> pass it to main? i'm not sure.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fishtez/blackjack/commit/4d59d27cba2e922c68066c189794da841d5953c6#commitcomment-11147747>
> .
>


Create one function
```
def ignore_files(self, field, value):
```


Here, everything is fine. ```m_reply``` already points to another object.


Here's the link for The Economist:

http://www.economist.com/style-guide/commas

But it's a very simplified treatment of the topic.


Don't remember why I didn't remove this debug spam...
Fixed.


do you want to print to stdout? might be better to write to log instead


@PragatiDoshi-BTC - Change method name "name" to "full_name" because this method returns client's full name.


Ну программа выдает всегда строку стандартного вида:
структура  minimum free energy = значение_mfe kcal/mol
И там везде по одному пробелу


Tinggal manggil method/fungsinya aja gan.


It can't cover advanced WM feature such as snap, drag to boder to maxmize.
[Way to solve this perfectly on Windows](http://stackoverflow.com/questions/5493149/how-do-i-make-a-wpf-window-movable-by-dragging-the-extended-window-frame), but Py can't win32api


it seems that here is not calling Unlock?


Hold on... is this actually the code review?!

I'm used to more someone taking apart everything, especially since I'm so
new to CSS:

1. design (why did I do CSS a certain way with certain classes instead of
some other way)
2. convention (normally, things are done this way.../written this way)

I expected a brutal code review that would force a total rewrite of my
crappy CSS (i.e. pointing out 20+ instances of how my code sucked)...
this is more like a tempest in a teapot.  I suppose you haven't done code
reviews before...?

On Mon, May 11, 2015 at 2:23 AM, ranganathk <notifications@github.com>
wrote:

> It is a bad idea to keep email addresses so openly. Spam bots easily pick
> email addresses posted on public pages.
> Obfuscating email addess is suggested.
> http://www.thedeviantco.com/protect_your_email_address_from_spam.php
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/chaitanya-m/CodeAstraFrontendHackathon1/commit/b1a77a023e569566323cf997e2bed7ba2cc1d0a0#commitcomment-11129729>
> .
>


In the specific case of king moves, wouldn't it be possible to avoid the MoveList computation and loop by something similar to the following bitboard manipulation?

kingMoves = ~pos.pieces(stm) & attackedBy[stm][KING] & ~attackedBy[~stm][ALL_PIECES];


Yeah ... looks like a leftover comment :)


i think that Lock/Unlock in one function may be better. how about u?


For diagnostics purposes, consider including some diagnostic output either always or when we're above a certain log level.

Say, after running

    build.cmd /p:WithCategories="innerloop;outerloop;perf

Instead of warning *{0} has no tests to run*, consider something like

````
blahblahblah.Test.csproj
[InnerLoop]          10 tests discovered
[OuterLoop]           3 tests discovered
[Perf]                0 tests discovered
````


LGTM


Let's bring this back.


`response's'` のつもりだったんですよね。`reses`とかの方が良いかもしれないです。


fixes #265


Can we make this more flexible so that it's easy to add supports for more dialects as well as different types of sql statement later? Consider using polymorphism.


Could you please also add this to formatters?


Whoops. That's supposed to say "to vidr/util"


Super+! No more optional homework this week. :)


I made my own version of this [here](https://github.com/thislooksfun/vidr/blob/c9f2f159898d27352fe719853bce8a4db16d7cdb/src/vidr/util/util.go#L83-L91) - feel free to comment


@kaiwren @meh re-writing this line as follows fixes the problem, so it's a leaky reference somehow (the *how* in "somehow" is left as an exercise for the reader):
```ruby
#{yield `chosen.slice()`}
```
:beers:

P.S. @kaiwren declaring all you `var`s once at the top of the function is a JS "best-practice" (according to Douglas Crockford). Love or hate the [straitjacket](http://en.wikipedia.org/wiki/Straitjacket) that is *JSLint*, old Doug usually turns out to be right in the end :wink:


I think CardDriver.cpp is our new main, could you add this to that?


sorry, i made a mistake. acquire/release is ok.


ee


Oh okay


Yes. Is that the updated check ace?
On May 11, 2015 11:03 PM, "goldfish" <notifications@github.com> wrote:

> I think CardDriver.cpp is our new main, could you add this to that?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fishtez/blackjack/commit/9f80ccf7b0f43880644e87c0628b0549deb086b2#commitcomment-11148039>
> .
>


yea


fixed in c8aa777


Я тут нашла один глюк. RNAfold недавно сделали новую версию. И я все обновила.
Новая версия выдает такую строку:
структура  minimum free energy = значение_mfe kcal/mol

Старая версия выдавала такую строку:
структура (значение_mfe)

И почему-то PyCharm обращается к старой версии RNAfold. И сам скрипт тоже, раз все работает из командной строки и при этом не падает, когда вид строки, которую выдает RNAfold, по идее изменился


Have you tested if this works with /root - I think the reason it was like this is because the content caches xpath ignores the /root part - not 100% sure tho - agree that /root would make the most sense


maybe here it would be better to use "__declspec(deprecated(message)) declarator"
according to: https://msdn.microsoft.com/en-us/library/044swk7y.aspx
By this we would have a proper warning with what we should replace.


We still need to add a ChannelCase for new apps. I hope to get to it still this week.


I really don't know what to do here to be honest, @Supermathie what should we do?


@PragatiDoshi-BTC - Here no need to convert id into integer. Please remove.


changes as per commit b206f3c


I tested it with MNTP and with this new xpath. Without this change $root just get's stripped and then nothing happens since the xpath is invalid. Would be good if others can test before this goes out. It would only have affected MNTP since this is the only place where this specific xpath logic was used afaik //cc @nul800sebastiaan


we need it to be global, though. it is used in int main().


this requires the settings that marathon consul has for granular security


Why not to use message in MSC as well: "__declspec(deprecated(message)) declarator"?
Seems this is possible: https://msdn.microsoft.com/en-us/library/044swk7y.aspx


remove the whitespace at the end of these lines


The only thing you changed about them AFAICT is the amount of whitespace at the end.


Care to write some code? I don't think we have attackedBy[] in the search

On Tue, May 12, 2015 at 7:56 AM, Stéphane Nicolet <notifications@github.com>
wrote:

> In the specific case of king moves, wouldn't it be possible to avoid the
> MoveList computation and loop by something similar to the following
> bitboard manipulation?
>
> kingMoves = ~pos.pieces(stm) & attackedBy[stm][KING] &
> ~attackedBy[~stm][ALL_PIECES];
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mcostalba/Stockfish/commit/278c9faa1fa3cfc6d58d7c5adc247cefc3c7868d#commitcomment-11147964>
> .
>


remove the uncessary local repo as per b206f3c


So i guess leave in the driver only then
On May 11, 2015 11:10 PM, "goldfish" <notifications@github.com> wrote:

> we need it to be global, though. it is used in int main().
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fishtez/blackjack/commit/4d59d27cba2e922c68066c189794da841d5953c6#commitcomment-11148104>
> .
>


Aqui modifique en la version 1.1


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/14787/testReport/) for this commit**


LGTM


Please rename `AOUT` to something like `test.out` to reflect its purpose.


omg omg

http://www.php-fig.org/psr/psr-1/#2.3.-side-effects

tetta ma alsekki


`ipsearch` _is_ leader-only; I'd be fine with giving leaders+ the ability to bypass this restriction.


Shouldn't we better omit `File::Spec->` considering the fact that the function is imported?


should this actually be a `seamless` iframe?


Autopep8 is good for handling whitespace btw


please add: `== 0`


Thank you for the changes.

I would appreciate if you could also omit loading the unnecessary macros via `use POSIX qw(...)`.


oh, you had to do this one manually because my PR bit rot. thanks, mayne.


Suggestion: add a .gitignore file and put
*.pyc
in there, to avoid committing the byte-code to your repo


same as https://github.com/mattn/qrintf/commit/f320f0c04824607633414d9366e87f1c9c54fc07#commitcomment-11148208


yeah - last weekend i wrote something exactly like this :-)
I think it makes sense to get this on lkml rigth away.
Also it would be best to rebase this specific branch to upstream - 4.1-rcX
That way u can get the needed upstrewm changes automatically and anyhow upstream worthy code is better off if it is closer to mainline !


We got a report that this commit breaks Jena clients requesting XML SPARQL results due to bug in the xerces library.
This was reported for dbpedia.org but it will affect all virtuoso instances.

An example using Jena that shows the problem is
```
sparql --data empty.ttl "SELECT * { SERVICE <http://dbpedia.org/sparql> { ?s ?p ?o } } LIMIT 1"
```
and produces this stack trace

```
Failed when initializing the StAX parsing engine
com.hp.hpl.jena.sparql.resultset.ResultSetException: Failed when initializing the StAX parsing engine
	at com.hp.hpl.jena.sparql.resultset.XMLInputStAX.<init>(XMLInputStAX.java:110)
	at com.hp.hpl.jena.sparql.resultset.XMLInput.make(XMLInput.java:81)
	at com.hp.hpl.jena.sparql.resultset.XMLInput.fromXML(XMLInput.java:39)
	at com.hp.hpl.jena.sparql.resultset.XMLInput.fromXML(XMLInput.java:34)
	at com.hp.hpl.jena.query.ResultSetFactory.fromXML(ResultSetFactory.java:309)
	at com.hp.hpl.jena.sparql.engine.http.Service.exec(Service.java:191)
	at com.hp.hpl.jena.sparql.engine.main.iterator.QueryIterService.nextStage(QueryIterService.java:56)
	at com.hp.hpl.jena.sparql.engine.iterator.QueryIterRepeatApply.makeNextStage(QueryIterRepeatApply.java:110)
	at com.hp.hpl.jena.sparql.engine.iterator.QueryIterRepeatApply.hasNextBinding(QueryIterRepeatApply.java:67)
	at com.hp.hpl.jena.sparql.engine.iterator.QueryIteratorBase.hasNext(QueryIteratorBase.java:112)
	at com.hp.hpl.jena.sparql.engine.iterator.QueryIterSlice.hasNextBinding(QueryIterSlice.java:76)
	at com.hp.hpl.jena.sparql.engine.iterator.QueryIteratorBase.hasNext(QueryIteratorBase.java:112)
	at com.hp.hpl.jena.sparql.engine.iterator.QueryIteratorWrapper.hasNextBinding(QueryIteratorWrapper.java:40)
	at com.hp.hpl.jena.sparql.engine.iterator.QueryIteratorBase.hasNext(QueryIteratorBase.java:112)
	at com.hp.hpl.jena.sparql.engine.iterator.QueryIteratorWrapper.hasNextBinding(QueryIteratorWrapper.java:40)
	at com.hp.hpl.jena.sparql.engine.iterator.QueryIteratorBase.hasNext(QueryIteratorBase.java:112)
	at com.hp.hpl.jena.sparql.engine.ResultSetStream.hasNext(ResultSetStream.java:75)
	at com.hp.hpl.jena.sparql.engine.ResultSetCheckCondition.hasNext(ResultSetCheckCondition.java:59)
	at com.hp.hpl.jena.sparql.resultset.ResultSetMem.<init>(ResultSetMem.java:97)
	at com.hp.hpl.jena.query.ResultSetFactory.makeRewindable(ResultSetFactory.java:419)
	at com.hp.hpl.jena.sparql.resultset.TextOutput.write(TextOutput.java:149)
	at com.hp.hpl.jena.sparql.resultset.TextOutput.write(TextOutput.java:132)
	at com.hp.hpl.jena.sparql.resultset.TextOutput.write(TextOutput.java:120)
	at com.hp.hpl.jena.sparql.resultset.TextOutput.format(TextOutput.java:65)
	at com.hp.hpl.jena.query.ResultSetFormatter.out(ResultSetFormatter.java:166)
	at com.hp.hpl.jena.sparql.util.QueryExecUtils.outputResultSet(QueryExecUtils.java:160)
	at com.hp.hpl.jena.sparql.util.QueryExecUtils.doSelectQuery(QueryExecUtils.java:199)
	at com.hp.hpl.jena.sparql.util.QueryExecUtils.executeQuery(QueryExecUtils.java:80)
	at arq.query.queryExec(query.java:208)
	at arq.query.exec(query.java:159)
	at arq.cmdline.CmdMain.mainMethod(CmdMain.java:102)
	at arq.cmdline.CmdMain.mainRun(CmdMain.java:63)
	at arq.cmdline.CmdMain.mainRun(CmdMain.java:50)
	at arq.sparql.main(sparql.java:28)
Caused by: java.lang.NullPointerException
	at com.sun.org.apache.xerces.internal.impl.XML11NSDocumentScannerImpl.scanStartElement(XML11NSDocumentScannerImpl.java:356)
	at com.sun.org.apache.xerces.internal.impl.XMLDocumentFragmentScannerImpl$FragmentContentDriver.next(XMLDocumentFragmentScannerImpl.java:2786)
	at com.sun.org.apache.xerces.internal.impl.XMLDocumentScannerImpl.next(XMLDocumentScannerImpl.java:606)
	at com.sun.org.apache.xerces.internal.impl.XML11NSDocumentScannerImpl.next(XML11NSDocumentScannerImpl.java:857)
	at com.sun.org.apache.xerces.internal.impl.XMLStreamReaderImpl.next(XMLStreamReaderImpl.java:558)
	at com.hp.hpl.jena.sparql.resultset.XMLInputStAX$ResultSetStAX.processHead(XMLInputStAX.java:379)
	at com.hp.hpl.jena.sparql.resultset.XMLInputStAX$ResultSetStAX.init(XMLInputStAX.java:195)
	at com.hp.hpl.jena.sparql.resultset.XMLInputStAX$ResultSetStAX.<init>(XMLInputStAX.java:187)
	at com.hp.hpl.jena.sparql.resultset.XMLInputStAX.worker(XMLInputStAX.java:147)
	at com.hp.hpl.jena.sparql.resultset.XMLInputStAX.<init>(XMLInputStAX.java:106)
	... 33 more
```


test repo


http://www.php-fig.org/psr/psr-2/#4.2.-properties

Method names SHOULD NOT be prefixed with a single underscore to indicate protected or private visibility.


fixed it. it needed to be extern const int in header file.


Lan Solo ^^


Better to set duplicate values in code than repeat them in the JSON file.


o that seems easier to do then what i did lol


This appears to be what is in production...?


Не, нельзя так делать! zram и zcache не должны быть загружены одновременно - иначе получается так, что одни и те же данные сжимаются дважды, что крайне неэффективно.


When possible, name the strategy the same as the scheme. Keeps things easier to follow.


Don't make life complicated. Someone will need to type this to test it.


Keep the callback signature consistent even when not using the arguments.


The commit message you used doesn't follow our convention: It should start (in this case) with "SHERLOCK: ". Also, it's too long and should have been cut in 2 lines.


for the record, it would be good to have in the ResourceManager a function like "getSkirmishLevelsPath" to avoid building paths like that.


Seems good.


Я уже понял. Просто некогда было менять.


@drmjo and @Myke79 , you both have the same problem.
You both need to update your symfony 2.3 until you get a [composer.json config more like this one](https://github.com/symfony/symfony-standard/blob/v2.3.28/composer.json#L12).
Basically, doctrine migration need doctrine 2.4 and symfony 2.3 supports it. So you should upgrade.
The [upgrade](https://github.com/doctrine/doctrine2/blob/master/UPGRADE.md#upgrade-to-24) will be easy.


If you rescale by 1 then you can remove that entirely.


i googled it. i knew there had to be a way to share a global constant
across different files. this way it is only in one place, so if we change
it, the change will be consistent across all files.

On Mon, May 11, 2015 at 11:27 PM, DavidL47 <notifications@github.com> wrote:

> o that seems easier to do then what i did lol
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fishtez/blackjack/commit/4d59d27cba2e922c68066c189794da841d5953c6#commitcomment-11148253>
> .
>


this will give the timestamp when the notification was received, not when the notification was sent


Either use the logging facilities or remove that `print` call.


What's `reference_id`? Is it optional? Because from the API documentation:
> Sources Account Management
> ==GET== : Add an account to the system
> #####Example Call
> `curl -X GET -d https://app.viralheat.com/mobile/api/v1/sources/:platform/authenticate?client_token=:token`
>
> #####Call Parameters
>
> Parameter | Description | Required
> ------------ | ------------- | -------------
> :token | the client token also known as the user token | Y
> :platform | The platform. Could be twitter, facebook, instagram, linkedin, or tumblr | Y
>
> #####Output
> After authentication, the system will redirect the user to a page of your choice.


so the program runs for you guys now?


It is mentioned upthread.Easiest place is the Releases tab.


I think you can remove that `_model` suffix, that's just noise ;-)


`temp.files = glob.sync(version + "/**/*");` is slow, no need to scan them all, maybe I'll add a option for force rebuild, sometimes we'll update the already exist versions, but very seldom.


@wilaw, I think the old version was correct - typeof returns string, so I believe we should compare it with "undefined" string, not with undefined value itself. Also I think that null should be returned here since we have another return point that returns an object.


Можно реализовать очень просто:
```python
def __iter__(self):
    return iter(self._snake_coord)
```