Welllll, I’ve hit a level of frustration I wasn’t expecting.   The information in the reading doesn’t seem to provide the material I need to complete these exercises.   Maybe I’m making it harder than it’s supposed to be.   I don’t know.   I have accessed numerous different resources looking for code that matches, or is at least somewhat similar to what I’m being asked to do on websites like Lynda.com, Apple iOS Developer Library, Ray Wenderlich’s website, weheartswift.com, etc.  I have purchased two electronic books on Objective C and Swift from NOOK.  I have been hunting through blogs and still I don’t feel like I am getting the exercises correct.   I have started my own separate playgrounds and projects trying to get the exercises to work by starting with less complicated exercises on the same topics from other websites.   Basically, I feel like I have been spending all of my time looking on the internet for information to solve the exercises and coming up empty handed.

I am a little confused by the teaching style.  Since I know nothing about programming it’s reasonable to think that this is a proven and effective training method.   However, even when I do solve the exercises it seems like the educational benefit isn’t very high for the time expended.

I finally gave up and thought if I looked at the answers in wiki of the Bloc/ios-exercises I could work backwards and solve the exercise.    The “answers” in the wiki file were also unclear and you indicated in the email something was still wrong.

I jumped to Swift versus Obj-C and completed almost all of it but I’ll be damned if I can get the Favorite Drink one right or the Worf/Picard part right.   I haven’t even tried the cereal one.   I attached a file that I used to work out stuff instead of wrecking the original file.

I am kind of reluctant about writing this to you because I don’t want to seem like an idiot.   I think what would help me at this point is are the full solutions to the Favorite Drink and Worf/Picard segments and the full answers to Arrays and Dictionaries.   What I mean by full answer is the finished code from top to bottom not just the lines that are supposed to go into the code.    I’d like that because just getting the answers themselves hasn’t always made sense to me.

I haven’t tried to redo the Arrays and Dictionaries yet, but I’d like to have the answers so that I can refer to them if I do get stuck.   I don’t know that the 30 minute meeting on Tuesday would be enough time to go over all of that code.   I feel like I’m getting behind.   Thoughts????

Cynthia

On Apr 25, 2015, at 8:27 AM, dothework <notifications@github.com> wrote:

> @Notsosof1 better to use functions whenever they exist isNumberEqualTo
>
> —
> Reply to this email directly or view it on GitHub.
>


Yes, I tried. But I only could come up with crappy solutions - basically because the return type of visit is `void`. If we changed it to `QVariant` I think it would be possible. But then again it would be hard to keep track of the context a node is used in. I don't know if this will be a requirement one day but if yes it will be a limitation again. Therefore I am not a big fan of it yet.


everything looks good, but can we remove the debug stuff since if this is going to be our last commit


attempt to fix fireball-x/dev#605


`createUseInfoTable.toString()` が一般的ですね。そっちにしましょー。

ただ、
AbstractStringBuilderを見ると、`String(int offset, int charCount, char[] chars)` を呼び出しているので、処理上のコストとかはあんまり変わらなそうですね。
nullだとNullPointerExceptionが起こるのも同じな気がします。


este in regula ca ai facut constructorul in clasa Person, si dandu-i access modifier-ul protected il poti apela din clasele derivate. Dar daca ai vrea sa instantiezi un obiect de tipul Person nu ai putea sa il construiesti prin constructorul acesta, var person = new Person("predescu", "ana", "17/06/1989") pentru ca acest constructor nu e vizibil din exterior.


That's fine, it doesn't really make a any difference I just know this is
the main Martin added a vistor pattern to it.  I suspect you would just
have a internal class that inherits from it and builds up a string that you
ask for when done once accept returns.

On Mon, 27 Apr 2015 at 16:11 Matthias Kuhn <notifications@github.com> wrote:

> Yes, I tried. But I only could come up with crappy solutions - basically
> because the return type of visit is void. If we changed it to QVariant I
> think it would be possible. But then again it would be hard to keep track
> of the context a node is used in. I don't know if this will be a
> requirement one day but if yes it will be a limitation again. Therefore I
> am not a big fan of it yet.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/m-kuhn/QGIS/commit/08e6f669b10a5914b9c9d909d0b21c7a4c2f5654#commitcomment-10909596>
> .
>


Good catch :+1:
Fixed with 68cefea95993a310618a6feb1d832caf067cc7f0


It looks like you didn't actually include XmlFinder.cpp, XmlFinder.h, ResourceLoader.cpp, and ResourceLoader.h in this commit.

Also, just as a general comment, it seems nicer to have one commit per issue, rather than squashing many changes into a single big commit.  This makes it easier to cherry-pick or revert individual changes later on, or to bisect bugs to specific commits.  (For instance, this makes it easier to bisect if any texture loading issues are due to the ResourceLoader changes or to the switch to wxGLCanvas.)  If anyone is looking back through the commit history to find the rationale for a particular change, this also makes it easier to see the reasoning behind specific changes.  Anyway, it's your repo, so feel free to ignore me and do whatever you want, but that's just my two cents.


it was for "pipelineEngine", but you can drop it if you want...


Nice! See how was not that hard? :dancer:


Meh, that's not better.

Let's just say yeoman-assert like the module name.


leaveType ar trebui sa fie de tipul LeaveTime (enumul) pentru a nu fi nevoie sa scrii in constructorul Leave LeaveType.Holiday.ToString()


大傻逼


Schimbarile sunt foarte bune, cu cele doua precizari facute.


__API Blueprint syntax error on line 788__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


make as many methods `static` as possible


Let's keep this like before. `yeoman-assert` is a dropin replacement for the `assert` core module.


Oren,

this will fail due to https://github.com/ayende/ravendb/blob/master/Raven.Client.Lightweight/Shard/ShardedDocumentStore.cs#L277

FOund this out, because core tests in v3.5 started failing after merge.


Maybe `HasProgressIndicator()` is more fitting because a job usually only has one of the two set. Having both set doesn't make much sense.


staticで4つ変数を宣言していますが、これだけ`s` が付いています。
Android的には付けるが正解なのかなーという気がしていますが、とりあえずはどちらかに統一して下さい。


インスタンス変数は`m`プレフィックスを付けたほうがいいかと。
コンストラクタで`mContext = context`みたいに、thisを省略できますし。


OK. what about changing it to:
if a default value for that parameter is present in the UIMAfit Annotator, we use that value


Please add the default value as a comment.


Hmmm. The downloaded file may end up in a (hard to find) temporary directory.

The people around me typically extract portable applications onto the desktop (or, less often, their home directory, because it is harder to get to, in the Explorer).


`MarkFinished()`?


please add a description, e.g.
/** used to separate the engine id when creating tmp xml descriptor */


* Excellent, thanks for refactoring that clunky piece of code!
* Maybe it's time to move the whole `ENGINE` handling code to its own class, WDYT?


No no I do not mind.
look this is link click : http://off.farazma.ir/
textbox in Google chrome (version 42.0) not work selection mouse .


This message title was wrong. Sorry. I really don't like GitHub's Desktop app. It's terrible. It told me I was committing deleted files and this is actually what was committed. Sorry for the misinformation.


A lot of tests are now failing in 3.5 without this.
You can change the calls to this to a virtual property that will call this,
instea.d

*Hibernating Rhinos Ltd  *

Oren Eini* l CEO l *Mobile: + 972-52-548-6969

Office: +972-4-622-7811 *l *Fax: +972-153-4-622-7811

On Mon, Apr 27, 2015 at 9:18 AM, Paweł Pekról <notifications@github.com>
wrote:

> Oren,
>
> this will fail due to
> https://github.com/ayende/ravendb/blob/master/Raven.Client.Lightweight/Shard/ShardedDocumentStore.cs#L277
>
> FOund this out, because core tests in v3.5 started failing after merge.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ayende/ravendb/commit/059e9baed602c5b724124525aa8ec31722624522#commitcomment-10909645>
> .
>


I've used inheritance, but the override method's signature needs to
match the one from the base class (including return type) in C++, so I
couldn't find a way.
Anyway, I planned to bring that topic up in a pull request and if a good
solution comes up it will be easy to move the code snippets from the
cases to methods.

On 04/27/2015 08:14 AM, Nathan Woodrow wrote:
> That's fine, it doesn't really make a any difference I just know this is
> the main Martin added a vistor pattern to it. I suspect you would just
> have a internal class that inherits from it and builds up a string
> that you
> ask for when done once accept returns.
>
> On Mon, 27 Apr 2015 at 16:11 Matthias Kuhn <notifications@github.com>
> wrote:
>
> > Yes, I tried. But I only could come up with crappy solutions - basically
> > because the return type of visit is void. If we changed it to QVariant I
> > think it would be possible. But then again it would be hard to keep
> track
> > of the context a node is used in. I don't know if this will be a
> > requirement one day but if yes it will be a limitation again.
> Therefore I
> > am not a big fan of it yet.
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/m-kuhn/QGIS/commit/08e6f669b10a5914b9c9d909d0b21c7a4c2f5654#commitcomment-10909596>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/m-kuhn/QGIS/commit/08e6f669b10a5914b9c9d909d0b21c7a4c2f5654#commitcomment-10909617>.
>


w


머리가 복잡하다 그냥 가만히있어야겠다


This should either go in the beta section or it should be a regular feature!


bugfix


If `out_size` is less than `sizeof(TEE_FS_PATH)` you'd be in trouble here. In the case where `out_size` is larger than `sizeof(TEE_FS_PATH)` `strncpy()` null terminates `out` only because `sizeof(TEE_FS_PATH)` includes the terminating zero of the string.


Sample project to test Stripe, Mixpanel, Mandrill and Twilio


Well this is the wrong assertion to make here. I'd expect a verify on the snakefood library function.


[相談] いろいろ設定してauthorization urlを取得してるだけなので、`executeOauth`ってのは微妙な気がしてます。意味が広すぎるというか。
かと言って、何が良いかと聞かれると困る系ですが。。。
自分だったら、`getAuthorizationURL`にしちゃうかもしれないです。


It's important to remember that this is just the default and can be overridden in the interface:

![PortableGit extraction interface](https://cloud.githubusercontent.com/assets/792695/7342673/50786894-ec6c-11e4-9ddf-a08dfa8d5675.png)


Final commit for WWDC 2015 submission. Not enough time to finish it, but the content is there!


couldn't `perform` be a class method as well? It seems that no real info contained in the object is passed...


RIP. You may have wanted us to forget you, but we won't.


Ar fi indicat ca atunci cand vrei sa expui in exterior fielduri ar trebui sa folosesti propietati, nu sa faci fieldurile publice. https://msdn.microsoft.com/en-us/library/x9fsa0sw.aspx


Sidekiq requires it to be an instance method. I could add
```ruby
def self.perform(*args)
  new.perform(*args)
end
```
to the `CollectiveProofAttemptWorker ` class, but this would only move confusion from here to the worker class, because that one wouldn't completely go with the convention.


What is that 4th commit, "+Subproject commit 0b39a412f5a9c12abd7a70c85ec7662515b7857b" to src/sponge-docs-theme... I'm not sure it belongs here. This artifact turned up in another recent PR too.


plus ca in cazul in care nu este nevoie ca fieldurile sa fie expuse in exterior, ex l.startingDate, atunci nici nu il faci public, il lasi private pentru a putea restrange accesul la ce se intampla in obiectul leave


Click me plz#a <


Modificarile au fost bune si apreciez faptul ca ai revenit asupra temei pentru a rezolva micile probleme. Spor


Yep, i see. I talk there about other WP functions. "Always escape, even if the values come from WP functions."


Use Project Type Service to load available projects' types.


Oren, but in ShardedDocumentStore we do not have access to URL, not following.


[提案]
起こらないであろう例外に関しては、とりあえず `throw new RuntimeException(e);` をおすすめします。
Androidであれば、強制終了し、デバッグログにはスタックトレースも自動で出てくれます。

この状態であれば、もしTwitterExceptionが発生しても、48行目でNullPointerExceptionが発生するか、タイミングによってはそれもすり抜けたりして、落ちたタイミングでは画面上で把握できない可能性があります。（ログを常時監視していれば別ですが）


Magnifique ! =D


fix this


That's true, but that's also a point against the path for class/interface types `T` which returns a `null` reference.

Regarding throw vs nothrow, a similar discussion took place at https://github.com/s-ludwig/std_data_json/issues/5 where I argued for `enforce`. In that case there is also an issue of memory safety so it's more serious, but the same question came up: do we present a throwing interface, a non-throwing interface, or both?

Providing *only* a throwing interface is the safest but also precludes `nothrow`, as well as `@nogc` until we figure out a robust mechanism for allocating `@nogc` exceptions. If we provide both interfaces here, then that begs the question: which of the two gets dibs on AliasThis? Another alternative would be to eliminate the invalid state altogether with `@disable this();`, leaving any uses of `Unique!T.init` as the user's responsibility.

Putting the short-term desirability of `@nogc` aside, my opinion is that `nothrow` is important. `Unique` is a highly generic primitive. Very recently it was pointed out in #3195 that `Unique` (with support for arrays) would be a useful building block even in Phobos itself. We can't use building blocks that don't allow `nothrow`, as it spreads throwness everywhere. That's why I think we should allocate AliasThis to the nothrow interface - `Unique` has to be as generic as we can practically make it.


Mmmm, sexy linebreaks.


sorry @yarbroughw


生存期間が長すぎると思います。
tryの直前か、`throw new RuntimeException(e);`の書き方をしてしまえば、
```
try {
    AccessToken acessToken = ...;
    //twitter.setOAuthAccessToken(token);
    //twitter.updateStatus("【テスト】ログインされました(Fragment実装ver)");
    return acessToken;
} catch (TwitterException e) {
    throw new RuntimeException(e);
}
```
と出来ます。


Never mind, I'll fix this and push it.

*Hibernating Rhinos Ltd  *

Oren Eini* l CEO l *Mobile: + 972-52-548-6969

Office: +972-4-622-7811 *l *Fax: +972-153-4-622-7811

On Mon, Apr 27, 2015 at 9:40 AM, Paweł Pekról <notifications@github.com>
wrote:

> Oren, but in ShardedDocumentStore we do not have access to URL, not
> following.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ayende/ravendb/commit/059e9baed602c5b724124525aa8ec31722624522#commitcomment-10909795>
> .
>


Do I understand correctly that `track()` will automatically age the table when it is full and so the `PcapFilter` app does not need an explicit call to `age()`? (Can you take a look at the way `PcapFilter` uses `conntrack` and see if that is OK?)


At least, in this case you could add this very explanation to the code.


Up until noticing https://github.com/EFForg/https-everywhere/pull/1577 I assumed your commits' correctness. But from that it's clear that your time is far too valuable to read git log, at least some of the time, and so that assumption was a faulty one.

So instead of relying implicit communication, which takes more cognitive resources and time, I'm asking explicitly: Which of https://github.com/EFForg/https-everywhere/commit/9708a60e6c47dcdc5994903141b213deff4993ad and this was the correct commit to revert?


後のことを考えると、何ボタンかを変数名で分かるようにした方が良いかもしれないですね。
`loginButton`とか。


ok .commited


змінні називаються з маленької літери, назви класів з великої


Oh, nice! I wasn't sure if MoonScript had any syntax like that.


I do not understand line 5. Why a second database query? Wouldn't this yield exactly the same object?


Try not to use cb, you can just return a promise.


I like this line.


public void LoginButtonCLick(){  - java code style

public void LoginButtonCLick()    - php, js code style
{


@saily @gforcada are you ok with this? I will do a few more tests today and then make a release. Thanks for your great work!


It's ok, but we should just move this code to spawnMocha, and remove the ability to configure the output (or think of another way.)


any logs? error messages?

2015-04-26 8:42 GMT+03:00 thakorjay <notifications@github.com>:

> I have already built the boost library and gateway code...still it stuck at
> get server info only....
> No device polling is done .
>
> Can you help me with this ?
>
> On 26/04/2015 10:49 AM, "Sergey Polichnoy" <notifications@github.com>
> wrote:
>
> > Hi, all you need is to rebuild boost and the gateway under your platform.
> >
> > 2015-04-25 13:27 GMT+03:00 thakorjay <notifications@github.com>:
> >
> > > hello,
> > > I want to run the same program of gateway on my INTEL PC. Is it
> possible
> > > to run the same ? Or any configuration is needed for this?
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/devicehive/devicehive-cpp/commit/3319387fab4cee5747676aa5bf61924a8ef893cc#commitcomment-10900630
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/devicehive/devicehive-cpp/commit/3319387fab4cee5747676aa5bf61924a8ef893cc#commitcomment-10904152
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/devicehive/devicehive-cpp/commit/3319387fab4cee5747676aa5bf61924a8ef893cc#commitcomment-10904192>
> .
>


好みの問題ですが、
```
new AsyncTask<Void, Void, String>() {
    ....
}.execute();
```
って書き方をすると、変数を減らせます。（つまり、脳内のキャッシュを消費しない）

変数を減らしすぎるとわけわかんなくなるのですが、不要な変数の場合は減らしたほうが脳容量的に優しいです。


It's working, that particular bug was fixed quite soon. ;)
Some stats were recorded for your project : http://buildtimetrend.herokuapp.com/dashboard/pyca/cryptography/index.html (select timeframe : month)


Should start by var deferred = Q.defer();


By the way, what about putting it directly under the repository/ location, in some buildinfo folder?
In the end, it would be much simple to keep the build metadata together with the p2 repo, as they're so strongly coupled.


return deferred.promise. instaed.


:sob: Ahaah, best commit message ever!


also realised that function fs_unmount() is meant to be fs_umount() because apparently the person who made it felt like that one letter made a huge difference :/


:+1: Activityのコードが減って、いい感じですね。


deferred.resolve/defered.reject  instead.


コンストラクタで代入しているので、staticにする必要は無いですね。


This is more elaborate that it needs to be. It can either


`.` should be on the next line.


Use `next` to skip iteration.


I just pushed it, that should resolve this issue.
I already merged to my 3.5 branch and testing it out now

*Hibernating Rhinos Ltd  *

Oren Eini* l CEO l *Mobile: + 972-52-548-6969

Office: +972-4-622-7811 *l *Fax: +972-153-4-622-7811

On Mon, Apr 27, 2015 at 9:43 AM, Oren Eini (Ayende Rahien) <
ayende@ayende.com> wrote:

> Never mind, I'll fix this and push it.
>
> *Hibernating Rhinos Ltd  *
>
> Oren Eini* l CEO l *Mobile: + 972-52-548-6969
>
> Office: +972-4-622-7811 *l *Fax: +972-153-4-622-7811
>
>
>
> On Mon, Apr 27, 2015 at 9:40 AM, Paweł Pekról <notifications@github.com>
> wrote:
>
>> Oren, but in ShardedDocumentStore we do not have access to URL, not
>> following.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/ayende/ravendb/commit/059e9baed602c5b724124525aa8ec31722624522#commitcomment-10909795>
>> .
>>
>
>


Use a guard clause instead of wrapping the code inside a conditional expression.


@bromix I have something for you, regarding the darker blurred fanart (after seeing it in use), done direct from vectors for  ^^resources/media/fanart.jpg**

Option A (darkened and blurred just text)
![fanart](https://cloud.githubusercontent.com/assets/3521959/7342859/be2bb766-ecb1-11e4-967c-a882fe848e83.jpg)

Option B (Option A + Red overlay at 35% opacity)
![fanart2](https://cloud.githubusercontent.com/assets/3521959/7342909/9476570e-ecb2-11e4-80a2-8b3f7f3cee2f.jpg)

Option C (Option A + youtube text @ opacity 80%)
![fanart3](https://cloud.githubusercontent.com/assets/3521959/7342954/41cac6ba-ecb3-11e4-9e70-90c4de038952.jpg)

Yes I agree in this view, a non focus texture works best here.


Cursorもたしか、closeしないといけなかったはずです。


Avand in vedere ca toate fieldurile sunt protected, ca nu ai un constructor prin care sa le initializezi, daca vrei sa faci o instanta de tip Persoana(clasa nu este abstracta asa ca cineva se poate gandi ca vrea sa creeze un obiect persoana) ex: var pers = new Persoana(); (folosind constructorul implicit) nu vei putea niciodata sa schimbi valorile fieldurilor si ele ar ramane cu valorile default.
O rezolvare la aceasta problema este ori sa faci un constructor de genul public Person(string lastName, string firstName, string dateOfBirth) si sa initializezi, fie sa faci niste propietati gen public string LastName{get{return lastName;} set {lastName = value;}}(eu as alege constructorul pentur ca il poti folosi sa il apelezi cand faci constructorul claselor care deriva din clasa Persoana. vezi comentariu de la constructorul Employee)


That's a data migration problem that I talked about last week. In the migrations, we should not assume that we have any methods (but `id`) on any model object. In order to change the data anyway, we need to operate directly on the database. Here we have two options: a) write plain sql and b) use the methods that rails supplies to manipulate database cells. I chose b) because it's easier to read in the rails project, even though it's not as fast.

This second query results from the mechanics of `pluck`:
`pluck` operates directly on the database and returns exactly one column of the table. As far as I know, we can only restrict it to specific rows with `where`, i.e. we can't call it on the object. In order to get the one row that we are interested in (the one of the current `logic_mapping`), we can only select the data this way.

In a later pull request, I add a class that has convenience methods to do exactly this on an arbitrary model.


This is useful and should be applied to the MVC example too.

Even better, the two examples should use [one and the same meteor-webix adapter](https://github.com/webix-hub/webix-meteor-data/blob/master/codebase/meteor-data.js instead of duplicating code. A solution for this would be to include webix-meteor-data as a git submodule.


with this commit, i can not select the candidate. when i press <enter> it just returns normal mode on unite prompt. also <tab> is not working. not showing the actions just blinks the cursor.


コンストラクタでオープンして、各メソッドでcloseすると、インスタンスの再利用出来ないっすね。。
writeUserInfoもですが、別クラスに切り出したほうがいいかもしれません。
コンストラクタでは何もせず、使う前にopenして、使ったらclose的な。


Dupa cum scrie in comentariul de la clasa Person, daca ai fi avut constructor la clasa Person atunci ai fi putut scrie astfel:
public Employee(string LastName, string FirstName, string dateOfBirth, string dateOfEmployment, double salary, uint availableDaysOff ):base(lastName, firstName, dateOfBirth).
Se executa mai intai constructorul din clasa de baza apoi intra in constructorul de aici.(liniile 18, 19, 20 ar fi disparut din constructorul acesta pentru ca ele se executau in constructorul de baza)


I think `Factory.create('alert')` without the document object would suffice here since we defined the factory with random data?


This flag should be set in device trees, rather than the common one. The Z3C is also shinano, but does not use the same resolution.


Should be fixed in 599b0311e073a8e23444b90e2bdb26e3c839e307; the compiler didn't realize that there's no way to leave `get_data_src_dst_mac()` without the pointers being set.


:+1:

I've submitted [a PR against the webix repo](https://github.com/webix-hub/webix-meteor-data/pull/8). Hopefully @mkozhukh will accept it soon.


Hi,

I wanted to understand if there is any option which would keep the session alive for the Login which is created at the time of starting the application.
Based on my reading of the PHP API I saw that there are 2 options for this from the PHP API perspective:
Option 1: SendActiveStatus()
Option 2: SendPing()

Can you please guide me to which one should be used for keeping the Login session active.
Thanks.


ace.min.css


3D graphics isn't really my forte, so there's probably a better way to do it.

I could see through experimentation with the single-node version that it appeared to be applying some parts of the rotation after others, so I used 3 nodes to ensure that each component of the rotation was applied independently of the others.


fieldul acesta ar trebui sa fie de tipul DateTime
https://msdn.microsoft.com/en-us/library/system.datetime%28v=vs.110%29.aspx


This should be `.Any()`, as `Count()` will enumerate the whole collection, whilst `Any()` will terminate as soon as it finds a single object.


`.` should be on the next line.


Ar fi trebuit sa declari un enum, LeaveTypeEnum, care sa aiba valorile Holiday, Medical, Other si leaveType sa fie de acest tip
https://msdn.microsoft.com/en-us/library/system.enum(v=vs.110).aspx


`APP_DATA`以外のstatic変数4個とも、PreferencesのKEYだと思いますが、
USER_EXISTSだけ大文字でスネークケースになっています。
とりあえず統一感だけは意識して下さい。


In general cand cream un tip care deriva din Exceptie ar trebui sa implementam cel putin 3 constructori diferiti. Mai multe detalii aici: https://msdn.microsoft.com/en-us/library/87cdya3t(v=vs.110).aspx


Am vazut ca toate clasele tale nu sunt publice. Trebuie sa fii constient ca daca acestea sunt internal, aceasta este modifierul implicit, ele nu vor putea fi folosite in exteriorul assemblyului.


This should be in a `private const string` at the top of the file, with an appropriate name, rather than inline.
e.g.
```C#
private const string ValidVoterNameRegex = @"[^\w\.@ -]";
...
ballot.VoterName = Regex.Replace(voterNameInput, ValidVoterNameRegex, "",RegexOptions.None);
```


Either all or none of the arguments to a method call should be on a new line.


Poate salary ar fi trebuit sa fie de tip double sau decimal pentru a putea introduce si numere cu zecimale.


staticなメソッドしか外部に提供しないのであれば、privateなコンストラクタを用意して、インスタンスを生成できないようにするべきです。

読んだ人が、「このクラスはインスタンス生成して使うものでは無い」とわかりやすいので。


hhh


@skeptical thanks!


StartingDate trbuie sa fie de tipul DateTime


Yes fine, if all windows are the same - BUT will need a Linux test as well

On 26 April 2015 at 21:02, Ifetayo Agunbiade <notifications@github.com>
wrote:

> we don't really need this, for now. But I will try.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/csm24/M24ChessCode/commit/14408a923fefb3327a01ca31d60177eb254c2c49#commitcomment-10907012>
> .
>

--
Simon Hewitt
(+44) 07584 658227


@lxnay it  seems there was a problem with portage, it didn't take my package.env, so I created a directory path /etc/portage/package.env/ and 00-kogaion.package.env. This way it took the FEATURES="-userpriv" mode
Note this down for the next time someone has the problem with envs'


not sure, but i'd probably cache that require at the top of the file

also how do you clear notifications?


hello.pls give me tips for how to install
i use ubuntu 12.04
can you help me?


Variablia i nu cred ca este folosita la nimic. cand folosesti liste si parcurgi listele cu foreach, atunci nu iti trebuie un counter ca sa stii unde te afli in lista respectiva.


Thanks. I hope, that the next IE will also be available for Windows 7 and Windows 8. Otherwise it will take many years, until every browsers supports ES6.


このメソッドの先頭とかで`finish`しておくと、backstackにこのActivityが残らないので良いと思います。
戻るボタンで戻った時に、白い画面が表示されると微妙なので。


its working now... just permission is denied in normal user...so with super user this is working good....

now i want to use Eclipse or any IDE for the code development and debugging...So is it possible to run the same program??

Give me some suggestion ...How to approach this??


Du ser det inte här i diffen, men i verkligheten står här:

    +<U+FEFF>---

<U+FEFF> är Byte Order Mark som har en tendens att förstöra för Jekyll.
Jag fixar detta i nästa commit men jag undrar vilken texteditor som förstörde för dig (på vilket OS).


master blanchの方は消しといたから、あとでこっちをmasterにmarge（合体）しよう。


OK, do you remember which pull request, which commit?


Что это? :frowning:


Thanks @fblundun! may also be useful to mention that the migration requires re-granting to users on the new table


A `std::map` is also complicated if you look at its implemenation... I think the bucket thing is probably not abstracted properly since this kind of code should not be necessary in the `MapContent` class.

Of course the `IdManager` is probably efficient enough to be used in this case as well.


Mielőtt betenném ezt a javítást, hadd kérdezzem meg: van RMNy jelzet a kézirattárban? Ha van, akkor ma is tanultam valamit, ha nincs, akkor ezt a sort innen ki kellene venni.
HU/B1/RMNy/0342;12500/12524


クラス内かつメソッドの外、では宣言以外できない、って感じなのかね？


あ


]] Richard Warburton

> According to: http://godoc.org/golang.org/x/net/ipv4#ControlMessage
> The Src (in the control msg) is only set when sending out (not a receive). Therefore it won't be set to the destination ip on receive. I think it will default to the interface ip when sending, which is probably desirable.

I dumped the incoming messages, and got things like (local address is .7):

    (*ipv4.ControlMessage)(0xc20809e040)(ttl: 0, src: 172.16.158.145, dst: 172.16.158.7, ifindex: 2)

This is then used unmodified when writing to the socket, leading to
conn.WriteTo erroring with `write udp4: invalid argument`.  I straced
and gdb-ed and discovered that this is in fact due to src then being set
to .145.  Getting gdb to print out the complete cmsg struct is somewhat
involved, so I'm not including that output here.

--
Tollef Fog Heen
UNIX is user friendly, it's just picky about who its friends are


Such an inner class `Reference` appears at various places. Is its purpose documented somewhere?


Aici ai fi putut folosi apelul de constructor din clasa de baza
public Employee(string LastName, string FirstName, string dateOfBirth, string dateOfEmployment, double salary, uint availableDaysOff ):base(lastName, firstName, dateOfBirth).
Asa ar fi disparut liniile 18,19,20 din constructorul Employee


Avand in vedere ca toate fieldurile sunt protected, ca nu ai un constructor prin care sa le initializezi, daca vrei sa faci o instanta de tip Persoana(clasa nu este abstracta asa ca cineva se poate gandi ca vrea sa creeze un obiect persoana) ex: var pers = new Persoana(); (folosind constructorul implicit) nu vei putea niciodata sa schimbi valorile fieldurilor si ele ar ramane cu valorile default.
O rezolvare la aceasta problema este ori sa faci un constructor de genul public Person(string lastName, string firstName, string dateOfBirth) si sa initializezi, fie sa faci niste propietati gen public string LastName{get{return lastName;} set {lastName = value;}}(eu as alege constructorul pentur ca il poti folosi sa il apelezi cand faci constructorul claselor care deriva din clasa Persoana. vezi comentariu de la constructorul Employee)


I've moved it to HA webserver API. I pushed it allready into dev branch.


@jacobmarshall For some reason I must've copied MIT from some random dude's repo instead of from the official site. Weird, no idea why I would've done that.


It's 8862ce1051f1815eec52528b8e6a04ee5955a8a4 of #1327. I assume, I need to add some comments and let the migrations that already use `pluck` inherit from this class. I will do the latter right after merging #1327 into staging.


Gg


ok, we should differentiate real dimension and topological dimension. Here it is only topological except for the 3d case of course.


why fix compilation by the way ?


Nu ai dat commit cu proiectul de tema. In acest repository se gasesc doar cele doua fisiere de ignore


Thanks Theo i will use yours now :+1:


Since these parameters are not overriding equivalent parameters in the base class, `override` should not be set ( `mandatory=False` may also be omitted as that is the default).


good!


I remember that I have documented it in the `doc/` directory, but I can't find the corresponding pull request.


where does `object` come from?


Will fix tonight


It's part of the `ActiveModel::Serializer`, which is the base class of `ApplicationSerializer`. It's the object that's being serialized.


How is it misleading? _BSD_SOURCE was superseded by _DEFAULT_SOURCE, no?


@hira-pa-  早い！確認しました。　遅くなりましたー


Thank you!


@tisto, it was a complete rewrite of the package, only the tests stayed. ok - from testing team point of view it was probably not :laughing:


[PR accepted](https://github.com/webix-hub/tracker/pull/210) on March 10.


// attributes file (.gitattributes), if it not already exists.


The default timeout for `execute` is 30 seconds. Since `seconds` parameter may be set to a higher value, this may time out. Timeout for this command should be set based on the `seconds` parameter (plus some small "buffer").


Update please


else a mesh with dim=1 and realdim=3  return is_3d=true and this function
void updateNumGlobalElements( typename std::enable_if<is_3d<MT>::value>::type* = nullptr ) is call with inside beginPointWithProcessId() which only available when dim=3.

Also I think it's strange to have 2 dimension trues for a mesh when dim!=realdim


In general cand cream un tip care deriva din Exceptie ar trebui sa implementam cel putin 3 constructori diferiti. Mai multe detalii aici: https://msdn.microsoft.com/en-us/library/87cdya3t(v=vs.110).aspx


Thanks you for taking this issue raised by me, seriously, as it was very important on iPad device.


@shuji-mwed 確認OKでございます


Well but std::map is probably better tested :)


Sure, on more recent dists - this branch of the conditional is for EL6 which doesn't have __python3 or __python2 macros.  __python3 is used on line 4, __python2 on line 10 and __python as a fallback here.


As with the other pull request, `override` should not be set, because these parameters aren't overriding anything; `mandatory=False` may be omitted.


Thanks for the pull request! However you need to use `tab` here instead of `space`, otherwise markdown won't render it as a list item.


In .Net exista notiunea de property care contine in ea doua metode, set si get, get-ul facand cam acelasi lucru pe care ai vrut tu sa il faci prin GetLeaveDuration
Mai multe detalii gasesti aici https://msdn.microsoft.com/en-us/library/x9fsa0sw.aspx


Why not

```sh
$(cd / && ls etc/post-install/*.post)
```


multiple definition of 'android::Singleton<qdutils::QCCompositionType>::sInstance'


This code can be written better by using `reduce` or `map`


@zekzekus Please create new issue.  I will check it.


Since we want this only in the end user installations, maybe we should move this file into `build-extra/` and include it in `make-file-list.sh` explicitly -- instead of letting it be part of the package that might well be installed in a setup where the user has removed the tell tale `msys2.ico`.


This will flash a `mintty` window. Probably a better way would be to call `usr\bin\bash.exe -c exit` -- if that works without a flashing window.


Please also note that this will not run if the user unpacks the `.7z` file manually -- as she is fully entitled to. On the other hand, the next time a shell is run, the scripts will be executed. Hmm. Maybe we should rely on that first-time execution, anyway, and simply skip this commit?


Thanks Pak Agung . . . .


Buna,
Nu s-au adaugat fisierele Employee.cs si Leave_types.cs in proiect. Daca te uiti in repository, ele nu apar. Astept sa le urci si imi dai un semn.

Multumesc


As in my comment to the change in the portable installer, this would flash a `mintty` window.


Nu apar in repository fisierele astea desi vad ca le-ai facut.


Maybe we can move this into the livingdocs engine.


Will 30 seconds (the default timeout for `execute()`) always be enough? If possible, the timout should be set based on specfied parameter values. If that is not straight forward, a `run_timeout` parameter should be added, so that the user can adjust it manually if they need to.


The easy way - adding usb to the existing bcm5974 driver does not work. The force trackpad uses bcm5976. The specs for that one would have to be reverse engineered, but I don't have the time. I'll not be able to do it.


The problem is, in V5 we used `Subscope` on address to create the name. As of V6 we use transport-specific implementation of `Subscope` so in theory it might change the name of the counters.


Of course, the `IdManager` uses 32-bit rather than 16-bit and does not reuse them. Hence it's not a direct replacement since the network protocol currently uses 16-bit IDs, which requires them to be reused.

I guess I would prefer to change the protocol to use 32-bit entity IDs.


At the top level, rather than defining seperate regex constants, you could define a dict mappling label to the corresponding `(regex, units)` tuple, you could then do something like

    for label, (regex, units) in regex_map.iteritems():
        match = match.search(line)
        if match:
            context.result.add_metric(label, int(match.group(2)), units)

rather than repeating that code for each regex


__API Blueprint syntax error on line 3210__: Undefined resource model 1039 (Autogenerated message from apiary.io)


:clap:


Fixed, thanks!

You clear badges in electron with `app.dock.setBadge('')`. I've updated this API to be `app.setBadge()` to increment the current and `app.setBadge(false)` to clear: 9d78f4ab395c3f093be86927a5dcaebf8d4e8887


Is this comment intentional?


have a try


The right name should also be runes_improved_heal.png .


done


r+


I'm not sure what the commit message has to do with the diff. Isn't that the reverse?


Thanks ! fixed in #3634


Perhaps @fehguy would have an idea.


you can use any IDE you want. All you need is to provide appropriate
include and lib paths for boost

2015-04-27 10:21 GMT+03:00 thakorjay <notifications@github.com>:

> its working now... just permission is denied in normal user...so with
> super user this is working good....
>
> now i want to use Eclipse or any IDE for the code development and
> debugging...So is it possible to run the same program??
>
> Give me some suggestion ...How to approach this??
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/devicehive/devicehive-cpp/commit/3319387fab4cee5747676aa5bf61924a8ef893cc#commitcomment-10910172>
> .
>


Hi @marmat,
Can you explain, how did you get to this FOV formula?

Thanks. Or


Regarding: Issue #8


ewt


> This will flash a `minty` window. Probably a better way would be to call `usr\bin\bash.exe -c exit` -- if that works without a flashing window.

That'll just cause a command window to flash.  We're pretty much stuck with some window appearing quickly no matter what we do.


@sogimu: that may be interesting for you.


@KozhinM I see that we have changed the logic around seeking to a common time between audio and video start. But I feel it is broken as the segments are downloaded out of order. Say audio buffered range is 0.005-12. And vidoe is yet to download. We seek to 0.005.
2nd segment of media gets downloaded first. Buffered range for video is 8-12. We seek to 8 and start playback.
the firstAppended is per period but it should be for period and type? Only when first segment for both audio and video has been appended should we seek. But then we should check if its not an audio only stream.


Problem is, you cannot just do:
```python
User.query.filter(User.affiliation.name.ilike('CERN'))
```


> Maybe it's time to move the whole ENGINE handling code to its own class, WDYT?

Yes, that's what I had in mind for the next step: supporting git protocol.


Technically you should never get less than P256_LIMBS in the context of ECDSA.


First version


@zekzekus Confirmed.  I will fix it later.


`05-home.post`, which we don't want to include, might still be in there and `07-pacman-key.post` and `08-xml-catalog.post`, which are useful for the SDK but useless for a distribution, will definitely still be in there.


This one cannot extends `GenericJQueryContainer`?


<br>???????


Probably a bit late now, but I'm not too keen on the name `Event`, especially given it's associations in C#. I've not got any better suggestions right now, though.


My bad, I forgot to add the link to the header nav and already corrected it. There was a reason not to add that link on the dashboard, though.
I couldn't decide: (1) whether to present the number of bookings received, pending, or the sum of both; and (2) whether to link to the received or pending table.
By the way, the fact that I was unable to make a quick decision on these points should be strong evidence that those two states (received and pending) are redundant. If we remember that users' contacts are already available on their profile, there isn't much value on having that extra step.
Also, what are you thoughts regarding this new table layout? I really think we should stick to a single line per booking, as multiple lines will be confusing to most users.


> Maybe we should rely on that first-time execution, anyway, and simply skip this commit?

We could do that, but I figured this was a decent way to avoid scaring users with errors on the first run:

![post-install-errors](https://cloud.githubusercontent.com/assets/792695/7336310/beb101f6-ebac-11e4-9e1c-3d86fb0b01e9.png)


I guess the commit message is a bit misleading. The main question is: did we make incompatible API changes?

Given a version number MAJOR.MINOR.PATCH, increment the (http://semver.org/):

    MAJOR version when you make incompatible API changes,
    MINOR version when you add functionality in a backwards-compatible manner, and
    PATCH version when you make backwards-compatible bug fixes.

Additional labels for pre-release and build metadata are available as extensions to the MAJOR.MINOR.PATCH format.

@saily I would say this is a MINOR release, because we added new features in a backwards-compatible manner, right? Please correct me if I'm wrong.


Should be an `IVotingContext`.


Oh god... when I back home I will fix formatting.
BTW. I think Orchard should use StyleCop it is extremely usefull.


bn_wexpand return value is not checked anywhere in the code.
constant time in that regard should not matter, because for the sign case all inputs and outputs supposed to be 256bit.


I'm wondering: if we completely ignore the annotator's class annotations, and just use the parameters from the engine description, will the annotator still work? (I'm not yet 100% familiar with how ruta will initialise the engines...)

If that's not possible, I could swap the two loops so that it:

 1. read the default value from the annotations
 2. override those values with anything from the config file.

Does that sound reasonable to you?

> On 27 Apr 2015, at 08:20, Renaud Richardet <notifications@github.com> wrote:
>
> OK. what about changing it to:
> if a default value for that parameter is present in the UIMAfit Annotator, we use that value
>
> —
> Reply to this email directly or view it on GitHub.
>


This one cannot extends GenericJQueryContainer too? Is it about `List` which is not Serializable?


OK, that would work for an exact match. Otherwise, I need `.ilike(...)` anyway.
Any suggestions?


Helló,
szerintem is várjunk azzal, volt még egy ilyen jelzet, írtam nekik erről.

Üdv
Zsolt

Király Péter <notifications@github.com> ezt írta (időpont: 2015. ápr. 27.,
H 9:22):

> Mielőtt betenném ezt a javítást, hadd kérdezzem meg: van RMNy jelzet a
> kézirattárban? Ha van, akkor ma is tanultam valamit, ha nincs, akkor ezt a
> sort innen ki kellene venni.
> HU/B1/RMNy/0342;12500/12524
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/simonzsolt/MEKReferenceLinkResolver/commit/d25c8cdc2abbdc670af7344e48118ccb22a4b434#commitcomment-10910183>
> .
>


If it returns 0, then it attempts to do BN_mod_exp_mont_consttime. If that also fails then BN_mod_inverse is performed. But that is how it is currently done. If you think it is wrong, it should be fixed in master first.


New Update to what?


remove "style=" and create class.
into css us !important for override bootstrap


Better :

```python
return cls(host, HTTPDigestAuth(username, password))
```
Otherwise potential subclasses of Connection could not use make_connection as expected.


aok once said, that spell_area should be used only in exp >= wotlk


This is just an experiment to see if I can speed things up a bit. Currently, all helper method calls are delegated from `method_missing`, which incurs an overhead.


What's this for?


Please add a `function_exists()` to avoid conflicts if both a theme and a plugin have TGMPA included.


@tisto, of course we did make api changes, because we didn't have a defined API and we merged all methods into classes. So if someone built functionality on 1.x release, it's possible broken with the new OO based architecture.

Thats why i chose `2.0.0`. Aligned to the versioning policy MAJOR.MINOR.PATCH.

My vote still goes to `2.0.0` but if you prefer `1.2` i don't claim about it.


If an extension uses `'includeBlankOption'=>true` the `select.get('value') === ''` check is still needed, we shouldn’t remove it.


Well, looking back to `GenericJQueryContainer`, you didn't mention `Serializable` so... I give my tong to the cat (it's a french expression, I've no idea if/how it can be translated, lol)


@KarishmaBagga, thanks for pointing out, I will take a look.


Setting self.headers is a potential bug. This object can be used by multiple threads at once in some situations. So you may get one request using the headers of another.

It would be far more reliable to just pass the request headers to the _process and _process_include function in the arguments of those functions.

i.e.
```
def _process(self, body, orig_scheme):
```

becomes something like:

```
def _process(self, body, orig_scheme, headers):
```


Current master test/widget-eaw.js: ![test/widget-eaw.js](http://i.imgur.com/A8X82GX.png)


yes, breaking it into two lines would be fine / removes ambiguity.
my guess is that this doesn't truly need to `return true;` but rather it could simply `return;` (just not `return false;`)


结构完美


yeah right. Ok we need to define better naming and documentation for the traits like I said above.


Why do we change `$GLOBALS['TL_CROP']` now? I think this is very confusing. `$GLOBALS['TL_CROP']` would be different depending on `Sysem::getImageSizes()` being called or not.


Better call it `thresholding` or something similar?


I can see why this method is here, but the whole method feels out of place in the base class to some extent. There's nothing in the name or the rest of the class that indicates that there's anything to do with a poll here.
However, if the poll retrieval was centralised to the base class, as we've talked about before, then it might not seem so strange.


That should work just fine.. :o

On April 27, 2015 10:04:23 AM Pedro Ferreira <notifications@github.com> wrote:

> Problem is, you cannot just do:
> ```python
> User.query.filter(User.affiliation.name.ilike('CERN'))
> ```
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/indico/indico/commit/9d74bfec48bf0e9a0529ed99a4903bfbbbb2eeb5#commitcomment-10910687


the description doesn't really add something


You are welcome. Thanks for taking the initiative to start the tabs for
Adventures.

On 27 April 2015 at 05:55, aarongilly <notifications@github.com> wrote:

> Thanks for cleaning this up - I did a more shoddy job than expected.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/robertre2/dominiontabs/commit/c60009987eeea1e2bb92ca4bd9d4e35ae6d37513#commitcomment-10908944>
> .
>


Yeah ilike for the wildcard match

On April 27, 2015 10:10:12 AM Pedro Ferreira <notifications@github.com> wrote:

> OK, that would work for an exact match. Otherwise, I need `.ilike(...)` anyway.
> Any suggestions?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/indico/indico/commit/9d74bfec48bf0e9a0529ed99a4903bfbbbb2eeb5#commitcomment-10910774


Yes. Do you think it would be easier to read if the queue field was not
final and assigned in #init ?

2015-04-26 22:27 GMT+02:00 Julien Viet <notifications@github.com>:

> keep in mind, queue will be used in the context event loop. it should not
> be used outside of this loop (here, constructed -> final field, ok )
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tsegismont/vertx-monitor/commit/9474c08be4a2dcb7443d72f8d2162d3804715de3#commitcomment-10907098>
> .
>


nice, thanks!


and the braces around `x, y` can be too


Cool! I thought that 30% was way too low...


Yep, I don't use erlang.el and find current form more readable (because it takes less vertical space) but I will change that.


This should be an `IVotingContext`.


nice


It would be nice if the comments below reflected the function's arguments change.


*чукча писатель*
я не смотрел в каком порядке подключается, вот и спросил. отлично тогда )


不需要这种评论


> me demander avant

Je [viens de comprendre](https://github.com/sgmap/mes-aides-ui/commit/4493be1216df0631ced14798d87316db94a151bd). Désolé, j'ai appliqué cette màj en batch, elle ne te visait pas personnellement  ;)  FYI, elle est également appliquée sur [`ludwig-ui`](https://github.com/sgmap/ludwig-ui/commit/e681d226f11de79ac49aae694df0149923881ed6).

> l'intérêt de mettre un email qui n'existe pas

Crédit avec affiliation. C'est une personne + une société. C'est assez standard, justement parce que dans le contrat qu'un développeur signe, il cède ses droits de propriété intellectuelle à son employeur. Sa responsabilité s'arrête donc à son engagement au sein de la société qui l'emploie. La gestion des adresses en legacy est de la responsabilité de l'employeur.
Mais si tu préfères être joignable pour répondre du code que tu as publié dans tes missions précédentes, c'est cool  :)


`exception.Response` is repeated rather a lot.


I haven't done this on the RPI, but it should be possible. You need to get OpenCV compiling on the RPI, which I heard isn't easy, but maybe somebody has a good tutorial.

After that, the code should just work, provided the webcam is easily accessible. I don't have a webcam for my RPI, so I don't know how it works. Assuming it's the same for any default Linux box, then you should be able to just use it.

Let me know how it goes.


Oh nvm, didn't read the context... I'm sure it's somehow possible ;) maybe ask in #sqlalchemy how to query like this when an association proxy is used


This is a `JQueryPanel` that extends Wicket's `Panel`.
We could introduce `GenericJQueryPanel<T> extends GenericPanel<T>` ...


更新了


Per the method name, should the parameter not be `HttpRequestMessage`?


Do you mean `this::accept` ? In fact, that wouldn't work either as `#runOnContext expects a `Handler<Void>`.


yes it is, as the commit comment says.


@tisto I guess @saily is right here, anyone doing imports of p.r.codeanalysis code will have his code completely broken.

Maye the user interface remained the same (mostly) but the inner guts completely changed, so I'm +1 on 2.0


what does `mycila-junit` do?


Again commented with no context... Forget my previous comment


Thanks for the review!

2015-04-26 22:27 GMT+02:00 Julien Viet <notifications@github.com>:

> looks good to me.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tsegismont/vertx-monitor/commit/9474c08be4a2dcb7443d72f8d2162d3804715de3#commitcomment-10907099>
> .
>


__API Blueprint syntax error on line 3575__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


unneeded doc imo


神经病


Not a fan of using whitespace as a delimiter.


Nit: indentation


蛇加比


shouldn't it be done in a `finally` block?


@saily you are of course right if we consider classes and methods our API. If we consider the recipe configuration our API, we just added a few new features. plone.recipe.codeanalysis is a buildout recipe, not a library.


Can this not be confusing if the requestPayload contains spaces? Should we not use a delimiter that isn't likely to occur in any of the errors?


别在更新了


@secrettriangle, what happens if you run `$ node test/widget-eaw.js vte`? If it looks okay that means whatever terminal you're using has the same bug as libvte.


IP is not used here?


弄更新倒蛮快的嘛


`String.Empty` is clearer.


Wicket's GenericPanel [1] also doesn't mandate `T` to be Serializable. It is implied :-)

1. https://github.com/apache/wicket/blob/2ad3057d5ef21c511573d9d20f64cde66f55cbd0/wicket-core/src/main/java/org/apache/wicket/markup/html/panel/GenericPanel.java#L28


github里面别人发了一条评论，会自动出来的，不需要刷新页面，你可以试试看。


This chain goes on for a bit. Would it read better on separate lines?


"mycila-junit" can made some concurrency test by running UT on multi-thread

But I don't use it in the project, that's why it is deleted on a another commit


We could lose this line here...


Maybe `User.affiliation.any(UserAffiliation.name.ilike(...))`


inject `GraphJdbcConnector` by constructor rather than hardwiring it here


... and maybe you could add a blank line here


unneeded doc imo


There's no good way around that, unfortunately.  As I mentioned in [my comment over there](https://github.com/elyscape/git-build-extra/commit/cf721dd04f982724fb6462b09eec01669f1de334#commitcomment-10910665), running bash at all will result in a window of some sort flashing on the screen.  Unfortunately, with the installed version, we can't just wait for the first time the user launches bash, as we need the elevated permissions to be able to write to the Program Files directory.  So here, at least, we're stuck.


How can I get the Date object?


This parameter can be an `HttpRequestMessage`.


不高兴试


public int buttom[] = {1,2,3};
ではだめか？


First off, thanks a lot for the pull request!

Now for the comments: It's hard to see the actual changes in this file due to the formatting changes, could you please undo the formatting changes and just apply the actual code changes?

Thanks a lot!


![Uploading ELXI]@PR1M`([`@}AF})_SC.png . . .]()


iTerm2 `node test/widget-eaw.js vte`, this is the same issue I had before updating to latest master. Sometimes that one messed up `|` appears twice in one line, sometimes not at all, sometimes on multiple lines.

![node test/widget-eaw.js vte](http://imgur.com/TBmBCnq.png)


I think I mentioned how to search for any of multiple emails in one of my comments from Saturday - that should provide a good start for this kind of search if the code in my last comment doesn't work (I'm on mobile right now so searching for comments is kind of a pain)


You should explicitly disallow `null` `args` when calling `executeQuery `. Instead of writing boilerplate-y defensive code, just overload `executeQuery` with a single-arg signature.

The for-each loop won't execute for empty arrays, no need to check for emptiness.


`Micropost` を `Micropost.includes(:user)` に変えないとデータベースがかわいそうだよー
http://guides.rubyonrails.org/active_record_querying.html#eager-loading-associations


厳密にはそうじゃないけど、似たような感じでやったのは無理だったよ。
そしてもう出来てるからいいや、ってなってる。ｗ


dead code


The `||` operator will make impossible writing of empty files, because it will treat empty string af `false`. I believe that in this case comparison with `undefined` is more proper way.


s/managed/manages/


No, like the date. And ?

Date & IP are just some useful information to display to users when we found a lock


I agree that there are too many forks of miniSProg (which is why I didn't upload my version with minor changes).  Ideally, we could roll all the desired changes into the scarabhardware build (since that's the "official" github for the hardware, and where their documentation points to).  I'd guess that they don't have any desire to develop/maintain it though... but I think with these few minor changes, it should be a pretty stable build (except for future features, like selecting miniSpartan6+/miniSpartan3, etc).


unneeded doc imo


Shouldn't the name of a field type be part of the public API ?
Reasoning: the name of the field type is not strictly related to the form. For instance, we'll need it when we want to display a content type where we'll be outside of a form. We might (should IMHO) expose that in the REST API as well.


Oddly enough, `user-select` doesn't work in Chrome. There you have to specify `-webkit-user-select`.


After moving it around a bit:

![worse effect after moving around](http://i.imgur.com/LoeeJRF.png)


do call with null. either overload `executeQuery` with a single query or pass a `new String[0]`


s/remove/removes/


Ahh, it is inside of %else part. I overlooked that.


> What does erlc do in that case?

It simply complains that it cannot find the parse transform.

> Still, isn't the current filter too broad, and what's wrong with applying foo_transform to all of src/*.erl except src/foo_transform.erl? Is that a bad idea we should disallow?

I really think that erl_first_files should be compiled before any internal parse transform because you explicitly told rebar they should come first. On the other hand you don't specify any compilation order for (internal) parse transforms, that one is only implicitly derived from application logic. Also note that if you have more parse transforms in erl_opts they also cannot be applied to each other (because at least one of them is not yet compiled).

I believe my code implements exactly the rules stated in the above paragraph.


Right, its a *Panel* :) I will try to awake this morning...


s/liquid graph/Liquigraph/ ;)


Hi @pocallaghan, I'm original author of the pull request that added this line of code...

http://tiswww.case.edu/php/chet/bash/bashref.htm

See the section entitled **3.5.3 Shell Parameter Expansion**

which states:

${parameter^pattern}
${parameter^^pattern}
${parameter,pattern}
${parameter,,pattern}

This expansion modifies the case of alphabetic characters in parameter. The pattern is expanded to produce a pattern just as in filename expansion. Each character in the expanded value of parameter is tested against pattern, and, if it matches the pattern, its case is converted. The pattern should not attempt to match more than one character. The `^' operator converts lowercase letters matching pattern to uppercase; the `,' operator converts matching uppercase letters to lowercase. The `^^' and `,,' expansions convert each matched character in the expanded value; the `^' and `,' expansions match and convert only the first character in the expanded value. If pattern is omitted, it is treated like a `?', which matches every character. If parameter is `@' or `*', the case modification operation is applied to each positional parameter in turn, and the expansion is the resultant list. If parameter is an array variable subscripted with `@' or `*', the case modification operation is applied to each member of the array in turn, and the expansion is the resultant list.

By using this pattern I was aiming to make the Y or N reply to the question case insensitive - if this is not working on a Mac we can try another way... However I don't have a Mac to test on, so perhaps you could try some variations and find one that works?


I have started this PR on the code of issue #10, so with the logging system.
This log is very important because it's a quiet exception.

But for now indeed it's a dead code


Nit: we don't need this whitespace


@pixeltrix I'll take a look at the `nil` array. However, I don't think we should check for `supports_foreign_keys?`. `add_foreign_key` is a no-op for adapters that don't support it. This is to address users testing for example with SQLite and deploying with Oracle or PostgreSQL.

There is also the use-case of applications supporting multiple database backends.


Why don't we expose the exception description, Id etc.. instead of showing a static message?


s/manage/manages/


Could we have a slightly more descriptive name, please?


Interesting. Still an improvement over the last screenshot, but I think I see what's wrong.

The first line has a combining character (it's the only line that has that). Look closely at the first `s` in that line. There is a glyph after it: __this is an incorrect rendering on the terminal's part__. The glyph after it is actually a zero-width combining character which is supposed to overlap with the `s`. It looks like iTerm2 is __worse__ than libvte - not only does it get surrogate pair double-width characters wrong, it also cannot render combining characters. We should really start posting issues on the iTerm2 issue tracker.

NOTE: When I say surrogate pair, I really just mean "higher than the BMP", since not all utf's have surrogate pairs.


I have now realized that we are commenting to the outdated commit. Please see 43fc271b.


bra jobbat


Cool!

Small suggestion: maybe consider moving the imported interfaces (`IReservedSpace` etc.) into the API package? I've run into some trouble with mods referencing internal stuff from their API before, is why I'm asking.

I've had another look through the API, but I don't think I missed a commit: the other thing we were talking about was a dynamic way to provide content (i.e. something like a `PageLoadEvent` with the page name/path in it, and a string list representing the page content that'd be populated by a handler). Or just something new to be registered, like the interpreters. Is there something like that, yet, and I missed that? If not, that'd be really helpful to allow re-using existing documentation.

Oh, and one more small suggestion: maybe change the `interpret` method to include the name of the page? That could make things a good bit easier to detect, because one could just go by file ending, e.g. (unless page names don't include file ending?)


Do we really, really need those files? `/dev/shm` cannot really be used, anyway... the links cannot be used by Git, either...


document this on the method Javadoc instead


Fair enough. Could you include the information in the commit message?


I'm not convinced about writing Javadoc for private methods. They're usually short and highly focused, i.e. self-explanatory/


Is a source code of DLLs (lib) here?
Without using a commercial project!


Looks good to me; r+ with nits addressed.


@nfeske, @chelmuth: please add to staging


[PRaven 4/27] Solved Remove spaces


Rather than doing null checks whenever we want to use the eventHandler, would it make more sense to perform the check in the constructor, and set a handler which is just a no-op if we get given a null handler?


`$locationProvider.html5Mode(true);`
https://github.com/adaptive-learning/anatomy-tagging/blob/master/anatomy_tagging/static/js/app.js#L37
https://docs.angularjs.org/api/ng/provider/$locationProvider


Cyclomatic complexity for cast is too high. [10/6]


Separate every 3 digits in the integer portion of a number with underscores(_).


阿道夫


yep, just saw it :) my bad


So, to date, the most accurate demonstration of correct unicode width calculations are in xterm and rxvt-unicode. xterm can't render all the glyphs, but it gets the width right. rxvt-unicode is still the _only_ terminal I've seen that can render all of unicode correctly.


>while I figure out what the real message is

Pretty sure it's fail and not blocked


`EventHandler` isn't the best term for this, as it invokes a different meaning within C# code (i.e. a handler for a C# `Event`).


Goodness me, that sounds gnarly! You are a brave man :) I'm hoping you'll be able to offer a very wide array of font properties in PD, such as gradient outlines. Text in image apps has always been slightly weak, it's obviously not an easy thing to do.


I'd probably rather these parameters were extracted to variables, as it's hard to glance at it and see what it's doing.


Should be an `IVotingContext`.


na to jsem asi taky pak došel, ale říkal jsem si, že nemá cenu tam dávat závislost (musel by se přidávat ngRoute, že?), jenom kvuli této věci. Ale zas by se to měnilo bez reloadu, to je fakt.


Этот синтаксис есть только в седьмой джаве. Идея собирается с шестой.


I changed it to thread_id.


I think you can easily hit a race condition here, you can test by launching several Liquigraph clients at once (you can do this with a [CountDownLatch](http://docs.oracle.com/javase/7/docs/api/java/util/concurrent/CountDownLatch.html) I think).


while the test data only includes one or two arcs in a line, it will actually be possible to use any number of arcs.  I've introduced a test case that includes four arcs.  I guess you need to
"walk" along the line until you get to the quote mark, or unless you get to a point
where there is just one remaining token.


흠..


Thanks. Let's discuss tomorrow.


Alles klar. Dann machen wir eine Draufsicht und es bleibt dabei, dass der Spielcharakter immer im Zentrum ist?!?


use `angular.element`, please (see discussion in #2914)


Thanks for your investigation on this.  I've seen that error message too.  Rather than setting the src to either 0 or broadcast, would you have any objections to setting src to nil (always), so that it picks up the interface's IP?  (I'm against the client being able to specify our src, I'm also not sure about setting a zero ip.)  Thanks again.


Perfect, thanks Marco for the speedy fixes!


Favour Java 7 [`try-with-resources`](https://docs.oracle.com/javase/tutorial/essential/exceptions/tryResourceClose.html) over the "old" `commons-io`.


same as above


s/return/returned/
s/by//


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


this sometimes seems to lead to add the clear button more than once - you could check in `addClearButton`, whether or not that button has been added, or delete it before adding it.


Nice one ;)


@amirm3hdi Welcome to dev <3


Тоже самое -  BufferedReader только в java 7+ реализует интерфейс AutoCloseable.
И несмотря на то, что это пример из референса -- совсем нечитаемо, лучше объявить до try, инициализировать в try и закрыть в finally


- Added more information on About page
- Added link to upload page on the nav bar
- Added CSS styling on upload page
- Removed image upload functionality from image upload modal
- Changed upload button under 'User Gallery (Home page)' to a link styled as a button
- Upload link on nav bar brings up a modal saying the user must Sign In or Sign Up if they want to upload


Hi Martin,
is there a reason why you not cherrypick it back to OTRS 3.3?
This would be also a good improvement/bugfix for OTRS 3.3 in my eyes.

Best regards,
Rolf :-)


```java
@Rule
public ExpectedException thrown = ExpectedException.none();
// [...]
@Test
public void parses_schema_validation_changelog_without_author() {
    thrown.expect(IllegalArgumentException.class);
    thrown.expectMessage("Schema validation failed on changelog <changelog/schema/changelog-without-author.xml>");
   thrown.expectMessage("author");

    parser.parse(classLoader, "changelog/schema/changelog-without-author.xml");
}
```

Isn't it equivalent or am I missing something?


@Peco - could you add support for export to REP?  I expect that frequently users will wish to edit the SensorArc in Debrief before exporting it back to REP format.


@jj-lucas - Keep in mind for the future, that we don't use ```:``` or ```;``` in our stylus syntax


Use `described_class` instead of `SavePosition`


Thanks for the patch @rchikhi


Would `HandleErrorEvent` be more appropriate?


`EventHandler` already has specific connotations in C#.


@floriank can't you just add this to the HTML markup?


switch тоже работает только для char, byte, short и int


As discussed previously, we don't know that there will only be one or two beams supplied.  So, I guess we need to be able to provide a series of arcs, or add an "addArc()" method to the class.


Why nth-child(4)?


Do you need click here?


I agree and use `dependencyManagement` whenever a dependency is spread across several modules.
Before that happens, I find `dependencyManagement` quite verbose.

This is minor anyway, we can agree to disagree :)


I think this can still be `static`?


alright, I get the "moving part", but it seems the XSD itself has changed, or?


Woo!


Consider this largely (read: entirely) untested, but filled with confidence.


What my custom version of `freemind.el` differs from the original version is just the map version number, which keep in pace with the most recent Freemind version. Different version number cause Freemind to report an error, and corrupt the file when it tries to "convert" the old one into a new format.
Simply change the map version number or disable the autoconvert function in Freemind solve the problem.

From now on, all of the files/functions that I manipulated for my specific usage will be added a suffix `_amt`, not sure of the license problem, the license will be leaved there 'as is'.


Thomas, this should at least pop up a major warning, as it is exposing the seed, deviating from Trezor mode of operation.


The Telenor Internet APN is incorrect. Should read mvno_match_data="Telenor DK". Same for Telenor DK MMS


IRC discussion summary: Next release will be 2.0. I don't want us to waste energy on this. A major release won't hurt. I will just document my reasoning behind a 1.2 version number for future versioning discussions:

semver.org says you need to first define a public API and then do versioning according to that public API. The public API for a buildout recipe (if the recipe itself is not meant to be extended) are the recipe configuration options. We did not change the recipe configuration options in a backwards-incompatible way. Therefore a minor release would be ok. Though, semserv is about developers expectations and it seems most people vote for a 2.0 release. Therefore we will go with 2.0.


`Where` isn't required, as the predicate can be passed to `SingleOrDefault()`
i.e.
```C#
Poll matchingPoll = context.Polls.SingleOrDefault(p => p.ManageId == manageId);
```


Revert: https://github.com/plone/plone.recipe.codeanalysis/commit/b43c7cfc98c10068cc385387a5701b2cf58c4e48


error: multiple definition of 'android::Singletonqdutils::QCCompositionType::sInstance'

note: after this patch you have to run 'make clean'


Shouldn't StandardResultPagination be where the rest of the API is?


I suspect the theme for sponge has been updated past what we are currently using.


Should we not fix PollByManageId and prevent a loop, rather than just avoiding it and hoping we don't reintroduce it again?


Tu as supprimé l'image de fond ?


Honestly, I have no idea.  We could probably leave out `06-windows-files.post`, but the script that creates `/dev/shm` and `/dev/mqueue` (`01-devices.post`) is also the script that creates `/dev/fd`, which is necessary for Bash process substitution to work.  It also creates `/dev/stdin`, `/dev/stdout`, and `/dev/stdout`.


Short question: Why did you remove the directory mappings? This does not work with a fresh Composer installation..


Sure.


This sounds like a C# `Event`, rather than something that's happened (if that makes sense?).


@MichaelBurgmaier Isn't that the year of death?


That's too bad. I was using your container for local testing. Thanks for the work you put into this.
Did they say why they were concerned?


sadfsdf


Should be an `IVotingContext`.


This should really go in /usr/local/bin, right? I.e. the package manager should own everything in /usr/bin


yep
```ruby
return unless project.valid?
```


What happens here if docker fails to run for this user? (End up with empty BEFORE?)


Do we just use the string messages to distinguish types of events in the database?


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.334](http://teamcity.monogame.net/viewLog.html?buildId=6378&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 484, ignored: 6 Build time: 00:14:50


my commit


Should this possibly be `validate`?


@bennibu Für mich persönlich ist disk space nicht sooo entscheidend. Ist mir also ein bisschen egal. Inwiefern drückts bei euch / den anderen?


And what then happens here when BEFORE/AFTER are empty? (I think it explodes?)

Can you quote here to avoid this?


Hmm. There is no way to let `.7z` pack the special files (because that's what they are, really) directly, is there?

Alternatively, is there an option to the `Exec` function to suppress the command-line window?


You're welcome


Makes sense.  Will change.


Good spot, thanks!


use first-child


I hope this way it will be clearer... I can't test it now, though. I will do it this afternoon.


You're right. Notably, Boot2Docker does not come with bash.
The script is updated.


I like **option A**, option B is little bit to dark for me...and option A compared to option C shows the YouTube Logo nicer...
If nothing speaks against it, you can make a PR with **option A** to the master branch I would like to use it the version **5.1.1**


- Corrected sentence structure on 'About' page


Fixed in f625f07cfe2cb0600a6bcfbf51ceaf0473341809


There's no `null` check here, so this can blow up.


I have tried it out and it made the error less frequent but didn't remove it completely.

First I thought it was related to fabrication with a block and when it overrides certain things but even the simplest fabrication gets the issue sometimes. A class with one attribute set in the Fabrication and Fabricate without any arguments still got this error (attribute was not set, `nil`).

I wonder if it is also some part of the code that is related because I have some parts that is using `fabrication` but never got this issue even with the latest version.

I will see if I can find a reproducible isolated test-case, or if this is just considered to not be the intended use case for fabrication?


to impl.


@Natashfine


I see, makes more sense if you want to use it that way. How about making it `if (!condition) assert(false)` though to ensure check is kept in release mode?


`SingleOrDefault()` will take a predicate, which removes the need for the `Where`
i.e.
```C#
Poll matchingPoll = context.Polls.SingleOrDefault(p => p.UUID == guid || p.ManageId == guid);
```


Typo in commit log: s/13/18/


 Hi, @regit

I reordered in [devel2](https://github.com/chamaken/ulogd2/tree/devel2) and tagged ``netflow9''. Should I send the patchset to you by email?


I'm not a fan of ternary statements in general, and doubly so in returns.


= not needed


I think you need to chmod that xml


Should be an `IVotingContext`.


ps: the redsocks shell script does the same ;-)


I want to recognize if the user has labels from Personal Labels category.
Do I have to run a loop on all the labels for each document? Is there any processing power-efficient method to achieve this goal?


read what?


Yes, pokeaim told me that, too. Feel free to fix it.


Why I cant's see this pull request through codefresh?


i logged in with verchol?


I'm just saying this to inform you. UTCDate has getDate() method. I will add unit tests for getDate() method as soon as possible.


ggdgdfg


why?


I can, then again, this will also influence the `my_project_page` plugin i guess - also, the issue is everywhere for now.


gata :) le am adaugat

2015-04-27 10:47 GMT+03:00 anitzapredescu <notifications@github.com>:

> Nu apar in repository fisierele astea desi vad ca le-ai facut.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Pitis1991/tema_Victor/commit/84b144d4bf67174e9481aed82566b8b933cdcea1#commitcomment-10910446>
> .
>


Could this be behind an option as well? A lot of noise in the terminal


Ok I see the idea.
Maybe to resolve this need, we could add a method to add an instance (and maybe a list of instances) at initialization instead of listeners.
Something like :
```java
initializer.setInstanceForObject(3, deviceInstance);
```
The listener is still necessary but maybe for observe needs (see https://github.com/jvermillard/leshan/issues/61#issuecomment-93990832)

Does it sounds good ?


I know. We need to fix this in multiple places. I’d like to find a better fix for https://github.com/zurb/foundation-apps/issues/575, but won’t have time to dig into the issue until later this week (at the earliest).

On 27 Apr 2015 at 11:13:43, Florian Kraft (notifications@github.com) wrote:

I can, then again, this will also influence the my_project_page plugin i guess - also, the issue is everywhere for now.

—
Reply to this email directly or view it on GitHub.


Partly, yes.
But when you check `man setenv`, _DEFAULT_SOURCE isn't mentioned.
```
SYNOPSIS
       #include <stdlib.h>

       int setenv(const char *name, const char *value, int overwrite);

       int unsetenv(const char *name);

   Feature Test Macro Requirements for glibc (see feature_test_macros(7)):

       setenv(), unsetenv():
           _BSD_SOURCE || _POSIX_C_SOURCE >= 200112L || _XOPEN_SOURCE >= 600

```

So it is absolutely unclear why _DEFAULT_SOURCE would be needed for setenv.
(Defining _DEFAULT_SOURCE will cause _POSIX_C_SOURCE being defined appropriately. But to know about this, you have to study /usr/include/features.h of a very recent Linux distro.)


Just for my understanding, how is this helpful in container formats?


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


Add gitter badge


Webstotrm 9.0.1 generated files?


I'm not sure that having `EventType` be a `string` is the best idea.


This is a hack. Look at spell 39291.


This is a `DateTime` that we're storing as a string. We should probably be explicit about the format we're using.


> Hmm. There is no way to let `.7z` pack the special files (because that's what they are, really) directly, is there?

It *might* be possible by falling back to the command line, but they need to be marked with the system attribute and I don't know if 7-zip supports that.

> Alternatively, is there an option to the `Exec` function to suppress the command-line window?

Turns out there is!  Will add that now.


ça a jarté, tu l'as remis qq part ?


Same as my previous comment.


I don't still understand how m_clDeviceId is released in OpenCL1.1
As I understand, OpenCL1.1 can create only one clDevice, right?
If so, what's happen if user deletes webcl object. clDevice is released automatically? or leak?


Should be an `IVotingContext`.


There's always an empty string there. One of the reasons why we have to decouple this from Selectize's implementation. Fixing this now requires additional Backbone code, but we're getting rid of it, hence this weird method and a note above it.


M-am uitat pe tema ta. In afara de implementarea exceptiei a fost bine implementata dar nu complet, nu prea am alte comentarii. Keep the good work!


не использует поля класса -> можно сделать статическим


It would also be great if it created the directory here if it doesn't exist instead of failing.


This is very seldom what you want to do. Either you should fail on the sql exception (like you did before) and see if the container cares, or then you should just handle the exception here. Logging exceptions like this should only be done for troubleshooting purposes.


Yeah you're right that was probably better - reverting


Hi Rolf, just that I didn't consider it as an important fix so far. I'm open.


повторно создается myRelatedCharts (первый раз инициализируется при объявлении класса)


var declaration not required


I believe this line may be your problem. What I tested:

```js
const bigArray = new Array(Math.pow(10, 8)).fill(0)

function concatenate (array) {
  return array.join(',')
}

console.log(concatenate(bigArray))
```

What I got:

```
FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - process out of memory
Abort trap: 6
```


I think as an owner I get notifications anyway (if not, I can configure it from the panel) :D


Really not sure on the naming of `Event`, as at first glance it's not obvious that it's distinct from C#'s events.


> > Alternatively, is there an option to the Exec function to suppress the command-line window?
>
> Turns out there is! Will add that now.

Perfect! That is most likely the easiest workaround.


[image: Inline image 1]

On Mon, Apr 27, 2015 at 4:27 PM, KanonTang <notifications@github.com> wrote:

> 不高兴试
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kanon1109/PureMVC-csharp/commit/0b9ba869d541eb901934515189d891b354ea8810#commitcomment-10911052>
> .
>


thanks @daliwali for testing.
The same with `_.reduce` does not suffer the same problem?


Новый коммент


what if it's nil?


我是文盲，fix


:+1:
```ruby
next unless project.valid?
```


@ndeloof why you merged without PR and review from maintainer while there are other PRs related to this topic? For example https://github.com/jenkinsci/sectioned-view-plugin/pull/7 is not mergeable now


@phamann its seems to be the block of code above where we need channel adding to as we're still not passing this through.

 s.linkTrackVars = 'events,eVar11,prop41,eVar43,prop43,eVar44,prop44,prop9,channel';
            s.linkTrackEvents = _.values(events).join(',');
            s.events = event;
            s.tl(true, 'o', eventName || event);
            s.prop41 = s.eVar44 = s.prop44 = s.eVar43 = s.prop43 = undefined;


I don't see any use of backgroundColor beyond this point.


Does this allow for nested lists of Records as well somehow?

For example could go from:

```js
something = {
  title: "foo",
  fields: [
    { value: "foo", isFocused: true },
    { value: "bar", isFocused: false }
  ]
}

new SomeRecord(something)
```

And have it auto-cast to:

```js
SomeRecord {
  title: "foo",
  fields: List [
    FieldRecord { value: "foo", isFocused: true },
    FieldRecord { value: "bar", isFocused: false }
  ]
}
```

If so, how would `SomeRecord` be defined to make that happen?


Hmpf, you are correct, I completely forgot that the extra files are added differently.

But I really do not like to use the `msys2.ico` file as a tell tale. What do you think about this test instead?

```sh
test ! -f /ReleaseNotes.html &&
test ! -f /README.portable ||
rm -r /etc/post-install
```


Yes, it is fixed. Thanks.


it will throw 500 if you don't provide role to position


The activity itself doesn't get removed from the DB, only from the sequence. Should it?


Here you go Usman! Have to use both names to avoid weird formatting


я бы добавила какое-нибудь разумное предупреждение в случае если у пользователя не установлен matplotlib


checking backgroundColor == nil seem confusing here now because we are checking if backgroundColor is solid right? May be change this to dedicated BOOL foundSolidBackgroundColor = NO and replace the `break` with foundSolidBackgroundColor = YES


This formatting string is wrong.


@KostyaSha conflicting PR by nature will prevent other to get mergeable


That would work, sure.


The only thing to note is that still won't apply for the portable distributions.


Manual page feature_test_macros(7) on my Fedora 22 Beta 3 system has this:

```
       _BSD_SOURCE (deprecated since glibc 2.20)
              Defining this macro with any value causes header files to expose
              BSD-derived definitions.

              In glibc versions up to and including 2.18, defining this  macro
              also  causes  BSD definitions to be preferred in some situations
              where standards conflict, unless one or  more  of  _SVID_SOURCE,
              _POSIX_SOURCE,          _POSIX_C_SOURCE,          _XOPEN_SOURCE,
              _XOPEN_SOURCE_EXTENDED, or _GNU_SOURCE is defined, in which case
              BSD  definitions  are disfavored.  Since glibc 2.19, _BSD_SOURCE
              no longer causes BSD definitions to be preferred in case of con‐
              flicts.

              Since glibc 2.20, this macro is deprecated.  It now has the same
              effect as defining _DEFAULT_SOURCE, but generates a compile-time
              warning   (unless   _DEFAULT_SOURCE   is   also  defined).   Use
              _DEFAULT_SOURCE  instead.    To   allow   code   that   requires
              _BSD_SOURCE  in  glibc  2.19  and earlier and _DEFAULT_SOURCE in
              glibc 2.20 and later to compile without  warnings,  define  both
              _BSD_SOURCE and _DEFAULT_SOURCE.
```


a lot of calls to database here, maybe stub some stuff instead?


Does this condition colorsWithAlpha.count == 0 needed? I first color is alpha it will go to else block and if the next color is solid it won't reach this block due to the fact that colorsWithAlpha is 1.

Is this the right behavior?


Jag använde vanliga notepad i Windows. Får skaffa en ordentlig editor
istället antar jag.
Le 27 avr. 2015 09:21, "Erik Bjäreholt" <notifications@github.com> a écrit :

> Du ser det inte här i diffen, men i verkligheten står här:
>
> +<U+FEFF>---
>
>  är Byte Order Mark som har en tendens att förstöra för Jekyll.
> Jag fixar detta i nästa commit men jag undrar vilken texteditor som
> förstörde för dig (på vilket OS).
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/Code-at-LTH/code-at-lth.github.io/commit/350f0b34613464f8596203ce5dfb883fab0ba82f#commitcomment-10910173>
> .
>


`Value` is the new value, but it's only obvious when looking at the code.


Missing ```nps_reg_or(net_priv, REG_GENET_RX_CTL, TX_CTL_ENABLE);```?


Hi, your code just help me a lot ,but i still have many questions, can I have your email address or other contact information?my email address is jelllw1024@hotmail.com. Thanks a lot!


Hmm, still not sure this is the problem. The delta between versions 0.6.3 and 0.6.4 is small, so maybe by process of elimination this is the problem. You'd only run into this problem with massive arrays.


Change made.


I like the `enum`, but I don't like using the `enum` name as the value in the database. In fact, using an `enum` is confusing, as it kind of implies we can rename the values in code and it won't break anything.

A static class with `public const` properties is more what we're using this as.


@ndeloof have you reviewed other PRs? I see no comments, does your change superseed other?


Scipy конвертит разреженные матрицы в плотные, чтобы выполнять с ними операции


Change made to the regular install script.


What else is using this? It should be `protected` only.


As above.


`Closes:` is for the Debian bug tracker. Can you change this to `LP:` so that it refers to the Launchpad bug? Not a blocker - I can fix manually to upload the SRU, but please commit a fix here for Vivid+1.


Should be a separate commit, as it's not really relevant to adding metrics.


the do-block should still probably be indented - looks like everything from here to line 77 is one level less indented than it should be


yandaki eskisi. altinda daire olan urunun kendisi


@afc163 我认怂！


It's ok as we're currently just using the `enum` as a list of strings, but we should be explicit about the values of the `enum`.


@hirtie-maxim wrong condition


@dikkeduif yes. I got an email from their lawyer. While they did not particularly tell me to remove the prebuilt image from the docker index and wanted to resolve the issue over a skype call. Since they have not yet contacted me, I proactively removed the image from the docker index because the image on the docker index technically redistributes wowza.

The project in source code form does not redistribute anything as it is just a build script and gets the packages from wowza servers as-is.

However, if they still have a problem with the project, I will delete it from github. I have nothing to gain or loose from this project. Its just something I put together to ease up my demonstration and test setup and thought others could benefit from it as well.


@afc163 npmjs 什么鬼，找个404模块都不行，后来还是问了谷哥哥！


Bonjour, dude (ette ?)

You need to remove the database name (mangos1) from last UPDATE statement:
UPDATE mangos1.creature_template SET AIName = 'EventAI' WHERE Entry = 18797;

Also, it's advisable to include a DELETE statement for anything new being added, just in case of errors, such as the record being added already but then the user having to rerun these scripts.
e.g.
DELETE FROM creature_ai_scripts WHERE id = 1879701 AND creature_id = 18797;
INSERT ...

I hope that is clear :)

Adieu


Moin vowstar,

I wonder if we can use rodata even more.
Do you think it would be useful to add a hash table <hash,ro-string> to rodata
that makes fast lookup of existing ro-strings possible.
This can than be used when creating an lua string from *user* provided string
to use an existing ro string containing the same chars instead.
In java we call that an "identity map".

Cal


Multumesc!

2015-04-27 12:24 GMT+03:00 anitzapredescu <notifications@github.com>:

> M-am uitat pe tema ta. In afara de implementarea exceptiei a fost bine
> implementata dar nu complet, nu prea am alte comentarii. Keep the good work!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Pitis1991/tema_Victor/commit/2c4dfa87348fdebe4e404b065f29c5914a2fcbfc#commitcomment-10911918>
> .
>


溯妹妹


lineSeparator появился в System только в 7 java


@alexcrichton whoooooooops left in a debug print! Who r+'d that? ;)


This is also an issue for Debian 8


Fine :-)


crossreference: https://github.com/niklasso/minisat-haskell-bindings/issues/6


Please no.


AFAIU, current behavior checks `.ptr` only.


Third arg should be the moduleJSRef.


Maybe also mention `std.range.empty`


Markdown supported


Third arg should be module instead of method.


The old line looks like it was a mistake introduced when it was converted to use assertInstance. I think that splitting it into two assertions as I have done is correct.


@mathiasbynens, it seems there’s no use for these, so should I remove them or move to `opera-30`?


The more i look at this test the less sense it makes... Guess I need to have some actual good tests


module instead of context


I would do it like that:
```
validates :name, presence: true, message: 'mój własny message'


Fragment (consider replacing `.` with `,` or using parens). But I believe the main complaint is that non-null empty slices are true-ish.


Is this comment really necessary?


(Andrew Connolly) TestBranch
test stash


(Scott Daniel) Feature/CATSIM-134 variability unittest
This branch adds a unit test (testVariability.py) that tests all of the applyVariability methods defined in Variability.py on cartoon data.  It does not require access to the database to run.  In theory, this will alert us if something we do breaks the API for one of the variability methods.

* added comments in testVariability.py flagging the odd database schema
for BHMicrolensing and Amcvn events
* only print the difference between the baseline magnitude
and the variable magnitude to the test catalog
* make the DB maker methods more uniform
* remove unneeded variable
* clean up tests
* added helpful comments
* removed trailing whitespace
* removed unnecessary import
* improved readability
* un-commented clean-up line


(Scott Daniel) Hi Heather,

Would you rather I sent this review to someone else?

Thanks,

Scott


(Heather Kelly) Hi Scott, I'm happy to try - I just saw this once you poked me :) This would be my first attempt to review something in the LSST-world - is there a "guidelines for reviewers" doc for the sims group?


(Scott Daniel) hmmm.... Stash should have emailed you as soon as I assigned you as reviewer.  I will ping Iain Goodenow to figure out why that did not happen.

Anyway: Thanks.

There is not a sims-specific guide for reviewers.  We try to follow the style guidelines

https://confluence.lsstcorp.org/display/LDMDG/Python+Coding+Standard

but rules definitely get bent.  You would certainly not be unreasonable to ask for adherence to these standards (and, to be honest, it would probably be healthier for the stack if you did).

Ultimately we are looking for readability/understandability and the code not crashing (though we have the buildbot to enforce the latter).

Does this help?

thanks again,

Scott


hey @blaines i am super excited to see this work. Can we / did we run and publish and see if the artifacts go to rubygems.org and bump the version? I checked out rubygems.org but didn't see the new versions but I think if we can do that that'd be great


(Scott Daniel) Hi Heather,

Do you think you will be able to get to this review sometime this week?  There's another API change that I would like to make to the variability models, but I'd rather not do it until this unit test is checked in.

Thanks,

Scott


(Heather Kelly) The changes to this file all seem to be whitespace - nothing jumps out at me as wrong with that, I just wanted to understand if that was intended.


(Heather Kelly) I see this line in diff, but if view the "full source" in Stash or download the changes and view the file, I do not see this line..   am I crazy? :) I'm just wondering why that is.


(Heather Kelly) The added comment blocks are appreciated!


Wouldn't it be best to prepend `bestaudio/` to whatever the user had set the `format` to? Or maybe derive the `〜audio` variant of the first option the user has set.


(Scott Daniel) This is a change that was merged into master after this branch was created.  It exists in master now but did not exist in master when I was writing this branch.  It will appear after everything gets merged.  Give me 10 minutes and I will rebase the branch so that it incorporates everything from master.  That should make the diff easier to interpret.  Sorry for the confusion.


(Scott Daniel) the bulk of this branch is in the creation of testVariability.py

Because I had to touch Photometry.py a little bit, I took the opportunity to clean up the trailing white space that was hanging around everywhere.


(Scott Daniel) Okay: I have rebased to master, so the diff should be easier to interpret.


(Heather Kelly) Thanks, though we know any confusion is due to my taking too long to finish this off :)
I'm just going to try to run the new test and then I should be ready to sign off!


(Heather Kelly) Should I be surprised that if I do:
bash-4.1$  eups distrib install sims_photUtils
I get an error?
eups distrib: Trouble reading manifest for sims_photUtils master-g3e723a404c+6 (generic): First line of manifest file /tmp/eups/distrib/sims_photUtils_a8HVBi is corrupted:
        <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">


(Scott Daniel) to get the sims package you need to install the version tagged as 'sims'. Try

eups distrib install sims_photUtils -t sims

if that doesn't work

eups distrib install lsst_sims -t sims

will install the full sims package (though this will involve installing afw, which can be time consuming)


(Heather Kelly) ran testPhotometry.py:
Ran 8 tests in 12.614s

OK


 OSU!
A rhythm music game where the user moves the mouse and click on screen beats in sync with the audio.
Rhythm is just a "click" away!
Created in Java


(Andrew Connolly) Feature/CATSIM-105 request to store mflare variability
Change mflares input to accept gzip files

This will require gzipping  sims_sed_library/2014.04.23/mflare/* and repackaging the seds before release

We should do this before too many people install last_sims with maf


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.360](http://teamcity.monogame.net/viewLog.html?buildId=6409&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 484, ignored: 6 Build time: 00:15:06


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


(Scott Daniel) looks good to merge


(Scott Daniel) Bugfix/CATSIM-142 add all to modules
This branch adds the __all__= [] formalism to the python source code

* for some reason, Variability was still trying to import compound
from sims_coordUtils.  Changed that so that it imports it from
sims_catalogs_measures, where compound actually lives
* changed EbvMap to EBVmap to be consistent with other classes in EBV.py
* updated __init__.py
* updated import statement
* added __all__ to files in sims_photUtils


(Yusra AlSayyad) `sims_catUtils/examples/baseValidation/utils.py` refers to `EBV.EbvMap` in lines 114 and 117, which would have to be changed before this goes through.  It's really hard, but DM best practices are to resist the urge to fix problems you find while fixing the primary problem on the Issue.


(Scott Daniel) Nice catch (I didn't even know that code existed).  I fixed it.

Incidentally: I feel like I overheard some discussions while you were away that the new policy is "go ahead an make incidental changes, just make sure that they are on their own commit."  (You probably shouldn't quote me on that, though).  There has been a lot of focus lately on having orderly commit histories.


Thanks, that makes sense!


(Scott Daniel) Bugfix/CATSIM-139 unittest variable scope
This pull request combines two issues:

CATSIM-139 in which we use setUp and tearDown to set and unset unittest instance variables so as to avoid collisions between different unittests

and

CATSIM-115 in which we demand that unittests clean up after themselves so that unnecesssary test outputs don't collect in whatever directory is running the unittests

* cleaned up whitespace in testApplyIGM.py
* changed name of test database in testPhotometry.py
* implemented setUpClass and tearDownClass in testPhotometry.py
* methods in testPhotometry.py that write catalogs now clean up after themselves
* cleaned up whitespace in testPhotometry.py
* testReadGalfast.py now cleans up after itself
* cleaned up whitespace in testReadGalfast.py
* added tearDownClass to testReadGalfast.py
* added to tearDown in testSed.py


(Peter Yoachim) Maybe the unlink should go in the tearDown so that it will get deleted even if something else fails in the test?


(Peter Yoachim) same, in teardown?


(Peter Yoachim) teardowns?